Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

feat(registry): add private registry documentation #201

Merged
merged 1 commit into from
Apr 26, 2016

Conversation

helgi
Copy link
Contributor

@helgi helgi commented Apr 26, 2016

No description provided.

@helgi helgi self-assigned this Apr 26, 2016
@helgi helgi added this to the v2.0-beta3 milestone Apr 26, 2016
@mboersma mboersma added the docs label Apr 26, 2016

In order to deploy Docker images from a private repository on to Deis Workflow the following steps
Copy link
Member

@mboersma mboersma Apr 26, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe simpler:

To deploy Docker images from a private registry, take the following steps:

  • Gather the username and password for the registry, such as a [Quay.io Robot Accounts][] or [GCR.io Long Lived Token][]

@krancour
Copy link
Contributor

I'd be careful about using registry and repository interchangeably. It might be more precise to indicate that this can be used for private registries (e.g. if you host your own) or private repositories within a public registry like Dockerhub or quay.io.

On a separate note, there's mention of compacting the JSON blob for a long-lived token, but there's no mention of how that can be applied. Is the compacted token just substituted as the value of the --password flag? Or has it got its own dedicated flag?

@helgi
Copy link
Contributor Author

helgi commented Apr 26, 2016

It's in the password - It's all listed out in the GCR documentation. I can update that

@helgi helgi force-pushed the registry branch 2 times, most recently from c92a9f4 to e526ab1 Compare April 26, 2016 18:51

To deploy Docker images from a private registry (either from self-hosted registry or one of the public ones),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: should say "from a self-hosted registry"


To deploy Docker images from a private registry (either from a self-hosted registry or one of the public ones),
take the following steps:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change:

"To deploy Docker images from a private registry or even from a private repository within a public registry, take the following steps:"


* Gather the username and password for the registry, such as a [Quay.io Robot Accounts][] or [GCR.io Long Lived Token][]
* Run `deis registry:set username=<the-user> password=<secret> -a <application-name>`
* Now do `deis pull` as normal, against an image in the private registry
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would "perform" sound better than "do"?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Total nit... non-blocking. LGTM2

@rimusz
Copy link
Contributor

rimusz commented Apr 26, 2016

👍

@skevy
Copy link

skevy commented Jun 16, 2016

@helgi Are there any thoughts on how to make an entire deis cluster have access to a private registry? Or is that not in the cards/very technically difficult?

@helgi
Copy link
Contributor Author

helgi commented Jun 16, 2016

Right now it is only per application. If we ever do deis/controller#383 then registry data may get included in that as well

@skevy
Copy link

skevy commented Jun 16, 2016

That would be great. Basically I don't want to have members of my team have to ask me for a registry key :)

@helgi
Copy link
Contributor Author

helgi commented Jun 16, 2016

My recommendation would be to make a read-only robot account credentials available to them, in the short term

@skevy
Copy link

skevy commented Jun 16, 2016

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants