Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : remove-brick status error handling ......................................................................
webadmin : remove-brick status error handling Errors like, 1. If some/all the hosts forming a distribute volume with atleast 2 bricks are down, and if rebalance status is triggered on such a volume, approprite error message is displayed. 2. If any n/w related or any other exceptions are thrown,they are also handled properly. And hence the popup doesn't hang anymore. Change-Id: I57bb82ebc0318266f32262c6f9bb245e26f2ddd1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1027675 Signed-off-by: Anmol Babu <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/RemoveBrickStatusModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java 4 files changed, 58 insertions(+), 33 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21086 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I57bb82ebc0318266f32262c6f9bb245e26f2ddd1 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Ramesh N <[email protected]> Gerrit-Reviewer: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: anmolbabu <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
