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 f746734 commit 4af15f0
Showing 1 changed file with 22 additions and 0 deletions.
22 changes: 22 additions & 0 deletions .werks/16121.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
[//]: # (werk v2)
# Fixed CMC crash when sending illegal metrics to InfluxDB

key | value
---------- | ---
date | 2024-08-14T14:17:10+00:00
version | 2.3.0p13
class | fix
edition | cee
component | cmc
level | 1
compatible | yes

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 4af15f0

Please sign in to comment.