Skip to content
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

Replace event reconciler with pod reconciler to reduce api-server load #33

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

toshipp
Copy link
Contributor

@toshipp toshipp commented Dec 2, 2022

Watching events is heavy task and produced information which we need can be generated from pods status. Thus, we replace event reconciler with pod one.

Co-authored-by: Satoru Takeuchi satoru.takeuchi@gmail.com
Co-authored-by: Shinya Hayashi shinya-hayashi@cybozu.co.jp

@toshipp toshipp requested a review from a team as a code owner December 2, 2022 03:05
cmd/controller.go Outdated Show resolved Hide resolved
@peng225 peng225 force-pushed the get-event-time-by-pod-reconciler branch from a68683d to 1d6b190 Compare December 2, 2022 04:21
@peng225 peng225 self-requested a review December 2, 2022 04:21
peng225
peng225 previously approved these changes Dec 2, 2022
Watching events is heavy task and produced information which we need can
be generated from pods status. Thus, we replace event reconciler with
pod one.

Co-authored-by: Satoru Takeuchi <satoru.takeuchi@gmail.com>
Co-authored-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
@peng225 peng225 force-pushed the get-event-time-by-pod-reconciler branch from 1d6b190 to 354668e Compare December 2, 2022 04:24
@toshipp toshipp removed the request for review from llamerada-jp December 2, 2022 04:36
@peng225 peng225 merged commit 1ba89cf into main Dec 2, 2022
@peng225 peng225 deleted the get-event-time-by-pod-reconciler branch December 2, 2022 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants