Federico Simoncelli has posted comments on this change.
Change subject: lvm: refresh on mda permission mismatch ......................................................................
Patch Set 1:
(1 comment)
.................................................... File vdsm/storage/lvm.py Line 1299: changelv(vg, lv, ("--permission", permission)) Line 1300: except se.StorageException: Line 1301: l = getLV(vg, lv) Line 1302: if l.attr.permission == 'R': Line 1303: refreshLV(vg, lv) I am still not convinced that this is the appropriate fix. I'd prefer to fix this in lvm. Line 1304: l = getLV(vg, lv) Line 1305: if l.writeable == rw: Line 1306: # Ignore the error since lv is now rw, hoping that the error was Line 1307: # because lv was already rw, see BZ#654691. We may hide here