Change in vdsm[master]: vdsm: Accents in the vm name break the migration
fsimonce at redhat.com
fsimonce at redhat.com
Tue Oct 8 09:19:42 UTC 2013
Federico Simoncelli has posted comments on this change.
Change subject: vdsm: Accents in the vm name break the migration
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Decoding is already taken care of in the regular flow.
....................................................
File vdsm/vm.py
Line 4876: self._pathsPreparedEvent.wait(prepareTimeout)
Line 4877: if not self._pathsPreparedEvent.isSet():
Line 4878: self.log.debug('Timeout while waiting for path preparation')
Line 4879: return False
Line 4880: srcDomXML = self.conf.pop('_srcDomXML').encode('utf-8')
nit: adding a comment
Line 4881: self._updateDevicesDomxmlCache(srcDomXML)
Line 4882:
Line 4883: for dev in self._customDevices():
Line 4884: hooks.before_device_migrate_destination(
--
To view, visit http://gerrit.ovirt.org/19684
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9db5ea4f0745ef42633e3d265bcf40fb4733b2ce
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpoledni at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list