Francesco Romani has posted comments on this change.

Change subject: virt: Initial support for post-copy migration
......................................................................


Patch Set 6:

(1 comment)

https://gerrit.ovirt.org/#/c/62873/6/vdsm/virt/migration.py
File vdsm/virt/migration.py:

Line 480:                          self._compressed else 0) |
Line 481:                      (libvirt.VIR_MIGRATE_AUTO_CONVERGE if
Line 482:                          self._autoConverge else 0))
Line 483:             if self._use_convergence_schedule:
Line 484:                 # TODO: Replace this whole block with simple flag 
setting once
> I'm not sure Vdsm is already capable of running on Fedora 24, but OK, this 
Vdsm *should* be run in F24, or, to word it differently, I'm not aware of any 
bug in this regard.
Line 485:                 # Vdsm depends on proper versions of QEMU and libvirt.
Line 486:                 try:
Line 487:                     flags |= libvirt.VIR_MIGRATE_POSTCOPY
Line 488:                 except AttributeError:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4c9911f47331120a1b78326044a4949abf35d5fe
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to