Ticket #503 (new enhancement)

Opened 5 years ago

Last modified 1 year ago

munin server config file should support 'include' statements

Reported by: pere Assigned to: janl
Priority: high Milestone: Munin 3.0
Component: master Version:
Severity: normal Keywords:
Cc:

Description (Last modified by janl)

t would be more convenient and avoid edit conflicts if the munin server config file supported include statements. Here at the university of Oslo we generate the list of hosts to monitor based on those reporting in to a central server, and we had to write our own system to generate the complete munin.conf file using fragments. It would be more elegant and convenient if the manually edited parts were in one file, and the autogenerated stuff were in another file.

Please add support for something like this:

  # Configuration file for Munin
  [...]
  dbdir /var/lib/munin
  [...]
  include /var/lib/sitesummary/munin-nodes.conf
  [...]
  # End of the config file

This would make it a lot easier to handle autogenerated munin server config.

Perhaps some 'includedir' statement could be added as well, to make it easier for other packages to provide munin configuration without messing with the configuration files of munin directly?

Friendly, -- Petter Reinholdtsen

Change History

09/20/07 11:23:21 changed by janl

  • description changed.

02/27/09 10:36:45 changed by janl

  • owner changed from nobody to kjellm.

10/21/09 00:41:49 changed by janl

  • owner changed from kjellm to janl.
  • version deleted.
  • milestone set to Munin 1.4.

And "includedir".

(follow-up: ↓ 6 ) 11/06/09 09:36:08 changed by janl

  • milestone changed from Munin 1.4 to Munin 1.5.

We have a simple includedir now. Not closing as I want to add "include" and make "includedir" more versatile.

(in reply to: ↑ 5 ) 12/03/10 11:36:31 changed by snide

  • milestone changed from Munin 2.0 to Munin 3.0.

As janl explained privately :

The "includedir" statement is actually not processed during reading. It is just stored, and then checked for after, and then that (one) directory is processed for configurations. To get it the "right" way we have to change the config input processor to handle "include" and "includedir" while reading the input

I'm pushing it for 3.0, since there already is an "includedir" statement usable for now.