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