Francesco Romani has submitted this change and it was merged. Change subject: Live Merge: Remove volume run link after live merge ......................................................................
Live Merge: Remove volume run link after live merge When deleting a volume while the VM is running, volume teardown doesn't remove the volume run symbolic link: /run/vdsm/storage/sdUUID/volUUID. In patch Iec3b6a (Live Merge: teardown volume on HSM after live merge) we added volume teardown logic that, for block storage it deactivated the volume. In this patch we extend volume teardown logic to unlink volume run link. Note that this change isn't required for file storage as no symbolic links are created. Change-Id: Ib88bf92e702ac6c324b87c9459b01adf165eaca4 Bug-Url: https://bugzilla.redhat.com/1321018 Signed-off-by: Ala Hino <ah...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/59725 Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Adam Litke <ali...@redhat.com> Continuous-Integration: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/64986 Reviewed-by: Francesco Romani <from...@redhat.com> --- M vdsm/storage/blockSD.py 1 file changed, 15 insertions(+), 0 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve; Passed CI tests Francesco Romani: Looks good to me, approved Ala Hino: Verified Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/64986 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib88bf92e702ac6c324b87c9459b01adf165eaca4 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org