Skip to content
This repository has been archived by the owner on Jan 19, 2018. It is now read-only.

Changing namespace format to namespace_IMAGENAME_hashvalue #397

Closed
cdrage opened this issue Nov 12, 2015 · 5 comments
Closed

Changing namespace format to namespace_IMAGENAME_hashvalue #397

cdrage opened this issue Nov 12, 2015 · 5 comments
Assignees
Milestone

Comments

@cdrage
Copy link
Member

cdrage commented Nov 12, 2015

When deploying an atomicapp it's namespace is commonly used as default. From our use cases as well as multiple examples that have appeared, many users are encountering the issue of "this namespace is already in use" when deploying another example without a configured answers.conf file.

@dustymabe and I encountered these issues recently when working on a multi-inherited Nuleculized application.

I propose that we change namespacing containers from:

atomic_NAMESPCE_HASHVALUE

to

NAMESPACE_IMAGENAME_HASHVALUE

In order to indicate what containers have been deployed by atomicapp.

I also propose that we change the value 'default' as a namespace to 'atomicapp' to better indicate what has been deployed.

@dustymabe
Copy link
Contributor

@charliedrage To be clear we are only talking about doing this if the user didn't provide "--name" in the docker run command. We can't change that right now because of docker links.

I like NAMESPACE_IMAGENAME_HASHVALUE. I don't know if I like making atomicapp the default namespace though. I like keeping it set as default, but I also like some indication that this is an atomicapp. Perhaps we can just do atomicapp_NAMESPACE_IMAGENAME_HASHVALUE.

@cdrage
Copy link
Member Author

cdrage commented Nov 17, 2015

@dustymabe
Running into a naming issue.

Only [a-zA-Z0-9][a-zA-Z0-9_.-] are only allowed in the image name.

For example, if we have centos/httpd:latest. What should we convert it as, centos-httpd-latest, centos.httpd.latest?

@dustymabe
Copy link
Contributor

I would go with dashes

@dustymabe
Copy link
Contributor

cdrage is this fixed now with #422?

@cdrage
Copy link
Member Author

cdrage commented Dec 3, 2015

Yup. Thought that this would close automatically with the merge comment ^^ Closin'.

@cdrage cdrage closed this as completed Dec 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants