-
Notifications
You must be signed in to change notification settings - Fork 71
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
Introduce a request counter to the auditor #359
Conversation
bb0e24a
to
7a2a632
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.
Replaced all mention of "query" counting with "request" counting.
Thanks! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: listx, tylerferrara 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 |
Introduce a request counter to the auditor
Introduce a request counter to the auditor
Introduce a request counter to the auditor
Introduce a request counter to the auditor
What type of PR is this?
/kind feature
What this PR does / why we need it:
This change creates a simple request counter, able to concurrently keep track of all HTTP requests made to GCR. The counter is only enabled when the user passes the
--verbose
flag. In the future, this counter will be logged to gain visibility in the network performance of the auditor.Supersedes #356
Which issue(s) this PR fixes:
Partially satisfies #358
Special notes for your reviewer:
The auditor does not yet call
Increment()
to add to the counter. This would perpetually increase the value of the counter if enabled. Once the concurrent logger is implemented (part of #358), which resets the counter every 10min, will the auditor callIncrement()
.Less importantly, this package is made specifically for the auditor, it can just as easily be used for other commands, like
cip run --snapshot
.Does this PR introduce a user-facing change?
NONE
cc: @listx @amwat @justaugustus @kubernetes-sigs/release-engineering