Shubhendu Tripathi has posted comments on this change. Change subject: gluster: Gluster volume snapshot actions with georep case ......................................................................
Patch Set 6: (2 comments) https://gerrit.ovirt.org/#/c/38150/6/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteAllGlusterVolumeSnapshotsCommand.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteAllGlusterVolumeSnapshotsCommand.java: Line 127: > May be we can allow this and warning message before delete action can tell As part of https://gerrit.ovirt.org/#/c/38469/ will add mechanism to ignore georep based on user's input https://gerrit.ovirt.org/#/c/38150/6/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGlusterVolumeSnapshotCommand.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGlusterVolumeSnapshotCommand.java: Line 100: } Line 101: Line 102: for (GlusterGeoRepSession session : georepSessions) { Line 103: if (session.getSlaveVolumeId() == null || session.getSlaveNodeUuid() == null) { Line 104: return failCanDoAction(VdcBllMessages.ACTION_TYPE_FAILED_REMOTE_CLUSTER_NOT_MAINTAINED_BY_ENGINE); > May be we can allow with proper warning in confirmation dialog As part of https://gerrit.ovirt.org/#/c/38469/ will add mechanism to ignore georep based on user's input Line 105: } Line 106: } Line 107: Line 108: return true; -- To view, visit https://gerrit.ovirt.org/38150 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0eccbcd4aa3e218ba0d910bd9150bdb0baa2db68 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtri...@redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
