-
Notifications
You must be signed in to change notification settings - Fork 62
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
Disable default docker config provider #255
Conversation
FYI: There appears to be some incompatible changes made upstream (kubernetes) to auth providers. Not sure when / if those changes will eventually land on the fork we are using... Specifically... amazon-ecr is always enabled: https://github.com/kubernetes/kubernetes/blob/master/pkg/credentialprovider/aws/aws_credentials.go#L77-L80 And will return an empty dockerconfig if it is 'disabled': https://github.com/kubernetes/kubernetes/blob/master/pkg/credentialprovider/aws/aws_credentials.go#L173 This change will require changes on imgpkg, since we will treat that case as 'we found a cred for your image... try using an empty username / password' |
- This interferes with the ordering of auth imgpkg uses. -- Specifically, it will prefer the docker config.json file over the auth cli flags. - There doesn't seem to be a nice way to configure / turn this provider off. Patching the vendor directory. Will try to make the change upstream. Authored-by: Dennis Leon <leonde@vmware.com>
Authored-by: Dennis Leon <leonde@vmware.com>
657abd0
to
e8ca6f5
Compare
Due to the inclusion of imgpkg as a library we need to ensure that the change made in carvel-dev/imgpkg#255 stays in effect. This will behavior can be removed when we complete the issue carvel-dev/imgpkg#397 Signed-off-by: Joao Pereira <joaod@vmware.com>
Due to the inclusion of imgpkg as a library we need to ensure that the change made in carvel-dev/imgpkg#255 stays in effect. This will behavior can be removed when we complete the issue carvel-dev/imgpkg#397 Signed-off-by: Joao Pereira <joaod@vmware.com>
Due to the inclusion of imgpkg as a library we need to ensure that the change made in carvel-dev/imgpkg#255 stays in effect. This will behavior can be removed when we complete the issue carvel-dev/imgpkg#397 Signed-off-by: Joao Pereira <joaod@vmware.com>
Due to the inclusion of imgpkg as a library we need to ensure that the change made in carvel-dev/imgpkg#255 stays in effect. This will behavior can be removed when we complete the issue carvel-dev/imgpkg#397 Signed-off-by: Joao Pereira <joaod@vmware.com>
TODO:
- [ ] Submit an issue/PR upstream to allow disabling the default docker provider. Extend this change: kubernetes/kubernetes@a947c32#diff-0ebbc5bb19e182a89f5992c919d5fe2254af767f4611835010367b324b271df9R51-R53