Milan Zamazal has posted comments on this change.

Change subject: virt: vm: Update time on VM after resume
......................................................................


Patch Set 7:

The delay doesn't make the VM unavailable (e.g. there should be no disruption 
during migration because of it), it just shifts finishing the operation. I can 
think about the following options:

1. Move syncGuestTime to later stage of the operation (if needed). Still delays 
the response, maybe some hooks or other finalization operations.
2. Utilizing some thread. Doesn't block anything, but may be inconvenient or 
dangerous?
3. Convincing libvirt developers to resolve that some way (any chance of 
success?).

Let's first ask libvirt developers about the problem. Then we can think further.

-- 
To view, visit https://gerrit.ovirt.org/48860
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ieb583cd5d21e56d7730b0ba21d75ed93b9d34025
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskri...@redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to