Omer Frenkel has posted comments on this change. Change subject: gluster: Modified to fail remove host if already removed from CLI ......................................................................
Patch Set 2: (1 comment) i think org.ovirt.engine.ui.frontend.VdsmErrors.java should be modified as well .................................................... File backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties Line 347: GlusterVolumeProfileStartFailed=Gluster Volume Profile Start Failed Line 348: GlusterVolumeProfileStopFailed=Gluster Volume Profile Stop Failed Line 349: GlusterAddHostFailed=Gluster Server Add Failed Line 350: GlusterHostRemoveFailedException=Gluster Host Remove Failed Line 351: GlusterHostRemoveFailed=Gluster Server Remove Failed so this one above is relevant? i think not Line 352: GlusterPeerListFailed=Gluster Peer List Failed Line 353: GlusterVolumeStatusFailed=Could not fetch advanced volume details Line 354: GlusterVolumeProfileInfoFailed=Gluster Volume Profile Info Failed Line 355: GlusterHookFailed=Gluster hook operation failed -- To view, visit http://gerrit.ovirt.org/20655 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1e8c79bcbf9c3dc733641812f505170e369448a1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
