Change in vdsm[master]: Add migration network support.

danken at redhat.com danken at redhat.com
Tue Mar 19 17:01:44 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: Add migration network support.
......................................................................


Patch Set 3: Verified

(1 inline comment)

we've just reviewed this patch with ALona's code. We had one dubious libvirt crash, but besides that, the proper arguments are passed to libvirt and qemu below.

....................................................
File vdsm/libvirtvm.py
Line 478:                        self._vm.conf['_migrationParams']['dstqemu']
Line 479:             else:
Line 480:                 muri = 'tcp://%s' % self.remoteHost
Line 481: 
Line 482:             self._vm.log.debug('starting migration to %s', duri)
would be nicer to log 'muri'
Line 483: 
Line 484:             t = MigrationDowntimeThread(self._vm, int(self._downtime),
Line 485:                                         self._vm._migrationTimeout() / 2)
Line 486: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I17d760c7421e765454e13ee5b7d8be23e6b7e383
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list