Dan Kenigsberg has submitted this change and it was merged. Change subject: tests: FakeLVM add lvPath test ......................................................................
tests: FakeLVM add lvPath test Test the lvPath function. Allow FakeLVM to be created without a temporary directory when it's not needed. In that case, lvPath will return values just like real LVM: /dev/<vg_name>/<lv_name> . Change-Id: I46be518e68069412199dfb7ee44c7b7a8a207236 Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/47863 Reviewed-by: Nir Soffer <nsof...@redhat.com> Tested-by: Nir Soffer <nsof...@redhat.com> Continuous-Integration: Jenkins CI --- M tests/storagefakelibTests.py 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Nir Soffer: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47863 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I46be518e68069412199dfb7ee44c7b7a8a207236 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches