Change in vdsm[master]: vdsm: fix Vm cleanup routine

peet at redhat.com peet at redhat.com
Fri Apr 5 08:34:21 UTC 2013


Peter V. Saveliev has posted comments on this change.

Change subject: vdsm: fix Vm cleanup routine
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/libvirtvm.py
Line 1875:                            'message': e.message}}
Line 1876:         else:
Line 1877:             hooks.after_disk_hotunplug(driveXml, self.conf,
Line 1878:                                        params=params.get('custom', {}))
Line 1879:             self._cleanupDrives()
I tried not to change the previous logic so far, but what you said was exactly what I wanted to do. So thanks, I will do it.
Line 1880: 
Line 1881:         return {'status': doneCode, 'vmList': self.status()}
Line 1882: 
Line 1883:     def _readPauseCode(self, timeout):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie355022866d008a83257b1b6e3a4226a07d34c2f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken 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


More information about the vdsm-patches mailing list