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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃悰 Specify controller manager broadcaster options so events do not get discarded #1124
馃悰 Specify controller manager broadcaster options so events do not get discarded #1124
Conversation
heads up to reviewers: I am on holiday until September 25th. If you have feedback, feel free to cherry-pick this branch! |
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.
Some nits on the go.mod
changes, otherwise this looks great :)
@detiber do you want to merge & then do a follow-up, or close and open a replacement? |
c538606
to
89bea5b
Compare
89bea5b
to
b98d3ef
Compare
55980f6
to
d144d60
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.
/approve
/assign @ncdc
for final lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liztio, vincepri 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 |
d144d60
to
4b22bf9
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
What this PR does / why we need it:
Currently we are losing deletion events if they occur soon after cluster creation, and possibly other events we don't know about. This ups the limits of events reported in a given window so all events are submitted to the Kubernetes API.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #1068