[Cluster-devel] [RFC PATCH dlm 00/10] dlm_controld config settings can be

[heming.zhao@suse.com <heming.zhao@suse.com>]

On 9/21/21 1:57 AM, David Teigland wrote:
> On Sun, Sep 19, 2021 at 02:43:12PM +0800, Heming Zhao wrote:
>> This new feature gives dlm ability to change config settings dynamically.
> 
> Hi Heming,
> 
> Letting dlm_controld reload certain settings from dlm.conf makes sense,
> but I'd like something more basic.  Let the user edit dlm.conf, then
> run dlm_tool reload_config to tell dlm_controld to reread dlm.conf, and
> apply new settings.  This doesn't involve remote nodes, it's all local.
> If the user wants, they can make changes on other nodes the same way.
> 
> (We don't need to touch the "run" extension, which I'd like to remove
> some day since it doesn't really belong in the dlm.)
> 

Hello Dave,

Thank you for your comment.

One of the reasons for my patch based on run command is user could send
command to all nodes by only one-line command.
Dynamic settings (my patch) requires user the least time/work to change
dlm_controld behaviors. If not, it needs to separately edit dlm.conf &
run reload cmd on every local node.

But I am ok with the reload_config idea, it's more basic.
We could give dlm_controld a chance to change behavior on the fly.
If needed, I could file a new patch for feature "reload_config", can I do it?

Heming