Kanagaraj M has posted comments on this change.

Change subject: webadmin: Fix for NPE in remove brick status dialog
......................................................................


Patch Set 2: Code-Review+2

(1 comment)

minor comment

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeRebalanceStatusModel.java
Line 121:             }
Line 122:         }
Line 123:         if 
(GlusterTaskType.REBALANCE.equals(getEntity().getAsyncTask().getType())) {
Line 124:             
getStopReblanceFromStatus().setIsExecutionAllowed(rebalanceStatusEntity.getStatusSummary().getStatus()
 == JobExecutionStatus.STARTED);
Line 125:         }
change to == instead of equals
Line 126:     }
Line 127: 
Line 128:     public void cancelRefresh() {
Line 129:         if (refresh != null) {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibfe886c240a6d55798a1ad021c58dc874b8cff12
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Ramesh N <[email protected]>
Gerrit-Reviewer: anmolbabu <[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

Reply via email to