Yaniv Bronhaim has posted comments on this change.

Change subject: vdsm: packages: adjust to new method of vdsm configure
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

....................................................
File src/plugins/ovirt-host-deploy/vdsm/packages.py
Line 142:             rc, stdout, stderr = self.execute(
Line 143:                 (
Line 144:                     vdsm_tool,
Line 145:                     'configure',
Line 146:                     '--force',
old comment. please ignore - `vdsm-tool configure --force` is the right call to 
reconfigure all needed confs (assuming restart of external services and 
override all related conf files are acceptable in that scope)
Line 147:                 ),
Line 148:                 raiseOnError=False,
Line 149:             )
Line 150:             if rc == 0:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iec69d6ab5cdd00132198f882b0f1d0a1ff790f8b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to