Skip to content

Commit

Permalink
16121 FIX Fixed CMC crash when sending illegal metrics to InfluxDB
Browse files Browse the repository at this point in the history
When the CMC tried to send to a metric to an InfluxDB where the metric name
was malformed (e.g. starting with an underscore), the CMC crashed.  This
has been fixed, such metrics are now simply forwarded to the InfluxDB.  The
latter will probably be a bit unhappy about such metric names, but this can
be figured out via its logs, and the corresponding check can be fixed.

Note that in a future Checkmk version, we will probably drop such metrics
silently, logging this only for higher log levels to avoid log spam at the
default log settings.

Change-Id: Iadbd96ae2c7d14acbb92103a1173d86f45d1ae8c
  • Loading branch information
spt29 committed Aug 14, 2024
1 parent 001e546 commit 9185cd3
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions .werks/16121
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
Title: Fixed CMC crash when sending illegal metrics to InfluxDB
Class: fix
Compatible: compat
Component: cmc
Date: 1723645030
Edition: cee
Level: 1
Version: 2.2.0p33

When the CMC tried to send to a metric to an InfluxDB where the metric name
was malformed (e.g. starting with an underscore), the CMC crashed. This
has been fixed, such metrics are now simply forwarded to the InfluxDB. The
latter will probably be a bit unhappy about such metric names, but this can
be figured out via its logs, and the corresponding check can be fixed.

Note that in a future Checkmk version, we will probably drop such metrics
silently, logging this only for higher log levels to avoid log spam at the
default log settings.

0 comments on commit 9185cd3

Please sign in to comment.