Douglas Schilling Landgraf has posted comments on this change.

Change subject: is-configured pre-start task needs to manipulate upgrade of 
ovirt-node
......................................................................


Patch Set 2:

IMO, If you want to use the current vdsm-tool verb for this we must change the 
logic to: "if there is a difference in the conf files and it's ovirt-node 
blindly reconfigure vdsm". My opinion adding script into ovirt-node to 
workaround a vdsm problem should be considered a hack too.

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

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

Reply via email to