On Thu, Jun 21, 2012 at 10:07:11AM +0700, Philippe Alcoy wrote:
> # HG changeset patch
> # User Philippe Alcoy <phili...@alcoy.co.uk>
> # Date 1340192740 -25200
> # Node ID 8cf404c39c7e0c63ec4a86b967d8a3e5de60977b
> # Parent  f6ce2daddba6234643f715c46b28d5562940eb1f
> Fix chkconfig heartbeat on flag missing in ha_propagate
> 
> diff -r f6ce2daddba6 -r 8cf404c39c7e heartbeat/lib/ha_propagate.in
> --- a/heartbeat/lib/ha_propagate.in   Wed Jun 20 17:42:24 2012 +0700
> +++ b/heartbeat/lib/ha_propagate.in   Wed Jun 20 18:45:40 2012 +0700
> @@ -39,5 +39,5 @@
>      print "Propagating HA configuration files to node " + v + "."
>      res = os.system("scp " + cfgfile + " " + authfile + " root@" + v + ":" + 
> cfgdir)
>      print "Setting HA startup configuration on node " + v + "."
> -    res = os.system("ssh " + " root@" + v + " chkconfig `chkconfig 
> heartbeat`")
> +    res = os.system("ssh " + " root@" + v + " chkconfig `chkconfig heartbeat 
> on`")

Well, No.

Is is supposed to propagate the setting of the current node
to all nodes.

There are valid use cases for having heartbeat *off* on system boot.

Note the double chkconfig and the backticks.

When it was implemented, "chkconfig single-service-name" apparently 
returned the current setting as "on" or "off" to stdout,
at least on the distribution this was implemented on.

I guess this is no longer true, or diverges between distributions.
It is not even available on all distributions.

If anything, I'm tempted to drop that ha_propagate thing altogether.
People should be able to scp config files and runlevel settings
themselves, really...

-- 
: Lars Ellenberg
: LINBIT | Your Way to High Availability
: DRBD/HA support and consulting http://www.linbit.com
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to