Arik Hadas has posted comments on this change.

Change subject: core: trigger auto-start VMs runner job after live snapshot 
operation
......................................................................


Patch Set 2:

(1 comment)

this patch is currently on hold as I'm checking other solution (Allon, sorry 
for not mention it before)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
Line 257:         // Need to release the lock before triggering the auto-start 
VMs runner job
Line 258:         freeLock();
Line 259:         // This command might be long and while it is being executed 
the VM cannot be restarted
Line 260:         // so we're triggering the job that restarts HA VMs when it 
is finished so that if
Line 261:         // the VM is HA and it failed in the process, we'll try to 
run it immediately
then we would need to wait for the next iteration of the quartz job which is 
currently set to 1 minute.
Line 262:         Backend.getInstance().triggerAutoStartVmsRunnerJob();
Line 263:     }
Line 264: 
Line 265:     private void logMemorySavingFailed() {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2c6403d69d0b6725d8a1d5e065620d42cb8dfa64
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@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