cluster: STABLE31 - rgmanager: Fix dependency restart bug in CP mode

Lon Hohberger lon at fedoraproject.org
Tue Oct 25 21:37:37 UTC 2011


Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=5ce78b0a1a1c5851524497b305a568e52043cf09
Commit:        5ce78b0a1a1c5851524497b305a568e52043cf09
Parent:        a95fcca7ecd070c0fef1b85d3158e728b04c3398
Author:        Adam Drew <adrew at redhat.com>
AuthorDate:    Fri Oct 7 12:31:16 2011 -0400
Committer:     Lon Hohberger <lhh at redhat.com>
CommitterDate: Tue Oct 25 17:31:15 2011 -0400

rgmanager: Fix dependency restart bug in CP mode

If we:
 - had central_processing mode enabled,
 - had a service with 'depend' set on another
   service,
 - we attempted to restart the service with the
   dependency, and
 - the dependency was missing

rgmanager would leave the state in 'recovering'

This is the corrected patch from Adam.

Resolves: rhbz#743442

Signed-off-by: Adam Drew <adrew at redhat.com>
Signed-off-by: Lon Hohberger <lhh at redhat.com>
---
 rgmanager/src/resources/default_event_script.sl |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/rgmanager/src/resources/default_event_script.sl b/rgmanager/src/resources/default_event_script.sl
index 759215c..4c5ca0e 100644
--- a/rgmanager/src/resources/default_event_script.sl
+++ b/rgmanager/src/resources/default_event_script.sl
@@ -178,6 +178,7 @@ define move_or_start(service, node_list)
 		(,,, owner, state) = service_status(depends);
 		if (owner < 0) {
 			debug(service, " is not runnable; dependency not met");
+			()=service_stop(service);
 			return ERR_DEPEND;
 		}
 	}


More information about the cluster-commits mailing list