diff --git a/Avoid-false-positive-for-VG-activation.patch b/Avoid-false-positive-for-VG-activation.patch new file mode 100644 index 0000000000000000000000000000000000000000..d554ba9f243d68ed2ecc3ad9f1c52658604e1aec --- /dev/null +++ b/Avoid-false-positive-for-VG-activation.patch @@ -0,0 +1,49 @@ +From 65a066cf9066390db65c4875e21c4c391793b9ae Mon Sep 17 00:00:00 2001 +From: Arslan Ahmad +Date: Tue, 16 Jan 2024 09:11:17 +0530 +Subject: [PATCH] Avoid false positive for VG activation + +When lvm.conf file has `volume_list` parameter configured and the +cluster is managing the shared storage using `system_id_source`, +then the activation of the LV fails to happen. However it is +reported as a success. + +The fixes will avoid starting of `LVM-activate` resource when +the cluster is configured with both `system_id_source` and +`volume_list`. + +Signed-off-by: Arslan Ahmad +--- + heartbeat/LVM-activate | 9 +++++++++ + 1 file changed, 9 insertions(+) + +diff --git a/heartbeat/LVM-activate b/heartbeat/LVM-activate +index f6f24a3b..3858ed8d 100755 +--- a/heartbeat/LVM-activate ++++ b/heartbeat/LVM-activate +@@ -448,6 +448,10 @@ systemid_check() + { + # system_id_source is set in lvm.conf + source=$(lvmconfig 'global/system_id_source' 2>/dev/null | cut -d"=" -f2) ++ ++ # Is volume_list set in lvm.conf ++ vol_list=$(lvmconfig 'activation/volume_list' 2>/dev/null | cut -d"=" -f2) ++ + if [ "$source" = "" ] || [ "$source" = "none" ]; then + ocf_exit_reason "system_id_source in lvm.conf is not set correctly!" + exit $OCF_ERR_ARGS +@@ -458,6 +462,11 @@ systemid_check() + exit $OCF_ERR_ARGS + fi + ++ if [ -n "$source" ] && [ -n "$vol_list" ]; then ++ ocf_exit_reason "Both system_id_source & volume_list cannot be defined!" ++ exit $OCF_ERR_ARGS ++ fi ++ + return $OCF_SUCCESS + } + +-- +2.25.1 + diff --git a/resource-agents.spec b/resource-agents.spec index 6ecd1a62047da64d8298233613e513f997ac1c22..4b0e05550d5ceb68a5588d5e522ee76f7cfc60c2 100644 --- a/resource-agents.spec +++ b/resource-agents.spec @@ -1,7 +1,7 @@ Name: resource-agents Summary: Open Source HA Reusable Cluster Resource Scripts Version: 4.13.0 -Release: 5 +Release: 6 License: GPLv2+ and LGPLv2+ URL: https://github.com/ClusterLabs/resource-agents Source0: https://github.com/ClusterLabs/resource-agents/archive/v%{version}.tar.gz @@ -10,6 +10,7 @@ Patch0001: Fix-PRIMARY_IFACE-variable.patch Patch0002: fix-loopback-handling.patch Patch0003: fix-handler-out-of-scope-leak.patch Patch0004: fix-uninitialized-value-covscan-error.patch +Patch0005: Avoid-false-positive-for-VG-activation.patch Obsoletes: heartbeat-resources <= %{version} Provides: heartbeat-resources = %{version} BuildRequires: automake autoconf pkgconfig gcc perl-interpreter perl-generators python3-devel @@ -107,7 +108,10 @@ export CFLAGS="$(echo '%{optflags}')" %{_mandir}/man8/{ocf-tester.8*,ldirectord.8*} %changelog -* Fri Feb 23 2024 bixiaoyan - 4.13.0-5 +* Fri Feb 23 2024 bixiaoyan - 4.13.0-6 +- LVM-activate: Avoid false positive for VG activation + +* Fri Feb 23 2024 bixiaoyan - 4.13.0-5 - use memset() to fix "uninitialized value" covscan error, as qb_ipcc_recv() will always set a message (according to honzaf) * Thu Feb 22 2024 bixiaoyan - 4.13.0-4