Yaniv Bronhaim has posted comments on this change.

Change subject: engine: Improved duplicate host validation
......................................................................


Patch Set 1:

Both solutions require only one request for the host UUID before performing the 
installation. We can initiate the getVdsId command also as part of 
addVds.canDoAction and fail it there if the id already assigned, if the call 
fails we can assume the host is not responsive or not installed, and we can 
start the deploy process

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I37c5ee3546a21400678c2f1f7d1e54bb726988b3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Barak Azulay <[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