Skip to content
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 get-login" spills out non-existing parameter for "docker login" #3374

Closed
dinamic opened this issue Jun 12, 2018 · 3 comments
Closed
Labels
guidance Question that needs advice or information.

Comments

@dinamic
Copy link

dinamic commented Jun 12, 2018

Using aws ecr get-login spills out login credentials that work, but the command fails to because of non existing parameter. I would guess it was removed by docker at some point.

Trying to login using the command that the aws cli outputs results in:

unknown shorthand flag: 'e' in -e
See 'docker login --help'.

Here are the acceptable parameters by docker login

# docker login --help

Usage:	docker login [OPTIONS] [SERVER]

Log in to a Docker registry

Options:
  -p, --password string   Password
      --password-stdin    Take the password from stdin
  -u, --username string   Username
@tyhunt99
Copy link

tyhunt99 commented Jun 12, 2018

You can pass in --no-include-email to the get-login command and it will remove the -e flag from the output

@dinamic
Copy link
Author

dinamic commented Jun 12, 2018

@tyhunt99 thanks for that! it works, but feels like a workaround.

The -e parameter might have made sense back in the days, but is no longer relevant. IMO it needs to go.

@kyleknap
Copy link
Member

It would be nice to remove it but we are keeping it for backwards compatibility reasons as users may still be using an old version of docker that use it. The solution that @tyhunt99 provided is going to be your best option moving forward.

@justnance justnance added the guidance Question that needs advice or information. label Dec 7, 2018
thoward-godaddy pushed a commit to thoward-godaddy/aws-cli that referenced this issue Feb 12, 2022
…ation after build (aws#3374)

* increase default process timeout for tests and update image validation after build

* add images information to assertion message

* fix: update tag checking and replace "verify_pulling_only_latest_tag" with "verify_pulled_image"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
guidance Question that needs advice or information.
Projects
None yet
Development

No branches or pull requests

4 participants