Ticket #945 (new defect)

Opened 1 year ago

Last modified 1 year ago

Better error reporting on config errors et al (stop silently ignoring problems)

Reported by: nanardon Assigned to: nobody
Priority: normal Milestone: Munin 3.0
Component: master Version:
Severity: normal Keywords:
Cc:

Description

I did a typo in my configuration file, instead server;host I wrote server:host (notice the :).

As the : is used a separator in datafile I did get warning about "unknown variable" but nothing clearly saying I did a mistake, neither why nothing get output even rrd data file get created.

I suggest either: - to check a : appear in host definition and deny it with a clear message in log - to escape : in datafile if this letter is allowed.

But the problem must not be ignore (It took to me one hour to find the issue...)

The problem appeared on 1.3.4 version which is not in trac version list.

Change History

01/14/11 17:37:07 changed by jo

  • milestone changed from Munin 1.4.6 to Munin 3.0.

As : is not a legal character in a hostname, adding functionality to escape it has really low priority.

However, we could definitely get better at error reporting. Not just this case, but in general. Assigning this to the 3.x milestone, as we'll try to get both the graph and html engines rewritten for that milestone, and we really need better error handling (both from those two, and the rest).

01/14/11 17:38:22 changed by jo

  • summary changed from Host name in munin.conf must not include ':' to Better error reporting on config errors et al (stop silently ignoring problems).