Roy Golan has posted comments on this change.

Change subject: core: rerun Maintenace to host when preparing is IDLE
......................................................................


Patch Set 4:

(1 comment)

....................................................
File 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
Line 1856:                         "Updated vds status from 'Preparing for 
Maintenance' to 'Maintenance' in database,  vds = {0} : {1}",
Line 1857:                         _vds.getId(),
Line 1858:                         _vds.getName());
Line 1859:             } else {
Line 1860:                 if (System.currentTimeMillis() - 
_vdsManager.getStatusChangedAt() > Config.<Integer> 
GetValue(ConfigValues.HostPreparingForMaintenanceIdleTime)) {
not good yet. still need to understand what is the consoquence of goining to 
maintenance here and how to make the the next VURTI see this and not fire this 
again.

Omer what do you think?
Line 1861:                     // try to put the host to Maintenance again.
Line 1862:                     
ResourceManager.getInstance().getEventListener().runningVMsOnHostPreparingToMaintenance(_vds);
Line 1863:                 }
Line 1864:             }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0e18bcfbc566b6fd92d276c5c739ba607e2a53f0
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
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

Reply via email to