Douglas Schilling Landgraf has posted comments on this change.

Change subject: vdsm-upgrade: adds wrapper to ovirt-node-upgrade
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.ovirt.org/#/c/28244/2/vdsm_reg/vdsm-upgrade
File vdsm_reg/vdsm-upgrade:

Line 41: 
Line 42:     logging.debug(out)
Line 43:     logging.debug(msg)
Line 44:     print(msg)
Line 45:     return rc
> Douglas, <BSTRAP> element has a "message" attribute. I think that Alon sugg
Thanks for the clarification Dan. I have pinged Alon before that over IRC,  he 
also said that.
Line 46: 
Line 47: if __name__ == "__main__":


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b997d70a440545497246d1a19d9671b054a56a5
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Joey Boggs <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to