From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: metadata: get rid of _XMLWrapper ......................................................................
virt: metadata: get rid of _XMLWrapper
We added _XMLWrapper out of necessity, because we possibly need to *replace* (or at very least update) the object returned by the interal _domain_xml context manager.
In this patch we reduce the amount of cerimony replacing the class with much simpler plain list wrapper.
Also, we rename the _domain_xml helper to a more correct _metadata_xml. No changes in behaviour for the clients of the metadata module, tests included.
Change-Id: I2e454495b4189e1656f9357e1d26bda105c7ad50 Signed-off-by: Francesco Romani fromani@redhat.com --- M lib/vdsm/virt/metadata.py 1 file changed, 12 insertions(+), 26 deletions(-)
Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified Martin Polednik: Looks good to me, but someone else must approve
vdsm-patches@lists.fedorahosted.org