-
Notifications
You must be signed in to change notification settings - Fork 776
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
feat: use lister in registry client #5620
Conversation
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Codecov Report
@@ Coverage Diff @@
## main #5620 +/- ##
=======================================
Coverage 36.38% 36.38%
=======================================
Files 174 174
Lines 19157 19152 -5
=======================================
- Hits 6970 6969 -1
+ Misses 11407 11404 -3
+ Partials 780 779 -1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -92,7 +88,7 @@ func WithKeychainPullSecrets(ctx context.Context, kubClient kubernetes.Interface | |||
) | |||
return nil | |||
} | |||
return c.pullSecretRefresher(ctx, c) | |||
return nil |
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.
Return nil? We don't execute it?
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.
Nope, we run it lazily when a call has to be made.
The caller does not need to refresh anymore.
Signed-off-by: Md Sahil <Mohdssahil1@gmail.com>
Signed-off-by: MdSahil-oss <Mohdssahil1@gmail.com>
Signed-off-by: MdSahil-oss <Mohdssahil1@gmail.com>
Explanation
This PR uses a lister in registry client to fetch image pull secrets instead of making api calls.