[Openais] Corosync Patch: Fix the default for COROSYNC_RUN_DIR

Steven Dake sdake at redhat.com
Fri Apr 9 11:17:02 PDT 2010


On Fri, 2010-04-09 at 15:05 +0200, Andrew Beekhof wrote:
> This looks like a copy/paste error to me...
> 
> The "RUN" in COROSYNC_RUN_DIR would seem to imply /var/run
> Also /var/lib is persistent and doesn't need to be created at startup.
> On the other-hand, LSB states that the contents of /var/run is blow
> away at boot time.
> 
> So I'm reasonably sure the following patch is correct.
> Please ACK.

In general "rundir" should probably be renamed to "libdir" since the
idea is that data stored there is persistent.

Totem requires persistence between node boots of data stored with the
rundir path.

Regards
-steve
> 
> [02:59 PM] beekhof at f12 ~/Development/sources/corosync # svn diff
> ./exec/totemsrp.c
> Index: exec/totemsrp.c
> ===================================================================
> --- exec/totemsrp.c	(revision 2756)
> +++ exec/totemsrp.c	(working copy)
> @@ -775,7 +775,7 @@
> 
>  	rundir = getenv ("COROSYNC_RUN_DIR");
>  	if (rundir == NULL) {
> -		rundir = LOCALSTATEDIR "/lib/corosync";
> +		rundir = LOCALSTATEDIR "/run/corosync";
>  	}
> 
>  	res = mkdir (rundir, 0700);
> _______________________________________________
> Openais mailing list
> Openais at lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/openais



More information about the Openais mailing list