Omer Frenkel has posted comments on this change.

Change subject: core: run HA VMs that went down right before engine stopped
......................................................................


Patch Set 6: Code-Review+1

(1 comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoStartVmsRunner.java
Line 52:         // take all the HA VMs which are down because of an error and 
add them to the set
Line 53:         final DateTime now = DateTime.getNow();
Line 54:         LinkedList<Pair<Guid, DateTime>> initialFailedVms = new 
LinkedList<>();
Line 55:         for (VM vm: getVmDao().getAllFailedAutoStartVms()) {
Line 56:             log.infoFormat("Found HA VM which is down because of an 
error, trying to restart the VM, VM id: {0}", vm.getId());
log also vm name
Line 57:             initialFailedVms.add(new Pair<>(vm.getId(), now));
Line 58:         }
Line 59:         autoStartVmsToRun.addAll(initialFailedVms);
Line 60:     }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2d5876f196819b2a69be0b71287c5325a8ff9dc9
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to