Skip to content

chore(shared): Prometheus Server Utility #399

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
refcell merged 4 commits into from
Jan 13, 2026
Merged

chore(shared): Prometheus Server Utility #399

refcell merged 4 commits into from
Jan 13, 2026

Conversation

@refcell
Copy link
Contributor

@refcell refcell commented Jan 12, 2026

Summary

Adds a prometheus server utility that spins up a prometheus server at the given socket address.

This splits some basic logic out of #395 and will be used by base consensus.

@refcell refcell added this to the v0.4.0 milestone Jan 12, 2026
@refcell refcell requested a review from danyalprout January 12, 2026 23:14
@refcell refcell self-assigned this Jan 12, 2026
@refcell refcell added K-enhancement Kind: New feature or request A-shared Area: shared crates labels Jan 12, 2026
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 12, 2026 •
edited
Loading

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 2/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

@refcell refcell mentioned this pull request Jan 12, 2026
Closed
danyalprout
danyalprout reviewed Jan 12, 2026
View reviewed changes
danyalprout
danyalprout requested changes Jan 13, 2026
View reviewed changes
Copy link
Collaborator

@danyalprout danyalprout left a comment

Choose a reason for hiding this comment

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

needs a rebase, but lgtm

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 13, 2026

Review Error for danyalprout @ 2026-01-13 03:45:56 UTC
User failed mfa authentication, either user does not exist or public email is not set on your github profile. \ see go/mfa-help

@haardikk21
Copy link
Collaborator

haardikk21 commented Jan 13, 2026

LGTM too, can approve/merge after rebase

haardikk21
haardikk21 previously approved these changes Jan 13, 2026
View reviewed changes
@danyalprout danyalprout enabled auto-merge January 13, 2026 14:52
@danyalprout danyalprout added this pull request to the merge queue Jan 13, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 13, 2026
@refcell refcell added this pull request to the merge queue Jan 13, 2026
Merged via the queue into main with commit da69413 Jan 13, 2026
29 checks passed
@refcell refcell deleted the rf/prometheus branch January 13, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danyalprout danyalprout danyalprout approved these changes

@haardikk21 haardikk21 haardikk21 approved these changes

Assignees

@refcell refcell

Labels

A-shared Area: shared crates K-enhancement Kind: New feature or request

Projects

None yet

Milestone

v0.4.0

Development

Successfully merging this pull request may close these issues.

5 participants

@refcell @cb-heimdall @haardikk21 @danyalprout