anmolbabu has posted comments on this change. Change subject: engine,webadmin: Add an indicator if geo-replication is enabled ......................................................................
Patch Set 5: Code-Review+1 (2 comments) 1 small string modification suggestion otherwise looks good to me. https://gerrit.ovirt.org/#/c/38310/5/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/VolumeInfoCell.java File frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/VolumeInfoCell.java: Line 13: import com.google.gwt.safehtml.shared.SafeHtmlBuilder; Line 14: import com.google.gwt.safehtml.shared.SafeHtmlUtils; Line 15: import com.google.gwt.user.client.ui.AbstractImagePrototype; Line 16: Line 17: public class VolumeInfoCell extends AbstractCell<GlusterVolumeEntity> { > Activities is more for ongoing tasks, here it's about features being enable Done Line 18: Line 19: private static final ApplicationResources resources = GWT.create(ApplicationResources.class); Line 20: private static final ApplicationConstants constants = GWT.create(ApplicationConstants.class); Line 21: private static final ApplicationTemplates applicationTemplates = GWT.create(ApplicationTemplates.class); Line 31: if (volume == null) { Line 32: return; Line 33: } Line 34: if (volume.getIsGeoRepMaster()) { Line 35: SafeHtml geoRepMasterHtml = > this doesn't take care of the case where a volume could be both master and Done Line 36: SafeHtmlUtils.fromTrustedString(AbstractImagePrototype.create(geoRepMasterImage).getHTML()); Line 37: sb.append(applicationTemplates.statusTemplate(geoRepMasterHtml, constants.geoRepMasterVolumeToolTip())); Line 38: } Line 39: if (volume.getIsGeoRepSlave()) { -- To view, visit https://gerrit.ovirt.org/38310 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I28178b8ef03ea7cc83097cb99bc2d2be90061434 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com> Gerrit-Reviewer: anmolbabu <anb...@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
