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