Dabz / ccloudexporter

Prometheus exporter for Confluent Cloud API metric
https://docs.confluent.io/current/cloud/metrics-api.html
87 stars 53 forks source link

Misleading logs for metrics listener #83

Closed subkanthi closed 2 years ago

subkanthi commented 3 years ago

This looks like a substitution error, but its very misleading to think that there is something wrong with the host name

{ "level": "info", "msg": "Listening on http://:2112/metrics\n", "time": "2021-07-22T18:34:31Z" }

Dabz commented 3 years ago

Do you mean it's confusing because Listening could be understand as network listener? The default Listener is empty, meaning it is listening on all interfaces, I could understand why it could be confusing as I am adding http://%s/metrics to it. I will change this log in the next major iteration :+1:

Dabz commented 3 years ago

I changed the log message as part of #93 . Somehow, I really struggle to find a good name ^^ Do you have a better idea?