Tal Nisan has posted comments on this change.

Change subject: core,webadmin: new VM-Template - changeable disk alias
......................................................................


Patch Set 3: Looks good to me, approved

Seems ok to me, only one question, it seems that in any case the disk alias is 
changeable, any reason to make the check in so many places when it's always 
allowed?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5a6a47d1ba77eed84fa5561db342e03ab6bac81a
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Asaf Shakarchi <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to