Change in vdsm[master]: vdsm: Support guest agent selective ip reporting

danken at redhat.com danken at redhat.com
Wed Dec 19 10:54:39 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: vdsm: Support guest agent selective ip reporting
......................................................................


Patch Set 2: I would prefer that you didn't submit this

(2 inline comments)

minor style comment

....................................................
Commit Message
Line 5: CommitDate: 2012-12-19 01:14:13 +0200
Line 6: 
Line 7: vdsm: Support guest agent selective ip reporting
Line 8: 
Line 9: The guest agent might not report inet element,
could you explain why the guest agent does not report an empty list in this case? is it an agent bug?
Line 10: therefore processing its input should support it and
Line 11: return an emply ips list instead.
Line 12: 
Line 13: Change-Id: I5264058737cc2b53b9bfa22d344bf26c8d269f87


....................................................
File vdsm/guestIF.py
Line 136:             for iface in args['interfaces']:
Line 137:                 iface['name'] = _filterXmlChars(iface['name'])
Line 138:                 iface['hw'] = _filterXmlChars(iface['hw'])
Line 139:                 if 'inet' in iface:
Line 140:                     iface['inet'] = map(_filterXmlChars, iface['inet'])
I prefer the simpler syntax of

 iface['inet'] = map(_filterXmlChars, iface.get('inet', []))
Line 141:                 else:
Line 142:                     iface['inet'] = []
Line 143:                 if 'inet6' in iface:
Line 144:                     iface['inet6'] = map(_filterXmlChars, iface['inet6'])


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5264058737cc2b53b9bfa22d344bf26c8d269f87
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list