Skip to content

logthrdest: add active_worker_partitions and workers metrics#866

Merged
bazsi merged 3 commits intoaxoflow:mainfrom
MrAnno:partition-metrics
Dec 8, 2025
Merged

logthrdest: add active_worker_partitions and workers metrics#866
bazsi merged 3 commits intoaxoflow:mainfrom
MrAnno:partition-metrics

Conversation

@MrAnno
Copy link
Contributor

@MrAnno MrAnno commented Dec 2, 2025

Depends on #855

@MrAnno MrAnno requested review from bshifter and sodomelle December 2, 2025 12:59
@MrAnno MrAnno force-pushed the partition-metrics branch from 81582a6 to 18bf8d0 Compare December 3, 2025 15:08
@MrAnno MrAnno changed the title logthrdest: add active_partitions metric logthrdest: add active_worker_partitions and workers metrics Dec 3, 2025
Copy link
Member

@bazsi bazsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me.

@bazsi
Copy link
Member

bazsi commented Dec 8, 2025

I tried both of the new metrics and they do seem to report the right values.

syslogng_output_active_worker_partitions{driver="http",url="https://httpbin.org/status/200",id="#anon-destination0#0"} 3
syslogng_output_workers{driver="http",url="https://httpbin.org/status/200",id="#anon-destination0#0"} 128

@bazsi bazsi merged commit 396704e into axoflow:main Dec 8, 2025
22 checks passed
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

Successfully merging this pull request may close these issues.

2 participants