search

ClickHouse / graphouse

Graphouse allows you to use ClickHouse as a Graphite storage.
Apache License 2.0
259 stars 55 forks source link

handling of metrics with : colon in them seems broken #140

Open ericxnelson opened 4 years ago

ericxnelson commented 4 years ago

I'm just finishing up migrating from carbon to graphouse/clickhouse and found that we have a bunch of snmp metrics that failed to backfill/load. I can work around this by using numeric MIBs but thought that I should add an issue here to help others in case they don't have that workaround.

Something like the following will work on graphite/carbon but fails to ingest into clickhouse/graphouse.


root@admin02:~# echo "test.bash.st-a::ts 42 `date +%s`" | nc localhost 2023                                           
root@admin02:~# echo "test.bash.st-a::ts 42 `date +%s`" | nc localhost 2023
root@admin02:~# echo "test.bash.st-a::ts 42 `date +%s`" | nc localhost 2023 ```
ArtemChekunov commented 4 years ago

@ericxnelson

I got a similar issue

but in your case, it may be a problem with metric-validation settings looks like :: is not allowed 

graphouse.metric-validation.regexp=[-_0-9a-zA-Z\\.]*$

https://github.com/ClickHouse/graphouse/blob/master/src/main/resources/graphouse-default.properties#L90