Change in vdsm[master]: migration: Use jsonrpc during migration of vms

michal.skrivanek at redhat.com michal.skrivanek at redhat.com
Sun Apr 12 13:49:05 UTC 2015


Michal Skrivanek has posted comments on this change.

Change subject: migration: Use jsonrpc during migration of vms
......................................................................


Patch Set 6:

(1 comment)

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

Line 132:             client_socket = self._createSocket(port)
Line 133:             self._destServer = jsonrpcvdscli.connect(self._vm.cif,
Line 134:                                                      client_socket)
Line 135:             self.log.debug('Initiating connection with destination')
Line 136:             self._destServer.ping()
just please confirm that the jsonrpcvdscli.connect() doesn't actually connect to the other side. If yes then we can drop the ping(), if not and this is the only way how to figure out the connection is established then we can keep the ping()
Line 137: 
Line 138:         except Exception:
Line 139:             if config.getboolean('vars', 'ssl'):
Line 140:                 self._destServer = vdscli.connect(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie38334e6cdcc4d7899bd5e836b3196567fc0bfd8
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list