Dan Kenigsberg has submitted this change and it was merged. Change subject: tests/functional: Use deleteVolume instead of deleteImage ......................................................................
tests/functional: Use deleteVolume instead of deleteImage Since we are creating a volume, the undo/rollback should do deleteVolume instead of deleteImage. deleteImage does not work well for derived filedomains (eg. glusterSD) as it does not honor the volumeClass associated with the derived domain, so its not possible to write a test using deleteImage for glusterSD. Technically too, deleteVolume fits the current flow of testcase better than deleteImage as there isn't a chain of volumes being created to be really justified to use deleteImage Change-Id: Id9b8f598fcad03b689e14d09ab33c98ec1626f3d Signed-off-by: Deepak C Shetty <[email protected]> --- M tests/functional/xmlrpcTests.py 1 file changed, 6 insertions(+), 5 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Deepak C Shetty: Verified -- To view, visit http://gerrit.ovirt.org/10195 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9b8f598fcad03b689e14d09ab33c98ec1626f3d Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Deepak C Shetty <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Deepak C Shetty <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
