graphouse icon indicating copy to clipboard operation
graphouse copied to clipboard

handling of metrics with : colon in them seems broken

Open ericxnelson opened this issue 5 years ago • 1 comments

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 ```

ericxnelson avatar Mar 11 '20 19:03 ericxnelson

@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

ArtemChekunov avatar Jul 10 '20 09:07 ArtemChekunov