ENH: order config as jail.conf, jail.d/*.conf, jail.local, jail.d/*.loca...

[grooverdan]

...l. closes gh-388

[coveralls]

Coverage remained the same when pulling d6d51e3 on grooverdan:config_order into 2d1bd54 on fail2ban:master.

[yarikoptic]

well... that might brake some installations where people deduced and relied upon current order. But what problem such change would solve? may be we should just document current order?

[grooverdan]

The problem to be solved is:

• provide distro maintainers of fail2ban the ability to overwrite jail.conf without extensively modifying it (like jail.d / fail2ban.d ordering and doco #388) and not worry if the user has created a jail.local file conflicting with it
• provide distro maintainers of other packages to place in jail.d/x.conf files leaving it to the user to enable them in jail.local or jail.d/x.local
• make the *.conf files the primary files and then the *.local files as overrides in a more logical way
• by adding {fail2ban|jail}.d/*.local encourages users to only make local changes to .local files files

This change would break are those where the user has overwritten a jail.local with a jail.d/x.conf file. I'd make the assumption that most users would use one or the other and not overwrite their own config in jail.local with a jail.d/x.conf file.

I think its important to set this logical priority of files before these features are extensively used.

[yarikoptic]

yeah -- sounds reasonable ;) cheers and thanks

On Thu, 10 Oct 2013, Daniel Black wrote:

The problem to be solved is:

 * provide distro maintainers of fail2ban the ability to overwrite
   jail.conf without extensively modifying it (like [1]#388) and not
   worry if the user has created a jail.local file conflicting with it
 * provide distro maintainers of other packages to place in jail.d/x.conf
   files leaving it to the user to enable them in jail.local or
   jail.d/x.local
 * make the *.conf files the primary files and then the *.local files as
   overrides in a more logical way
 * by adding {fail2ban|jail}.d/*.local encourages users to only make
   local changes to .local files files

Yaroslav O. Halchenko, Ph.D.
http://neuro.debian.net http://www.pymvpa.org http://www.fail2ban.org
Senior Research Associate, Psychological and Brain Sciences Dept.
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419
WWW: http://www.linkedin.com/in/yarik

[grooverdan]

yeah -- sounds reasonable ;) cheers and thanks

Your welcome. Sound of written this up first.

Needs a changelog entry but I didn't want to add a conflict with so many outstanding pull requests.

Merge?