[Cluster-devel] [PATCH 3/5] rgmanager: Don't immediately stop services that have started sucessfully
Fabio M. Di Nitto
fdinitto at redhat.com
Wed Jul 4 11:06:39 UTC 2012
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 <rmccabe at redhat.com>
> ---
> 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;
> }
> }
>
>
More information about the Cluster-devel
mailing list