[node-patches] Change in ovirt-node[master]: utils: Fix subprocess.check

mburns at redhat.com mburns at redhat.com
Tue Jun 25 11:46:23 UTC 2013


Michael Burns has submitted this change and it was merged.

Change subject: utils: Fix subprocess.check
......................................................................


utils: Fix subprocess.check

The python_26 call was use dincorrectly and always returned True
(because the existance of the function was checked, becauuse the () were
missed).

Change-Id: Ic980f637a689610c498a883fc2ea11260c48e7ee
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/process.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved


-- 
To view, visit http://gerrit.ovirt.org/16071
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic980f637a689610c498a883fc2ea11260c48e7ee
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list