From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio M. Di Nitto Date: Wed, 04 Jul 2012 13:06:39 +0200 Subject: [Cluster-devel] [PATCH 3/5] rgmanager: Don't immediately stop services that have started sucessfully In-Reply-To: <20120628195743.GA80007@redhat.com> References: <20120628195743.GA80007@redhat.com> Message-ID: <4FF423BF.8050305@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit ACK On 6/28/2012 9:57 PM, Ryan McCabe wrote: > This patch fixes a bug that caused services to be started, then > immediately stopped during recovery. This occurred when a service > had its recovery policy set to relocate and was in a restricted > failover domain. If starting the service failed on other nodes in the > failover domain, the service would be restarted on the original node, > then immediately stopped, even if it had started successfully. This > patch causes rgmanager to leave the service running if it restarted > successfully. > > Resolves: rhbz#789366 > > Signed-off-by: Ryan McCabe > --- > rgmanager/src/daemons/rg_state.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/rgmanager/src/daemons/rg_state.c b/rgmanager/src/daemons/rg_state.c > index a8b1e36..4357d21 100644 > --- a/rgmanager/src/daemons/rg_state.c > +++ b/rgmanager/src/daemons/rg_state.c > @@ -2061,7 +2061,7 @@ exhausted: > svcName); > if (svc_start(svcName, RG_START_RECOVER) == 0) { > *new_owner = me; > - return RG_EFAIL; > + return 0; > } > } > >