Francesco Romani has posted comments on this change. Change subject: virt: Initial support for post-copy migration ......................................................................
Patch Set 3: -Code-Review (2 comments) https://gerrit.ovirt.org/#/c/62873/2/vdsm/virt/migration.py File vdsm/virt/migration.py: Line 669: Line 670: now = time.time() Line 671: if self._in_post_copy: Line 672: self._vm.log.debug('Post-copy migration still in progress: %d', Line 673: progress.data_remaining) > To be clear: We need to fetch stats, progress is available here. OK, at this stage "progress" is probably the only thing we need. Line 674: elif not self._use_conv_schedule and\ Line 675: (0 < migrationMaxTime < now - self._startTime): Line 676: self._vm.log.warn('The migration took %d seconds which is ' Line 677: 'exceeding the configured maximum time ' Line 691: ' > lowmark (%sMiB).' Line 692: ' Refer to RHBZ#919201.', Line 693: progress.data_remaining / Mbytes, lowmark / Mbytes) Line 694: Line 695: if not self._in_post_copy and\ > Actually not: Post-copy migration, once successfully started, can finish in OK, this makes sense: once we attempt "postcopy", there is no way back. So, "postcopy" effectively marks the end of one convergence schedule (everything else that may follow is ignored). I'm fine with that, we just need to document this. It could be a good idea to document somewhere (Vdsm or Engine) the convergence schedule actions. Line 696: lastDataRemaining is not None and\ Line 697: lastDataRemaining < progress.data_remaining: Line 698: iterationCount += 1 Line 699: self._vm.log.debug('new iteration detected: %i', -- 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: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@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 https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org