CTDB ipreallocated event handling
amitay at gmail.com
Thu Jul 13 06:35:37 UTC 2017
On Thu, Jul 13, 2017 at 4:18 PM, Martin Schwenke via samba-technical <
samba-technical at lists.samba.org> wrote:
> On Thu, 13 Jul 2017 15:30:33 +1000, Martin Schwenke via samba-technical
> <samba-technical at lists.samba.org> wrote:
> > On Wed, 12 Jul 2017 13:28:13 +0200, Bjoern Baumbach <bb at sernet.de>
> > wrote:
> > > thank you very much for the explanation and updating the docs!
> > >
> > > On 07/12/2017 04:40 AM, Martin Schwenke wrote:
> > > > The documentation was out of date.
> > > >
> > > > Now that service_check_reconfigure() is gone, we check
> > > > ctdb_service_needs_reconfigure() explicitly in the "ipreallocated"
> > > > event and run ctdb_service_reconfigure(), which we still have
> > > > because it does a bit of magic that we don't quite want to get rid
> > > > of without thinking carefully.
> > >
> > > This was a good decision. Handling the changes in the "ipreallocated"
> > > event is much more comprehensible :-)
> > >
> > > The updated documentation looks much better.
> > Yay! :-)
> > > Adding the "updateip" event to the following part might help some
> > > users, who like to implement their own event scripts:
> > >
> > > -* An event script can use ctdb_service_set_reconfigure() in "takeip"
> > > - or "releaseip" events to flag that its service needs to be
> > > +* An event script can use ctdb_service_set_reconfigure() in "takeip",
> > > + "releaseip" or "updateip" events to flag that its service needs to
> > > be reconfigured. The "ipreallocated" event can then use
> > Are you happy for me to squash that change in with my patch and re-post?
> Assuming the answer to this is "yes", so updated patch is attached. :-)
> Please review and maybe push...
> peace & happiness,
Pushed to autobuild.
More information about the samba-technical