Change in vdsm[master]: virt: Lookup conf and device by path

Federico Simoncelli fsimonce at redhat.com
Tue Jun 10 14:27:38 UTC 2014


Federico Simoncelli has posted comments on this change.

Change subject: virt: Lookup conf and device by path
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.ovirt.org/#/c/27951/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:

Line 3164:         return {'status': doneCode, 'vmList': self.status()}
Line 3165: 
Line 3166:     def _lookupDeviceByAlias(self, devType, alias):
Line 3167:         for dev in self._devices[devType][:]:
Line 3168:             try:
(...continuing) It probably belongs to the extension patch then.
Line 3169:                 if dev.alias == alias:
Line 3170:                     return dev
Line 3171:             except AttributeError:
Line 3172:                 continue


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib5f63715cd5142ffaf21ae9ba8d451175ae09be0
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpoledni at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr 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