-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add Registry Credentials to Tests #13959
Conversation
8260680
to
cdc92f1
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.
I think that's a great change, I have one comment left inline.
4db6695
to
7b846fd
Compare
test-me-please |
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, minor nits
Also, I believe backporting to older branches will have a slight gotcha where the Helm option for |
@christarazi good catch. I'll make sure to do the backport myself, once this gets merged. |
7b846fd
to
0263ab0
Compare
test-me-please |
Builds was green, I'm re-trigerring to see if this PR helps with the current rate limiting. |
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.
One small nit on the commit message format: could you please add an empty line after the commit's subject? i.e.
Add Registry Credentials to Tests
In order to get around image registry pull limits,
credentials can be set.
This will make it easier to backport the commit, as otherwise our git grep
magic in the backport scripts doesn't seem to find commits formatted like this.
In order to get around image registry pull limits,
credentials can be set.
Signed-off-by: Nate Sweet nathanjsweet@pm.me