Change in vdsm[master]: virt: Vm._getUnderlyingSoundDeviceInfo() moved to core.py

fromani at redhat.com fromani at redhat.com
Fri Feb 19 08:23:12 UTC 2016


Francesco Romani has posted comments on this change.

Change subject: virt: Vm._getUnderlyingSoundDeviceInfo() moved to core.py
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/53616/2/vdsm/virt/vmdevices/core.py
File vdsm/virt/vmdevices/core.py:

Line 252:         :type domain: :class:`DomainDescriptor` instance
Line 253:         :param sound_devices: current sound devices for update
Line 254:         :type sound_devices: dictionary
Line 255:         :param device_conf: current device configuration for update
Line 256:         :type device_conf: dictionary
I believe it is sufficient if you have this doc only in the base method, to avoid repeating -and keeping it up to date- on each the subclasses.
Line 257: 
Line 258:         """
Line 259:         for x in domain.get_device_elements('sound'):
Line 260:             alias = x.getElementsByTagName('alias')[0].getAttribute('name')


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I05cb24e0a9776e21bd2546d62a56554d74559c61
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list