Omer Frenkel has posted comments on this change.

Change subject: engine: Delete Gluster Volume
......................................................................


Patch Set 3: Looks good to me, approved

(1 inline comment)

well it looks good to me but since this patch include also rest and ui changes, 
you should get ack from them before we can merge.

....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
Line 264:     DeleteGlusterVolume(1403, ActionGroup.DELETE_GLUSTER_VOLUME),
cool, i missed that, im ok with that.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ide0d89c7100d787a4c66c29b2c1cc0832c494b33
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Selvasundaram <[email protected]>
Gerrit-Reviewer: Dhandapani Gopal <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Selvasundaram <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to