Skip to content
This repository was archived by the owner on Feb 12, 2026. It is now read-only.

fix: match metrics names to old metrics names#10

Merged
kmchicoine merged 1 commit intomasterfrom
kaley/sidecrush-metric-names
Nov 17, 2025
Merged

fix: match metrics names to old metrics names#10
kmchicoine merged 1 commit intomasterfrom
kaley/sidecrush-metric-names

Conversation

@kmchicoine
Copy link
Contributor

Updates the metric names emitted by blockbuilding sidecar to match the old metric names to avoid redoing datadog dashboards

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Nov 17, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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 0
Sum 1

@kmchicoine kmchicoine merged commit dbc3102 into master Nov 17, 2025
10 checks passed
mw2000 pushed a commit that referenced this pull request Feb 12, 2026
* spike writer

* have dbwriter consume from queue

* add backoff retry

* make ingresswriter own service

* move publisher to ingresswriter

* nits

* move publish to own func

* rename to ingress-rpc and ingress-writer

* add ingress-writer dockerfile

* remove unused dep

* use TIPS_INGRESS_WRITER_ name
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants