Linux Archive

Linux Archive (http://www.linux-archive.org/)
-   Cluster Development (http://www.linux-archive.org/cluster-development/)
-   -   rgmanager: Fix return code when relocation fails and svc is running on original node (http://www.linux-archive.org/cluster-development/682436-rgmanager-fix-return-code-when-relocation-fails-svc-running-original-node.html)

Ryan McCabe 07-11-2012 01:14 PM

rgmanager: Fix return code when relocation fails and svc is running on original node
 
Return RG_ERELO (Relocation failure; service running on original node) when
relocation failed and the service was restarted successfully on the original
node.

This is the behavior that's currently in the STABLE32 and RHEL6 branches.

Signed-off-by: Ryan McCabe <rmccabe@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 4357d21..9000f1b 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 0;
+ return RG_ERELO;
}
}

--
1.7.10.4

Lon Hohberger 07-13-2012 08:09 PM

rgmanager: Fix return code when relocation fails and svc is running on original node
 
On 07/11/2012 09:14 AM, Ryan McCabe wrote:
> Return RG_ERELO (Relocation failure; service running on original node) when
> relocation failed and the service was restarted successfully on the original
> node.
>
> This is the behavior that's currently in the STABLE32 and RHEL6 branches.
>
> Signed-off-by: Ryan McCabe <rmccabe@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 4357d21..9000f1b 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 0;
> + return RG_ERELO;
> }
> }
>
>

Reviewed-by: Lon Hohberger <lhh@redhat.com>

-- Lon


All times are GMT. The time now is 03:07 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.