Omer Frenkel has posted comments on this change.

Change subject: core: minor cleanup in VdsUpdateRunTimeInfo
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
Line 1161:             if (vm.getstatus() != VMStatus.Down) {
Line 1162:                 continue;
Line 1163:             }
Line 1164: 
Line 1165:             // _vdsManager.getVm(vm.getvm_guid());
this comment can be removed
Line 1166:             VM vmTo = _vmDict.get(vm.getId());
Line 1167:             VMStatus status = VMStatus.Unassigned;
Line 1168:             if (vmTo != null) {
Line 1169:                 status = vmTo.getStatus();


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I21cc745d9034cbf97b1d39ad65a96c30c01e7dd2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to