RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 822393 - [vdsm] getVGInfo on specific VG fails due to access problem with different VG
Summary: [vdsm] getVGInfo on specific VG fails due to access problem with different VG
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: vdsm
Version: 6.3
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: rc
: ---
Assignee: Eduardo Warszawski
QA Contact: Haim
URL:
Whiteboard: storage rhev-3.0
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-05-17 08:59 UTC by Meni Yakove
Modified: 2014-01-13 00:52 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of: 736301
Environment:
Last Closed: 2012-05-31 13:23:15 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)

Description Meni Yakove 2012-05-17 08:59:01 UTC 
Description of problem:

getVGInfo command on newly created storage fails due to access problem with one
of the other VGs in the system. 

flow: 

- create new storage domain using RHEVM 
   * createVG command - returns successfully 
   * createStorageDomain command - returns successfully 
   * getVGInfo on vgUUID - fails; 
     AttributeError: Failed reload: 703256d8-0c4a-4138-a150-e4db0ad87db9

when I issue vgs command, I see VG, also, getStorageDomainInfo command returns. 
problem solved just after I restarted vdsm, which points on cache issue.
also, i'm curious why getVGInfo, which requests specific device, fails, in case
different and unrelated vg is down. 

attached vdsm log.
Comment 1 Haim 2012-05-17 10:51:02 UTC 
we have a patch on downstream\upstream: 


 git show fb7012c7
commit fb7012c752d80bb6b24d50389b84879988e90471
Author: Eduardo Warszawski <ewarszaw>
Date:   Tue Sep 13 10:39:31 2011 +0300

    BZ#736301 - Don't fail lvm.getVGbyUUID() if unreadable VGs found.
    
    Change-Id: Ief82082ebb3cfb6731dc9c7905f2844c60a7f425
    Reviewed-on: http://gerrit.usersys.redhat.com/930
    Reviewed-by: Dan Kenigsberg <danken>
    Tested-by: Dan Kenigsberg <danken>

diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py
index 0276f9e..9d734f7 100644
--- a/vdsm/storage/lvm.py
+++ b/vdsm/storage/lvm.py
@@ -833,12 +833,18 @@ def getVGs(vgNames):
 def getAllVGs():
     return _lvminfo.getAllVgs() #returns list
 
-
+# TODO: lvm VG UUID should not be exposed.
+# Remove this function when hsm.public_createVG is removed.
 def getVGbyUUID(vgUUID):
     # cycle through all the VGs until the one with the given UUID found
     for vg in getAllVGs():
-        if vg.uuid == vgUUID:
-            return vg
+        try:
+            if vg.uuid == vgUUID:
+                return vg
+        except AttributeError, e:
+            # An unreloadable VG found but may be we are not looking for it.
+            log.debug("%s" % e.message, exc_info=True)
+            continue
     # If not cry loudly
     raise se.VolumeGroupDoesNotExist("vg_uuid: %s" % vgUUID)
Comment 3 RHEL Program Management 2012-05-21 06:50:01 UTC 
This request was not resolved in time for the current release.
Red Hat invites you to ask your support representative to
propose this request, if still desired, for consideration in
the next release of Red Hat Enterprise Linux.
Comment 4 Dan Kenigsberg 2012-05-21 12:20:21 UTC 
(In reply to comment #0)
> 
> attached vdsm log.

Actually, no. Since we have the patch for upstream and form rhev-3.1, I am guessing you are refering to rhev-3.0.

I do not mind making this bug track this issue for the next 3.0.z build. Setting the Zstream keyword, as the 3.1 bug 736301 is already fixed.
Note You need to log in before you can comment on or make changes to this bug.