rgmanager: master - rgmanager: Preserve incarnations across config changes

rgmanager: master - rgmanager: Preserve incarnations across config changes

[Lon Hohberger]

Gitweb:        http://git.fedorahosted.org/git/rgmanager.git?p=rgmanager.git;a=commitdiff;h=7310545c230d441b9aa413d1ebd320cf01a3bc4d
Commit:        7310545c230d441b9aa413d1ebd320cf01a3bc4d
Parent:        202c45ed9d52bb5e3e13920b548790fe02c5c044
Author:        Lon Hohberger <lhh@redhat.com>
AuthorDate:    Fri Jul 31 15:41:58 2009 -0400
Committer:     Lon Hohberger <lhh@redhat.com>
CommitterDate: Tue Aug 11 15:27:45 2009 -0400

rgmanager: Preserve incarnations across config changes

Red Hat Bugzilla #506094, part 2/3

(Part 1/3 is in the resource-agents repository)

Signed-off-by: Lon Hohberger <lhh@redhat.com>
---
 rgmanager/src/daemons/groups.c |   27 +++++++++++++++++++++++++++
 1 files changed, 27 insertions(+), 0 deletions(-)

diff --git a/rgmanager/src/daemons/groups.c b/rgmanager/src/daemons/groups.c
index 573a973..ce55ff3 100644
--- a/rgmanager/src/daemons/groups.c
+++ b/rgmanager/src/daemons/groups.c
@@ -1575,6 +1575,31 @@ dump_config_version(FILE *fp)
 
 
 /**
+ Copy out the incarnations after doing CONDSTOPs
+ */
+static int
+copy_incarnations(resource_t **leftres, resource_t **rightres)
+{
+	resource_t *lc, *rc;
+	int ret;
+
+	list_do(leftres, lc) {
+		rc = find_resource_by_ref(rightres, lc->r_rule->rr_type,
+					  primary_attr_value(lc));
+		/* Resource does not exist */
+		if (!rc)
+			continue;
+
+		/* Ok, see if the resource is the same */
+		if (!rescmp(lc, rc))
+			rc->r_incarnations = lc->r_incarnations;
+	} while (!list_done(leftres, lc));
+
+	return 0;
+}
+
+
+/**
   Initialize resource groups.  This reads all the resource groups from 
   CCS, builds the tree, etc.  Ideally, we'll have a similar function 
   performing deltas on the two trees so that we can fully support online
@@ -1690,6 +1715,8 @@ init_resource_groups(int reconfigure, int do_init)
 		pthread_rwlock_unlock(&resource_lock);
 
 		do_condstops();
+
+		copy_incarnations(&_resources, &reslist);
 	}
 
 	/* Swap in the new configuration */