Skip to content

RA: Fix legacy override utilization metrics#7124

Merged
pgporada merged 6 commits into
mainfrom
ratelimit-fix-override-util
Nov 20, 2023
Merged

RA: Fix legacy override utilization metrics#7124
pgporada merged 6 commits into
mainfrom
ratelimit-fix-override-util

Conversation

@beautifulentropy

@beautifulentropy beautifulentropy commented Oct 30, 2023

Copy link
Copy Markdown
Member
  • Emit override utilization only when resource counts are under threshold.
  • Override utilization accounts for anticipated issuance.
  • Correct the limit metric label for CertificatesPerName and CertificatesPerFQDNSet/Fast.

Part of #5545

@beautifulentropy beautifulentropy requested a review from a team as a code owner October 30, 2023 18:13
@beautifulentropy beautifulentropy force-pushed the ratelimit-fix-override-util branch from bc19858 to 64dad81 Compare October 30, 2023 18:14
@beautifulentropy beautifulentropy marked this pull request as draft October 30, 2023 18:15
@beautifulentropy beautifulentropy force-pushed the ratelimit-fix-override-util branch from 64dad81 to ee548bc Compare October 30, 2023 18:25
@beautifulentropy beautifulentropy force-pushed the ratelimit-fix-override-util branch from 6bbc6bd to 85deaa2 Compare November 6, 2023 18:51
@beautifulentropy beautifulentropy marked this pull request as ready for review November 6, 2023 19:02
Comment thread ra/ra.go Outdated
Comment thread ra/ra.go Outdated
Comment thread ra/ra.go
@aarongable aarongable requested a review from a team November 10, 2023 00:32
@pgporada pgporada merged commit e1312ff into main Nov 20, 2023
@pgporada pgporada deleted the ratelimit-fix-override-util branch November 20, 2023 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants