-
Notifications
You must be signed in to change notification settings - Fork 346
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
AWS_ECR_IGNORE_CREDS_STORAGE=true still raises "not implemented" for docker logins against ECR repos #922
Comments
Works fine on Linux:
This may be a Windows specific issue. |
Can you try running this command in a PowerShell? It will set the variable machine-wide. [Environment]::SetEnvironmentVariable("AWS_ECR_IGNORE_CREDS_STORAGE", "true", "Machine") You will likely have to re-open any console windows after running this (and maybe even restart docker). |
Hmm, that's definitely a little strange. Here's my setup, wondering if you can spot any notable differences, in case I missed something important when reproducing PS C:\Users\Administrator> cat C:\Users\Administrator\.docker\config.json
{
"auths": {},
"credsStore": "ecr-login",
"currentContext": "desktop-linux",
"plugins": {
"debug": {
"hooks": "exec"
},
"scout": {
"hooks": "pull,buildx build"
}
}
}
PS C:\Users\Administrator> docker login -u AWS -p $password 123456789012.dkr.ecr.us-west-2.amazonaws.com
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Login Succeeded Also just as a sanity check might want to see if you have |
Seeing the 0.9 release with this enhancment:
I expected if I called "docker login" with ECR creds (as returned by Get-EcrLoginCommand) that this variable would swallow, but I'm still seeing "not implemented"
With the 0.9 helper installed, and the AWS_ECR_IGNORE_CREDS_STORAGE set as a system environment variable (I restarted Docker Desktop and even my machine to ensure this env var is in all processes):
I then attempt a docker login:
My docker config.json is:
Am I doing something wrong? Am I expecting the wrong behaviour from this new environment variable?
Sincerely
Pete
The text was updated successfully, but these errors were encountered: