import pacemaker-2.1.4-5.el9_1.2

SOURCES/014-abort-transition.patch

@@ -0,0 +1,59 @@
+From 04d1ba5ff20e135c900239f0ebadad42a41b5eba Mon Sep 17 00:00:00 2001
+From: Reid Wahl <nrwahl@protonmail.com>
+Date: Sat, 10 Sep 2022 03:39:12 -0700
+Subject: [PATCH] Fix: controller: Resource reordering doesn't cause transition
+ abort
+
+The te_update_diff_v2() function ignores all move operations. This is
+correct for most CIB sections. However, a move in the resources section
+affects placement order and can require resources to change nodes. In
+that case, since the diff handler does not cause a transition abort, the
+moves will not be initiated until the next natural transition (up to the
+value of cluster-recheck-interval).
+
+This commit modifies te_update_diff_v2() so that it no longer ignores
+moves within the resources section.
+
+This fixes a regression triggered by 41d0a1a and set up by 45e5e82.
+However, the underlying bug had already been present. Prior to 41d0a1a,
+the CIB replacement notification handler caused a transition abort, when
+the resources section was replaced, which hid this bug.
+
+Closes T549
+
+Signed-off-by: Reid Wahl <nrwahl@protonmail.com>
+---
+ daemons/controld/controld_te_callbacks.c | 10 ++++++++--
+ 1 file changed, 8 insertions(+), 2 deletions(-)
+
+diff --git a/daemons/controld/controld_te_callbacks.c b/daemons/controld/controld_te_callbacks.c
+index 6e0dd216e..87ad861a2 100644
+--- a/daemons/controld/controld_te_callbacks.c
++++ b/daemons/controld/controld_te_callbacks.c
+@@ -419,7 +419,13 @@ te_update_diff_v2(xmlNode *diff)
+             crm_trace("Ignoring %s change for version field", op);
+             continue;
+ 
+-        } else if (strcmp(op, "move") == 0) {
++        } else if ((strcmp(op, "move") == 0)
++                   && (strstr(xpath,
++                              "/" XML_TAG_CIB "/" XML_CIB_TAG_CONFIGURATION
++                              "/" XML_CIB_TAG_RESOURCES) == NULL)) {
++            /* We still need to consider moves within the resources section,
++             * since they affect placement order.
++             */
+             crm_trace("Ignoring move change at %s", xpath);
+             continue;
+         }
+@@ -434,7 +440,7 @@ te_update_diff_v2(xmlNode *diff)
+                 match = match->children;
+             }
+ 
+-        } else if (strcmp(op, "delete") != 0) {
++        } else if (!pcmk__str_any_of(op, "delete", "move", NULL)) {
+             crm_warn("Ignoring malformed CIB update (%s operation on %s is unrecognized)",
+                      op, xpath);
+             continue;
+-- 
+2.31.1
+

SPECS/pacemaker.spec

@@ -230,7 +230,7 @@
 Name:          pacemaker
 Summary:       Scalable High-Availability cluster resource manager
 Version:       %{pcmkversion}
-Release:       %{pcmk_release}%{?dist}
+Release:       %{pcmk_release}%{?dist}.2
 License:       GPLv2+ and LGPLv2+
 Url:           https://www.clusterlabs.org/
 
@@ -258,6 +258,9 @@ Patch009:      009-validate.patch
 Patch010:      010-regression.patch
 Patch011:      011-unfencing.patch
 Patch012:      012-crm_resource.patch
+Patch013:      013-rolling-upgrade-monitor.patch
+Patch014:      014-abort-transition.patch
+Patch015:      015-one_shot.patch
 
 Requires:      resource-agents
 Requires:      %{pkgname_pcmk_libs}%{?_isa} = %{version}-%{release}
@@ -855,6 +858,16 @@ exit 0
 %license %{nagios_name}-%{nagios_hash}/COPYING
 
 %changelog
+* Wed Oct 26 2022 Chris Lumens <clumens@redhat.com> - 2.1.4-5.2
+- Fix regression where crm_mon returns nonzero status at cluster shutdown
+- Resolves: rhbz2133911
+
+* Tue Oct 18 2022 Chris Lumens <clumens@redhat.com> - 2.1.4-5.1
+- Fix regression where reordered resources do not get moved
+- Execute resource metadata actions asynchronously
+- Resolves: rhbz2128035
+- Resolves: rhbz2128036
+
 * Wed Aug 10 2022 Ken Gaillot <kgaillot@redhat.com> - 2.1.4-5
 - Fix regression in crm_resource -O
 - Resolves: rhbz2089353