Francesco Romani has posted comments on this change. Change subject: caps: replacing minidom with ElementTree in caps.py ......................................................................
Patch Set 9: Code-Review-1 (1 comment) same as my comments in v6. I don't really like the iterator-based API used here. -1 for discussion. If we *really* want to use this API I can deal with that, we bear far worse horrors in vm.py. But still, I don't like it. https://gerrit.ovirt.org/#/c/41078/9/vdsm/caps.py File vdsm/caps.py: Line 207: if capabilities is None: Line 208: capabilities = _getFreshCapsXMLStr() Line 209: Line 210: caps = ET.fromstring(capabilities) Line 211: host = caps.iter(tag='host').next() the problem here is that I don't really like this iterator-based API. I _really_ prefer find()/findall(), and I really believe this will lead to more readable code. Line 212: cells = host.iter(tag='cells').next() Line 213: Line 214: sockets = set() Line 215: siblings = set() -- To view, visit https://gerrit.ovirt.org/41078 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8c1ca58807515922347e2255c77b4f950decd619 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
