[node-patches] Change in ovirt-node[master]: process: Raise an error for a known problem
fabiand at fedoraproject.org
fabiand at fedoraproject.org
Thu Sep 19 09:09:11 UTC 2013
Fabian Deutsch has submitted this change and it was merged.
Change subject: process: Raise an error for a known problem
......................................................................
process: Raise an error for a known problem
It is known that combining shell=True with an argumentlist doesn't work.
Thsi check is now added to call subprocess calls.
Change-Id: Ic0208de30aa5b8312384c0736f6f7ad38624ec89
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/process.py
1 file changed, 12 insertions(+), 0 deletions(-)
Approvals:
Fabian Deutsch: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/19225
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic0208de30aa5b8312384c0736f6f7ad38624ec89
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the node-patches
mailing list