Francesco Romani has posted comments on this change. Change subject: Replacing minidom with ElementTree in caps.py ......................................................................
Patch Set 5: (2 comments) I'd like to see (not necessarely from yours, I can play a bit on my own) how it looks with find(). https://gerrit.ovirt.org/#/c/41078/5//COMMIT_MSG Commit Message: Line 3: AuthorDate: 2015-05-18 20:57:49 +0200 Line 4: Commit: mmirecki <[email protected]> Line 5: CommitDate: 2015-05-20 18:48:49 +0200 Line 6: Line 7: Replacing minidom with ElementTree in caps.py please reformat the summary line: caps: replacing ... Line 8: Line 9: minidom is replaced by ElementTree.py in caps.py. This is because ET is Line 10: faster, and we want to elinate using two similar libs in the code Line 11: https://gerrit.ovirt.org/#/c/41078/5/debian/vdsm-tests.install File debian/vdsm-tests.install: Line 7: usr/share/vdsm/tests/caps_libvirt_intel_i73770.out Line 8: usr/share/vdsm/tests/caps_libvirt_intel_i73770_nosnap.out Line 9: usr/share/vsdm/tests/caps_numactl_4_nodes.out Line 10: usr/share/vdsm/tests/cpu_info.out Line 11: usr/share/vdsm/tests/cpu_map.xml > The file was added in the previous patch: https://gerrit.ovirt.org/#/c/4107 yep, please move this change in the previous patch. Line 12: usr/share/vdsm/tests/devices/parsing*.py Line 13: usr/share/vdsm/tests/devices/data/*.xml Line 14: usr/share/vdsm/tests/functional/*.policy Line 15: usr/share/vdsm/tests/functional/*.py -- 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: 5 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
