Skip to content

adds description for hosts.toml#5309

Merged
estesp merged 2 commits intocontainerd:masterfrom
mikebrow:document-hosts.toml
May 12, 2021
Merged

adds description for hosts.toml#5309
estesp merged 2 commits intocontainerd:masterfrom
mikebrow:document-hosts.toml

Conversation

@mikebrow
Copy link
Copy Markdown
Member

@mikebrow mikebrow commented Apr 6, 2021

Putting together some docs to describe configuring registries, authentication, and mirrors.

Signed-off-by: Mike Brown [email protected]

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 6, 2021

Build succeeded.

@mikebrow mikebrow force-pushed the document-hosts.toml branch from 165431f to 0c1dba0 Compare April 7, 2021 01:44
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 7, 2021

Build succeeded.

@mikebrow mikebrow force-pushed the document-hosts.toml branch from 0c1dba0 to cfe793b Compare April 7, 2021 01:56
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 7, 2021

Build succeeded.

@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Apr 8, 2021

Thanks for adding,

A few notes

  • This document does not need to be in the cri subdirectory since the format is not cri specific
  • Explaining what registry host namespace upfront is useful, this just needs to be mapped to the first part of an image name. For example docker.io for Docker Hub images, and some.domain.io:8000 an image like some.domain.io:8000/image:latest
  • Right now it is following the same endpoint resolution rules established within Docker, this includes using the "/v2" when it is not provided. If /v2 is provided in the server url, /v2 will not be appended. I think it is worth mentioning that the registry endpoint will always resolve to ending in /v2 as currently defined in distribution-spec

@kzys kzys added this to the 1.5 milestone Apr 22, 2021
@kzys
Copy link
Copy Markdown
Member

kzys commented Apr 22, 2021

I've added the issue to 1.5 milestone. While the release note explains the file format briefly, we should have this document in addition to the release note.

Thoughts?

Comment thread docs/cri/hosts.md Outdated
@mikebrow mikebrow force-pushed the document-hosts.toml branch 4 times, most recently from d4f4d57 to 6150556 Compare April 30, 2021 23:47
@mikebrow mikebrow changed the title [WIP] adds description for hosts.toml adds description for hosts.toml Apr 30, 2021
@mikebrow mikebrow force-pushed the document-hosts.toml branch 2 times, most recently from 427c282 to 0cfaaf5 Compare May 1, 2021 00:10
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented May 1, 2021

Build succeeded.

@mikebrow mikebrow force-pushed the document-hosts.toml branch from 0cfaaf5 to b59e297 Compare May 2, 2021 00:49
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented May 2, 2021

Build succeeded.

Comment thread docs/hosts.md Outdated
@dmcgowan dmcgowan modified the milestones: 1.5, 1.5.1 May 3, 2021
@mikebrow mikebrow force-pushed the document-hosts.toml branch 2 times, most recently from c9a00f3 to 07420d7 Compare May 4, 2021 09:56
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented May 4, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

Looks good overall; just some final formatting/typo comments/fixups

Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
Comment thread docs/hosts.md Outdated
@mikebrow mikebrow force-pushed the document-hosts.toml branch from 07420d7 to 6e249b1 Compare May 12, 2021 13:38
@mikebrow
Copy link
Copy Markdown
Member Author

Looks good overall; just some final formatting/typo comments/fixups

thx... resolved.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented May 12, 2021

Build succeeded.

@estesp estesp merged commit a5f19b0 into containerd:master May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants