-
Notifications
You must be signed in to change notification settings - Fork 138
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
chore: Add queue depth metrics for provisioning and termination #1060
chore: Add queue depth metrics for provisioning and termination #1060
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jonathan-innis The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
deab3d8
to
bca6d51
Compare
Pull Request Test Coverage Report for Build 8152913054Details
💛 - Coveralls |
) | ||
|
||
var ( | ||
schedulingSimulationDuration = prometheus.NewHistogramVec( |
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.
Should we add testing to ensure this metrics are working as expected?
97926d4
to
b461789
Compare
b461789
to
022e5d7
Compare
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.
/lgtm
Fixes #N/A
Description
This changes adds the
karpenter_nodes_eviction_queue_depth
andkarpenter_provisioner_scheduling_queue_depth
to give metrics on the number of items that are currently in these queues. This will allow us to get better insight into how quickly we are processing through items and also to know if we are making progress.How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.