[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Linux-cluster] [PATCH] cman/init.d/cman.in: don't assume chkconfig exists



Hi Guido,

in future, can you please send patches to cluster-devel redhat com? It´s
easier for me to spot them.

On 3/10/2010 10:27 PM, Guido Günther wrote:
> Hi,
> attached patch makes sure we don't rely on chkconffig (which doesn't
> exist on Debian based distros). It also checks additionally for
> network-manager since this is the name of the service on Debian/Ubuntu.
> Cheers,
>  -- Guido

I am not entirely sure why we need this patch.

> # deb based distros
> if [ -d /etc/default ]; then
>         [ -f /etc/default/cluster ] && . /etc/default/cluster
>         [ -f /etc/default/cman ] && . /etc/default/cman
>         [ -z "$LOCK_FILE" ] && LOCK_FILE="/var/lock/cman"
>         type chkconfig > /dev/null 2>&1 || alias chkconfig=local_chkconfig
> fi

local_chkconfig mimics chkconfig behavior on Debian based systems that
don´t have chkconfig.

It was tested successfully on Ubuntu, but I don´t see why it would work
on Debian. If the local_chkconfig is broken, then we need to fix that as
it is used also for xen_bridge workaround.

Thanks
Fabio


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]