From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lon Hohberger Date: Thu, 5 Jan 2012 10:57:33 -0500 Subject: [Cluster-devel] [PATCH 3/5] rgmanager: Fix call to service_status() (RHEL5) In-Reply-To: <1325779055-1424-1-git-send-email-lhh@redhat.com> References: <1325779055-1424-1-git-send-email-lhh@redhat.com> Message-ID: <1325779055-1424-3-git-send-email-lhh@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit This patch simply fixes the calls to service_status(), the s_trans variable was being incorrectly set. Resolves: rhbz#743214 Signed-off-by: Lon Hohberger --- rgmanager/src/resources/default_event_script.sl | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/rgmanager/src/resources/default_event_script.sl b/rgmanager/src/resources/default_event_script.sl index 2d5503f..f2c7bc1 100644 --- a/rgmanager/src/resources/default_event_script.sl +++ b/rgmanager/src/resources/default_event_script.sl @@ -500,7 +500,7 @@ define default_service_event_handler() continue; } - (d_trans,,,, owner, state) = service_status(services[x], 1); + (d_trans,,,,, owner, state) = service_status(services[x], 1); if ((service_state == "started") and (owner < 0) and (state == "stopped")) { info("Dependency met; starting ", services[x]); @@ -518,9 +518,12 @@ define default_service_event_handler() % as above is running and the dependent service was % started@or after the service, then stopping it % will result in unwanted service outage. - (s_trans,,,, s_state) = service_status(service_name); + (s_trans,,,,,, s_state) = service_status(service_name, 1); if ((s_state == "started") and (state == "started") and (d_trans >= s_trans)) { + %debug("S:", service_name, " trans ", s_trans); + %debug("D:", services[x], " trans ", d_trans); + debug("Skipping ", services[x], "; restart not needed"); continue; -- 1.7.3.4