-
Notifications
You must be signed in to change notification settings - Fork 667
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.
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
feat: Prometheus Block Fullness Metrics #3025
Changes from 3 commits
a7c0002
3b9a805
01975a1
e3fca43
6c41bfe
af311cc
eaa4a3a
b2700c4
4cf99f3
7fa6bcb
fde5901
ba45e6f
4be27a3
b88d8ce
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -91,6 +91,31 @@ lazy_static! { | |
"Total number of error logs emitted by node" | ||
)).unwrap(); | ||
|
||
pub static ref LAST_EXECUTION_READ_COUNT: IntGauge = register_int_gauge!(opts!( | ||
"execution_cost_read_count", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think these could be slightly more descriptively named for prometheus as There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the variable name or the caption part? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the prometheus metric name, i.e. pub static ref LAST_EXECUTION_READ_COUNT: IntGauge = register_int_gauge!(opts!(
"last_block_read_count", There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we also prepend There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Went with |
||
"`execution_cost_read_count` for the last block observed." | ||
)).unwrap(); | ||
|
||
pub static ref LAST_EXECUTION_WRITE_COUNT: IntGauge = register_int_gauge!(opts!( | ||
"execution_cost_write_count", | ||
"`execution_cost_write_count` for the last block observed." | ||
)).unwrap(); | ||
|
||
pub static ref LAST_EXECUTION_READ_LENGTH: IntGauge = register_int_gauge!(opts!( | ||
"execution_cost_read_length", | ||
"`execution_cost_read_length` for the last block observed." | ||
)).unwrap(); | ||
|
||
pub static ref LAST_EXECUTION_WRITE_LENGTH: IntGauge = register_int_gauge!(opts!( | ||
"execution_cost_write_length", | ||
"`execution_cost_write_length` for the last block observed." | ||
)).unwrap(); | ||
|
||
pub static ref LAST_EXECUTION_RUNTIME: IntGauge = register_int_gauge!(opts!( | ||
"execution_cost_runtime", | ||
"`execution_cost_runtime` for the last block observed." | ||
)).unwrap(); | ||
|
||
pub static ref ACTIVE_MINERS_COUNT_GAUGE: IntGauge = register_int_gauge!(opts!( | ||
"stacks_node_active_miners_total", | ||
"Total number of active miners" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to just group all of these statements inside a single
#[cfg(feature = "monitoring_prom")]
? Something like:There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did this