-
Notifications
You must be signed in to change notification settings - Fork 114
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
Update Get calls to use the lister cache #156
Update Get calls to use the lister cache #156
Conversation
/retest |
/hold waiting for review |
/retest |
1 similar comment
/retest |
/retest |
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.
couple nits, mostly lgtm though i'm slightly wary of whether this causes us problems in cases where the cache is not yet updated and thus our get fails (whereas the real api call would have succeeded).
hopefully our resync behavior is sufficient to resolve any temporary problems caused by such a thing.
in my ideal world we'd have a read-through cache though (attempt an api get if the cache get is a miss). Still doesn't address stale reads, but at least it would resolve straight up missing object cases.
/retest |
1 similar comment
/retest |
/retest |
Updates Get calls to use the lister cache. ClusterVersion and ClusterOperator are excluded as they provided a steady stream of events which caused a hotloop like issue of constant events being processed.
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bparees, coreydaley 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
4 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
/test e2e-aws-operator |
/retest |
8 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
Updates all(most) Get calls to use the lister cache.
ClusterVersion and ClusterOperator are excluded as they provided a steady stream of events which caused a hotloop like issue of constant events being processed.