Re: Re: Re: Re: [DISCUSS] KIP-972: Add the metric of the current running version of kafka

2023-10-19 Thread hudeqi
Hi, Mickael, Sophie, Doğuşcan. Thanks for your replys. I like the idea of adding the version and commitId as tags for existed metric named "startTimeMs". I will update the KIP doc and start the vote process for this KIP. It seems that there is not an existing metric named "uptime", Doğuşcan. bes

Re: Re: Re: [DISCUSS] KIP-972: Add the metric of the current running version of kafka

2023-10-11 Thread Doğuşcan Namal
Hello, do we have a metric showing the uptime? We could tag that metric with version information as well. I like the idea of adding the version as a tag as well. However, I am not inclined to tag each metric with a KafkaVersion information. We could discuss which metrics could be tagged but let's

Re: Re: Re: [DISCUSS] KIP-972: Add the metric of the current running version of kafka

2023-10-10 Thread Sophie Blee-Goldman
Just to chime in here since I recently went through a similar thing, I support adding the version as a tag instead of introducing an entirely new metric for this. In fact I just implemented exactly this in a project that uses Kafka, for these reasons: 1. Adding the version as a tag means that all

Re: Re: Re: [DISCUSS] KIP-972: Add the metric of the current running version of kafka

2023-10-10 Thread Mickael Maison
Hi Hudeqi, Rather than creating a gauge with a dummy value, could we add the version (and commitId) as tags to an existing metric. For example, the alongside the existing Version and CommitId metrics we have StartTimeMs. Maybe we can have a StartTimeMs metrics with the version and commitId) as tag

Re: Re: Re: [DISCUSS] KIP-972: Add the metric of the current running version of kafka

2023-08-30 Thread hudeqi
Thank you for your answer, Mickael. If set the value of gauge to a constant value of 1, adding that tag key is "version" and value is the version value of the obtained string type, does this solve the problem? We can get the version by tag in prometheus. best, hudeqi "Mickael Maison"