Federico Simoncelli has uploaded a new change for review. Change subject: lvm: refresh on mda permission mismatch ......................................................................
lvm: refresh on mda permission mismatch Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=905665 Change-Id: I6a77b967a057329a90499d7707074befe756b68a Signed-off-by: Federico Simoncelli <fsimo...@redhat.com> --- M vdsm/storage/lvm.py 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/21/20121/1 diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py index 0c2964e..3686570 100644 --- a/vdsm/storage/lvm.py +++ b/vdsm/storage/lvm.py @@ -1299,6 +1299,9 @@ changelv(vg, lv, ("--permission", permission)) except se.StorageException: l = getLV(vg, lv) + if l.attr.permission == 'R': + refreshLV(vg, lv) + l = getLV(vg, lv) if l.writeable == rw: # Ignore the error since lv is now rw, hoping that the error was # because lv was already rw, see BZ#654691. We may hide here -- To view, visit http://gerrit.ovirt.org/20121 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6a77b967a057329a90499d7707074befe756b68a Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimo...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches