Skip to content

Add support for exporting some basic metrics - #131

Open
refi64 wants to merge 1 commit into
mainfrom
wip/refi64/metrics
Open

Add support for exporting some basic metrics#131
refi64 wants to merge 1 commit into
mainfrom
wip/refi64/metrics

Conversation

@refi64

@refi64 refi64 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Only a few to start with, but they should be useful for helping to debug collabora/lava-gitlab-runner#129. Client crates can just set up the metrics exporter of their choice to be able to use this.

K: Eq + std::hash::Hash,
{
fn update_running_count(&self, n: usize) {
metrics::gauge!(METRICS_GAUGE_JOBS_RUNNING).set(n as u32);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should there be a gauge for this as i'ts derivable from finished - start ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I figured since the origin story of this was debugging-related, having a value tied directly to the runlist vs start/end being a layer further up might be useful. I'm not particularly attached though and would be fine with removing it if preferred.

Only a few to start with, but they should be useful for helping to debug
collabora/lava-gitlab-runner#129. Client
crates can just set up the metrics exporter of their choice to be able
to use this.
@refi64
refi64 force-pushed the wip/refi64/metrics branch from 9de6cf7 to 2140e68 Compare August 3, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants