Change in vdsm[master]: vdsm-reg: move OK message to caller.

dougsland at redhat.com dougsland at redhat.com
Thu May 2 22:52:42 UTC 2013


Douglas Schilling Landgraf has posted comments on this change.

Change subject: vdsm-reg: move OK message to caller.
......................................................................


Patch Set 7: (1 inline comment)

....................................................
File vdsm_reg/vdsm-complete
Line 71:             fOK = deployUtil.cleanAll(rnum)
Line 72: 
Line 73:         if fOK:
Line 74:             try:
Line 75:                 fOK = deployUtil.setVdsConf(vds_config_str, VDSM_CONF_FILE)
I have removed that but looks like, not in this commit :-/
Line 76:                 print "<BSTRAP component='VDS Configuration' status='OK'/>"
Line 77:             except Exception, err:
Line 78:                 fOK = False
Line 79:                 print "<BSTRAP component='VDS Configuration' status='FAIL' " \


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifaea56ca3f9f9a4f5e60c684dae9ae1f17f63a1e
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list