Alon Bar-Lev has posted comments on this change.

Change subject: Validate installed host by requesting for vdsm uuid
......................................................................


Patch Set 3:

Yair,

I prefer not to include this in vdc_options and remove the warning [if exists].

I also don't think you are correct about the cluster level, the fact that vdsm 
is backward compatible in API does not mean we cannot attempt to use the new 
feature (get the id). It will not break anything and reduce the chance we have 
a solution.

Thanks!

Alon

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c6c0a85daba47fabb9253963ff187a670f28ae6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to