Martin Polednik has posted comments on this change.
Change subject: virt: add test to allow custom XML parsing test
......................................................................
Patch Set 2:
I would be careful with just taking everything under data directory and trying to parse it - we could eventually use some command line data etc. - what I personally would like to see is an additional wrapper which can be given xml/file and then runs this verification against it.
--
To view, visit
https://gerrit.ovirt.org/39909
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3bb24e2854e6f7b93c5108eca8b6f79f17353e85
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik
mpolednik@redhat.com
Gerrit-Reviewer: Dan Kenigsberg
danken@redhat.com
Gerrit-Reviewer: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Ido Barkan
ibarkan@redhat.com
Gerrit-Reviewer: Martin Polednik
mpolednik@redhat.com
Gerrit-Reviewer: automation@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No