cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
* [Cluster-devel] [PATCH] Cman: test that corosync is not already running
@ 2011-01-06 10:28 Fabio M. Di Nitto
  2011-01-06 10:37 ` Fabio M. Di Nitto
  0 siblings, 1 reply; 2+ messages in thread
From: Fabio M. Di Nitto @ 2011-01-06 10:28 UTC (permalink / raw)
  To: cluster-devel.redhat.com

From: Jan Friesse <jfriesse@redhat.com>

Patch fixes init file so now before cman start is tested, if corosync is
running. If so, init script will refuse to start.

Resolves: rhbz#617234
---
 cman/init.d/cman.in |   14 ++++++++++++++
 1 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/cman/init.d/cman.in b/cman/init.d/cman.in
index 7dae24d..6dfc119 100644
--- a/cman/init.d/cman.in
+++ b/cman/init.d/cman.in
@@ -521,11 +521,25 @@ stop_configfs()
 	fi
 }
 
+corosync_running()
+{
+	[ -f /var/run/corosync.pid ] || return 1
+
+	read corosync_pid foo < /var/run/corosync.pid
+	if [ "$(pidof corosync)" == "$corosync_pid" ];then
+		errmsg="Corosync Cluster Engine is already running"
+		return 0
+	fi
+
+	return 1
+}
+
 start_cman()
 {
 	check_exec cman_tool || return $?
 	cman_running && return 0
 	cman_checkconfig || return 1
+	corosync_running && return 1
 
 	tmpfile=$(mktemp -t cmanstartup.XXXXXXXXXX)
 	if [ -z "$tmpfile" ]; then
-- 
1.7.2.3



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Cluster-devel] [PATCH] Cman: test that corosync is not already running
  2011-01-06 10:28 [Cluster-devel] [PATCH] Cman: test that corosync is not already running Fabio M. Di Nitto
@ 2011-01-06 10:37 ` Fabio M. Di Nitto
  0 siblings, 0 replies; 2+ messages in thread
From: Fabio M. Di Nitto @ 2011-01-06 10:37 UTC (permalink / raw)
  To: cluster-devel.redhat.com

Re-iterating the patch for ACK to go in RHEL6.

Fabio

On 1/6/2011 11:28 AM, Fabio M. Di Nitto wrote:
> From: Jan Friesse <jfriesse@redhat.com>
> 
> Patch fixes init file so now before cman start is tested, if corosync is
> running. If so, init script will refuse to start.
> 
> Resolves: rhbz#617234
> ---
>  cman/init.d/cman.in |   14 ++++++++++++++
>  1 files changed, 14 insertions(+), 0 deletions(-)
> 
> diff --git a/cman/init.d/cman.in b/cman/init.d/cman.in
> index 7dae24d..6dfc119 100644
> --- a/cman/init.d/cman.in
> +++ b/cman/init.d/cman.in
> @@ -521,11 +521,25 @@ stop_configfs()
>  	fi
>  }
>  
> +corosync_running()
> +{
> +	[ -f /var/run/corosync.pid ] || return 1
> +
> +	read corosync_pid foo < /var/run/corosync.pid
> +	if [ "$(pidof corosync)" == "$corosync_pid" ];then
> +		errmsg="Corosync Cluster Engine is already running"
> +		return 0
> +	fi
> +
> +	return 1
> +}
> +
>  start_cman()
>  {
>  	check_exec cman_tool || return $?
>  	cman_running && return 0
>  	cman_checkconfig || return 1
> +	corosync_running && return 1
>  
>  	tmpfile=$(mktemp -t cmanstartup.XXXXXXXXXX)
>  	if [ -z "$tmpfile" ]; then



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-01-06 10:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-06 10:28 [Cluster-devel] [PATCH] Cman: test that corosync is not already running Fabio M. Di Nitto
2011-01-06 10:37 ` Fabio M. Di Nitto

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).