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

reference management in image layout #374

Closed
xiekeyang opened this issue Oct 8, 2016 · 3 comments
Closed

reference management in image layout #374

xiekeyang opened this issue Oct 8, 2016 · 3 comments

Comments

@xiekeyang
Copy link
Contributor

Current spec doesn't care the layout management for ref folder. Actually consumer can set up many child folder and put the reference file to any where. It seems to be a little chaos. I think it had better to build rules for this. like:

<domain>/<repository>/<reference>

domain identifies a directory.
repository identifies the image name, which may be multi-level directory, like library/ubuntu
reference identifies the image tag.

the tree looks like:

.
└── refs
    └── docker.registry.io
        └── library
            └── ubuntu
                └── latest
                └── 14.04
            └── busybox
                └── latest

Any ref layout inconsistent to above will be regarded as illegal.

Previous comment shows that the image layout actually play as an image repository, which contains many images(as it contains manifest list). So I think the directory should be managed with multi-levels.

@wking
Copy link
Contributor

wking commented Oct 10, 2016

On Sat, Oct 08, 2016 at 01:10:57AM -0700, xiekeyang wrote:

Current spec doesn't care the layout management for ref
folder. Actually consumer can set up many child folder and put the
reference file to any where. It seems to be a little chaos. I think
it had better to build rules for this. like:

<domain>/<repository>/<reference>

This is not legal, because ‘/’ is not in the [A-Za-z0-9-._] whitelist
1. Previous discussion in #174 and #219.

@stevvooe
Copy link
Contributor

Closed by #438

@xiekeyang
Copy link
Contributor Author

closed by #438

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

No branches or pull requests

3 participants