-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Better metric name for nomad.broker.total_blocked #6480
Comments
This came up again recently while debugging #13407 and having a metric named const (
EvalStatusBlocked = "blocked"
EvalStatusPending = "pending"
EvalStatusComplete = "complete"
EvalStatusFailed = "failed"
EvalStatusCancelled = "canceled"
) The eval broker's internal state is never reflected on the The current stats are (ref
Some options:
|
The metric
broker.total_blocked
is poorly named. It is actually counting the number of pending evals for the same job in Nomad's eval broker.Proposing renaming this (better name TBD) with a deprecation path. Plan is to log the metric with two names in a point release. The old name will be removed in a subsequent major release with deprecation notes.
The text was updated successfully, but these errors were encountered: