[lvs-users] ldirectord bugfix patch - force unset parameters to default

Simon Horman horms at verge.net.au
Fri May 15 00:29:39 BST 2009


On Thu, May 14, 2009 at 05:35:31PM -0400, Sean Millichamp wrote:
> Patch 2/4
> 
> This forces resetting of unset configuration directives to defaults.
> 
> The default values for global configuration directives are set once
> early on in the start up.  If a configuration file specifies a global
> configuration option and then, at a later time, that option is removed
> from the configuration file, the global variable will neither get set
> nor reset during the configuration reload.  This results in the old
> setting persisting, instead of the expect and documented "default" value
> in  the absence of the configuration directive.
> 
> This patch puts all the defaults into a function (set_defaults) which
> is called once at startup and once at every configuration reload.  All
> global configuration defaults should be set in this function when added
> or changed.

This looks good to me.

As it depends on the failurcount change, so I will wait until
discussions about that are finalaised before applying.





More information about the lvs-users mailing list