Yaniv Bronhaim has posted comments on this change.

Change subject: engine: check if vds exists with same name
......................................................................


Patch Set 3: Verified; Looks good to me, but someone else must approve

It looks fine.. I played with that a little more , I don't know what was wrong 
with my environment before. you have condition that checks if you update the 
name before reading it from db, so if you reinstall, update.canDoAction won't 
check db at all, the name stays the name..

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I59bea31c78da3f718497bd1b58bada21c744fdc7
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[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