Federico Simoncelli has posted comments on this change.
Change subject: blockSD: do not try to parse metadata from missing pv ......................................................................
Patch Set 3: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/27442/3/vdsm/storage/blockSD.py File vdsm/storage/blockSD.py:
Line 670: pv = lvm.getPV(dev) Line 671: if not pv: Line 672: cls.log.warning("Could not get metadata from the device: %s", Line 673: dev) Line 674: continue You return an incomplete mapping swallowing the error? Why is it legal to create a volume storing an incomplete mapping? Line 675: pvInfo = {} Line 676: pvInfo["guid"] = os.path.basename(pv.name) Line 677: pvInfo["uuid"] = pv.uuid Line 678: # this is another trick, it's not the