Skip to content

apollo_network_benchmark: moved metric declarations to crate - #12724

Closed
sirandreww-starkware wants to merge 1 commit into
02-18-apollo_network_benchmark_renamed_node_binaryfrom
02-18-apollo_network_benchmark_moved_metric_declerations_to_crate
Closed

sirandreww-starkware wants to merge 1 commit into
02-18-apollo_network_benchmark_renamed_node_binaryfrom
02-18-apollo_network_benchmark_moved_metric_declerations_to_crate

Conversation

@sirandreww-starkware

@sirandreww-starkware sirandreww-starkware commented Feb 19, 2026 •

Copy link
Copy Markdown
Contributor

Note

Low Risk
Pure refactor that relocates metric declarations without changing metric names or runtime logic; risk is limited to potential import/path or linkage issues.

Overview
Moves the define_metrics! declarations for all apollo_network_benchmark gauges/counters/histograms from the apollo_network_benchmark_node binary into a new shared apollo_network_benchmark::metrics module, and exposes it via lib.rs.

Updates the node binary’s metrics.rs to pub use apollo_network_benchmark::metrics::* and drops now-unneeded imports/constants, while keeping the existing registration and helper functions intact.

Written by Cursor Bugbot for commit 3c1df25. This will update automatically on new commits. Configure here.

@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

sirandreww-starkware commented Feb 19, 2026 •

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@sirandreww-starkware
sirandreww-starkware marked this pull request as ready for review February 19, 2026 07:27
@sirandreww-starkware
sirandreww-starkware force-pushed the 02-18-apollo_network_benchmark_moved_metric_declerations_to_crate branch from f32a6eb to 9fd8303 Compare February 19, 2026 08:04

@guy-starkware guy-starkware left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@guy-starkware reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on sirandreww-starkware).

@sirandreww-starkware
sirandreww-starkware force-pushed the 02-18-apollo_network_benchmark_moved_metric_declerations_to_crate branch from 9fd8303 to 80cca2e Compare March 16, 2026 15:13
@sirandreww-starkware sirandreww-starkware changed the title apollo_network_benchmark: moved metric declerations to crate apollo_network_benchmark: moved metric declarations to crate Mar 22, 2026
@sirandreww-starkware
sirandreww-starkware force-pushed the 02-18-apollo_network_benchmark_renamed_node_binary branch from 1e9e611 to b2b0bd8 Compare March 22, 2026 17:09
@sirandreww-starkware
sirandreww-starkware force-pushed the 02-18-apollo_network_benchmark_moved_metric_declerations_to_crate branch from 80cca2e to 9bb267b Compare March 22, 2026 17:09
@sirandreww-starkware
sirandreww-starkware force-pushed the 02-18-apollo_network_benchmark_renamed_node_binary branch from b2b0bd8 to 8d3bd45 Compare March 30, 2026 11:08
@sirandreww-starkware
sirandreww-starkware force-pushed the 02-18-apollo_network_benchmark_moved_metric_declerations_to_crate branch from 9bb267b to 3c1df25 Compare March 30, 2026 11:08
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 28, 2026
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