Hi Mahsa, I see you have the definition of the metric value as: controller idle ratio = idle_time/active_time
Shouldn't the value for a ratio be: controller idle ratio = idle_time/total_time where total_time = idle_time + active_time? This lines up with the definition you outlined earlier in the metric value description. Best, Kevin Wu On Tue, Jul 22, 2025 at 11:54 AM Mahsa Seifikar <mseifi...@confluent.io.invalid> wrote: > Thanks Jonah and Kevin for the feedback. > > I have updated the KIP accordingly. We ideally want to use something like > the TimeRatio type for this metric, similar to how "poll-idle-ratio" is > measured in KafkaRaftMetrics. > > Please let me know if you have any further feedback. > > Best, > Mahsa Seifikar > > On Fri, Jul 11, 2025 at 4:08 PM Kevin Wu <kevin.wu2...@gmail.com> wrote: > > > Hi Mahsa and Jonah, > > > > Since we're adding this new metric to a metrics group that is still using > > Yammer, ideally I think we want to use RatioGauge to give us the sampling > > functionality we need. It's possible that we can get similar > functionality > > from Histogram, which I know other Yammer metrics in Kafka use. We are > > still able to get gauge metrics from the histogram, as they are the most > > straightforward for the operator to monitor (e.g. if the metric value > > X, > > alert). For example, metrics that are histograms, like EventQueueTimeMs, > > are often monitored via their p99 or p999 value. > > > > There are several other "thread-idle-ratio" metrics in Kafka, but those > are > > all using the newer, internal KafkaMetrics library's sensors. > > > > Best, > > Kevin Wu > > > > On Thu, Jul 10, 2025 at 2:09 PM Mahsa Seifikar > > <mseifi...@confluent.io.invalid> wrote: > > > > > Hi Jonah and Kevin, > > > > > > Thanks for your comments. I have now updated the KIP to address your > > > feedback. > > > > > > Please let me know if you have any further questions. > > > > > > Best, > > > Mahsa Seifikar > > > > > > On Thu, Jul 3, 2025 at 4:40 PM Mahsa Seifikar <mseifi...@confluent.io> > > > wrote: > > > > > > > Hello all, > > > > > > > > I wrote a short KIP to add a new metric for controller thread > idleness. > > > > > > > > Here is the link: > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1190%3A+Add+a+metric+for+controller+thread+idleness > > > > > > > > Thanks, > > > > Mahsa Seifikar > > > > > > > > > > > > > > > > > >
