resource-agents: master - halvm: Fix bug 506587: lvm agent incorrectly reports vg is in volume_list

Jonathan Brassow jbrassow at fedoraproject.org
Fri May 14 20:36:16 UTC 2010


Gitweb:        http://git.fedorahosted.org/git/resource-agents.git?p=resource-agents.git;a=commitdiff;h=0078bf0667c11d8359452823b3fb9c2a008aaac6
Commit:        0078bf0667c11d8359452823b3fb9c2a008aaac6
Parent:        9deb0cdb544086a9f053eeda21b6491ec3be1784
Author:        Jonathan Brassow <jbrassow at redhat.com>
AuthorDate:    Fri May 14 15:35:47 2010 -0500
Committer:     Jonathan Brassow <jbrassow at redhat.com>
CommitterDate: Fri May 14 15:35:47 2010 -0500

halvm: Fix bug 506587: lvm agent incorrectly reports vg is in volume_list

If the name of the VG controlled by rgmanager is a substring of the name
of the node, the HA-LVM script will complain of an improper setup.

The fix is to also look for the associated quotes that are necessary when
specifying strings in lvm.conf.  Thanks to John Ruemker for the patch.
---
 rgmanager/src/resources/lvm.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/rgmanager/src/resources/lvm.sh b/rgmanager/src/resources/lvm.sh
index 4b7155c..7f174ca 100644
--- a/rgmanager/src/resources/lvm.sh
+++ b/rgmanager/src/resources/lvm.sh
@@ -47,7 +47,7 @@ function ha_lvm_proper_setup_check
 	# lvm.conf:activation/volume_list; otherwise, machines
 	# will be able to activate the VG regardless of the tags
 	##
-	if lvm dumpconfig activation/volume_list | grep $OCF_RESKEY_vg_name; then
+	if lvm dumpconfig activation/volume_list | grep "\"$OCF_RESKEY_vg_name\""; then
 		ocf_log err "HA LVM:  Improper setup detected"
 		ocf_log err "- $OCF_RESKEY_vg_name found in \"volume_list\" in lvm.conf"
 		return $OCF_ERR_GENERIC


More information about the cluster-commits mailing list