-
Notifications
You must be signed in to change notification settings - Fork 197
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
EN-12330: export metric erd_are_vm_queries_ready #4097
EN-12330: export metric erd_are_vm_queries_ready #4097
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4097 +/- ##
==========================================
- Coverage 74.82% 74.82% -0.01%
==========================================
Files 609 609
Lines 80936 80938 +2
==========================================
- Hits 60562 60561 -1
- Misses 15737 15739 +2
- Partials 4637 4638 +1
Continue to review full report at Codecov.
|
@@ -536,6 +536,10 @@ const MetricP2PUnknownPeers = "erd_p2p_unknown_shard_peers" | |||
// MetricP2PNumConnectedPeersClassification is the metric for monitoring the number of connected peers split on the connection type | |||
const MetricP2PNumConnectedPeersClassification = "erd_p2p_num_connected_peers_classification" | |||
|
|||
// MetricAreVMQueriesReady will hold the string representation of the boolean that indicated if the node is ready | |||
// to process VM queries | |||
const MetricAreVMQueriesReady = "erd_are_vm_queries_ready" |
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.
maybe you can remove the verb from the const name and value
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.
I would rather keep the naming, as the are
prefix resembles a boolean value
The base branch was changed.
…ot-ready-for-vm-queries
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.
System test passed.
export a metric that will tell if the node is ready for processing VM Queries