Change in vdsm[master]: lvm: refresh on mda permission mismatch

fsimonce at redhat.com fsimonce at redhat.com
Fri Oct 11 15:32:34 UTC 2013


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 <fsimonce at 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 <fsimonce at redhat.com>


More information about the vdsm-patches mailing list