Omer Frenkel has posted comments on this change. Change subject: gluster: Modified to fail remove host if already removed from CLI ......................................................................
Patch Set 1: (1 comment) .................................................... File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java Line 375: GlusterVolumeProfileStopFailed(4159), Line 376: GlusterVolumeProfileInfoFailed(4160), Line 377: GlusterVolumeStatusAllFailedException(4161), Line 378: GlusterAddHostFailed(4404), Line 379: GlusterHostRemoveFailedException(4406), i think the word "exception" is not needed (or missing in vdsmErrors.props file) Line 380: GlusterPeerListFailed(4407), Line 381: GlusterHostUUIDNotFound(4408), Line 382: GlusterServicesListFailed(4409), Line 383: GlusterHookFailed(4500), -- 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: 1 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
