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

Support long Docker Registry passwords #5678

Closed
ghost opened this issue Jul 17, 2017 · 5 comments
Closed

Support long Docker Registry passwords #5678

ghost opened this issue Jul 17, 2017 · 5 comments
Labels
kind/enhancement A feature request - must adhere to the feature request template.

Comments

@ghost
Copy link

ghost commented Jul 17, 2017

From @wernight on March 18, 2017 8:57

Reproduction Steps:

  1. Go to /dashboard/#/administration URL
  2. Click "Add Registry"
  3. Fill in:
    • Address: https://gcr.io
    • Username: _json_key
    • Password: Content of the file generated.

Should support long password to support for example pulling from Google Container Registry (which generates a JSON file which is the password and is about 2000 characters long).

Codenvy version: 5.5.0

Copied from original issue: codenvy/codenvy#1948

@ghost
Copy link
Author

ghost commented Jul 17, 2017

@wernight as a workaround, you may add your gcr details in codenvy.env.

Marking this one as improvement.

@ghost ghost added kind/eclipse-che kind/enhancement A feature request - must adhere to the feature request template. labels Jul 17, 2017
@ghost
Copy link
Author

ghost commented Jul 17, 2017

From @wernight on March 21, 2017 19:44

Set in codenvy.env:

CODENVY_DOCKER_REGISTRY_AUTH_REGISTRY1_URL=https://eu.gcr.io
CODENVY_DOCKER_REGISTRY_AUTH_REGISTRY1_USERNAME=_json_key
CODENVY_DOCKER_REGISTRY_AUTH_REGISTRY1_PASSWORD='<content_of_file>'

It does no appear in the admin registry UI, which may be normal, but seems not working even though if I try manually on the CLI it works. Anyway using the codenvy.env should probably work and I probably made a mistake somewhere.

@ghost
Copy link
Author

ghost commented Jul 17, 2017

From @wernight on March 31, 2017 8:3

Note: This also affects Che.

@ghost
Copy link
Author

ghost commented Jul 17, 2017

@wernight yes, correct. And yes, not showing registries added in .env file in User Dashboard is expected.

@azatsarynnyy
Copy link
Member

fixed by #13569

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

No branches or pull requests

2 participants