[PATCH] ctdb-doc: Provide an example script for migrating old configuration (bug 13550)
martin at meltin.net
Mon Jul 30 06:10:36 UTC 2018
On Mon, 30 Jul 2018 14:32:19 +1000, Amitay Isaacs <amitay at gmail.com>
> On Sun, Jul 29, 2018 at 12:10 PM, Martin Schwenke <martin at meltin.net> wrote:
> > On Sun, 29 Jul 2018 01:20:44 +1000, Amitay Isaacs <amitay at gmail.com>
> > wrote:
> >> On Sat, Jul 28, 2018 at 5:11 PM, Martin Schwenke via samba-technical
> >> <samba-technical at lists.samba.org> wrote:
> >> The check for CTDB_NODES (and CTDB_PUBLIC_ADDRESSES) is not correct.
> >> The check for those should confirm that the variables are pointing to
> >> "standard" files in /etc/ctdb. Right now the script compares the path
> >> to a directory where the files are put. I would expect people to run
> >> this with "-d tmp_config" rather than "-d /etc/ctdb".
> > The problem is that the "standard" configuration is set at configure
> > time, so those who use /usr/local/etc/ctdb/ then wouldn't be able to use
> > this out of the box.
> > That's also why I made the script refused to overwrite without the -f
> > option. It is quite safe to use in-place.
> > However, you're right. It is still dodgy implied behaviour. :-)
> > So, how about the attached fixup? It still defaults to writing the
> > files in-place, but provides an option to put the output directory
> > somewhere else. I think this is better.
> Yeah, definitely better.
> Here is another fixup with improved feedback and some bike shedding. :-)
> You can squash both fixups. With these changes, Reviewed-by: me.
peace & happiness,
More information about the samba-technical