Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Incorrect postfix naming behaviour for tagged metrics #85

Open
andriicr opened this issue Feb 13, 2024 · 0 comments
Open

[Bug] Incorrect postfix naming behaviour for tagged metrics #85

andriicr opened this issue Feb 13, 2024 · 0 comments

Comments

@andriicr
Copy link

Hello.
I discovered that when sending metrics the postfix is not added.
Important note on task context: must be explicitly simulated statsd behaviour

When sending metrics without tags:
echo "test.counter:1|c" | ncat -v -u 127.0.0.1 52125
expected/actual result(ALL IS OK):
test.counter.count 1.0 1707753580

When sending metrics with tags(HERE IS THE PROBLEM):
echo "test.counter;foo=bar:1|c" | ncat -v -u 127.0.0.1 52125
actual result: (postfix ".count" is not set)
test.counter;foo=bar 1.0 1707753750

expected result:
test.counter.count;foo=bar 1.0 1707753750

used config.toml:

verbosity-console = "warn"
n-threads = 4
p-threads = 2
w-threads = 4
a-threads = 4
task-queue-size = 2048
start-as-leader = true
consensus = "none"
stats-interval = 10000
stats-prefix = "resources.monitoring.bioyino"

[carbon]
address = "127.0.0.1:50003"
interval = 10000
connect-delay = 250
connect-delay-multiplier = 2
connect-delay-max = 10000
send-retries = 30
chunks = 8

[network]
listen = "127.0.0.1:52125"
peer-listen = "127.0.0.1:52136"
mgmt-listen = "127.0.0.1:52137"
bufsize = 1500
multimessage = true
mm-packets = 100
mm-async = false
buffer-flush-time = 100
buffer-flush-length = 16384
snapshot-interval = 1000

[metrics]
log-parse-errors = true

[aggregation]
round-timestamp = "down"
aggregates.counter = [ "value" ]
aggregates.timer = [ "count", "rate", "min", "max", "sum", "mean", "median", "percentile-90" ]
aggregates.gauge = [ "value" ]
aggregates.set = [ "count" ]

[naming.default]
destination = "name"
prefix = ""

[naming.counter]
prefix = ""
postfixes = { "value" = "count" }

[naming.timer]
prefix = ""
postfixes = { "rate" = "count_ps", "min" = "lower", "max" = "upper", "percentile-90" = "upper_90" }

[naming.gauge]
prefix = ""

[naming.set]
prefix = ""
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant