Change in vdsm[master]: lvm: fix pv_count and pv_names check

fsimonce at redhat.com fsimonce at redhat.com
Tue Oct 15 12:36:28 UTC 2013


Federico Simoncelli has uploaded a new change for review.

Change subject: lvm: fix pv_count and pv_names check
......................................................................

lvm: fix pv_count and pv_names check

Change-Id: I9a3a2e7c7e5ddc4949eb690c1e4613284db9eafc
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm/storage/lvm.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/96/20196/1

diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py
index 0c2964e..02bacd8 100644
--- a/vdsm/storage/lvm.py
+++ b/vdsm/storage/lvm.py
@@ -410,8 +410,8 @@
                     vgsFields[uuid][pvNameIdx].append(pv_name)
             for fields in vgsFields.itervalues():
                 vg = makeVG(*fields)
-                if vg.pv_count != len(vg.pv_name):
-                    log.error("vg %s has pv_count %d but pv_names %s",
+                if int(vg.pv_count) != len(vg.pv_name):
+                    log.error("vg %s has pv_count %s but pv_names %s",
                               vg.name, vg.pv_count, vg.pv_name)
                 self._vgs[vg.name] = vg
                 updatedVGs[vg.name] = vg


-- 
To view, visit http://gerrit.ovirt.org/20196
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9a3a2e7c7e5ddc4949eb690c1e4613284db9eafc
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