Vered Volansky has posted comments on this change.

Change subject: webadmin: Add host SPM priority value in host general tab
......................................................................


Patch Set 3: Looks good to me, but someone else must approve

(1 inline comment)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostGeneralModel.java
Line 257:     }
Line 258: 
Line 259:     public void setSpmPriorityValue(Integer spmPriorityValue) {
Line 260:         if (this.spmPriorityValue == null || 
!this.spmPriorityValue.equals(spmPriorityValue)) {
Line 261:             this.spmPriorityValue = spmPriorityValue;
You showed me :)
Line 262: 
Line 263:             if (spmMaxPriorityValue == null || 
spmDefaultPriorityValue == null) {
Line 264:                 retrieveMaxSpmPriority();
Line 265:             }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9dc7b21ccb893b6d27443482b650659165dd40b6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to