Skip to content

Export job state metrics for Prometheus - #1114

Merged
QnJ1c2kNCg merged 1 commit into
ArroyoSystems:masterfrom
QnJ1c2kNCg:broy/job-status-metrics-oss
Jul 27, 2026
Merged

Export job state metrics for Prometheus#1114
QnJ1c2kNCg merged 1 commit into
ArroyoSystems:masterfrom
QnJ1c2kNCg:broy/job-status-metrics-oss

Conversation

@QnJ1c2kNCg

@QnJ1c2kNCg QnJ1c2kNCg commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Emit job state Prometheus metrics as part of the controller updater loop.


Open in Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

Comment thread crates/arroyo-controller/src/lib.rs Outdated
.unwrap();
}

fn metric_job_state(

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.

Is there a specific reason we re-map all states instead of exposing the states themselves? There is a lot of chance for drift between here and what states may actually exist.

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.

No strong reason, I initially thought having larger "buckets" would be nicer. However, I agree about the drift risk and it's easy to create "buckets" in dashboards/alerts.

I'll update the PR.

@QnJ1c2kNCg
QnJ1c2kNCg force-pushed the broy/job-status-metrics-oss branch from 2ce6239 to d356852 Compare July 27, 2026 20:16
@QnJ1c2kNCg
QnJ1c2kNCg merged commit e78f7b5 into ArroyoSystems:master Jul 27, 2026
4 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.

3 participants