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

Added restart and default name to container #73

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jimzucker
Copy link

I recommend adding a default name 'nexus' and setting the restart option.

@@ -1,8 +1,10 @@
version: "2"
version: "2.1"
Copy link
Member

Choose a reason for hiding this comment

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

Why the change to 2.1? I did a quick review of 2.1 docs and couldn't really tell for sure why you made it. This would limit use to some newer Docker clients, hence the question.

@DarthHater
Copy link
Member

@jimzucker if you could update your description to describe why you recommend adding these options, that would be good. Otherwise we are left to guess as to why this would help you (and many others). It'll help us in determining if we should bring this change in for all of our users.

@jimzucker
Copy link
Author

@DarthHater I was using the docker container to setup a new client with a POC of nexus and i noticed for a shared env if the server was restarted the nexus disappeared and you would have to get somebody that knows the details to login and restart it. By adding restart option it makes for a much more reliable installation. Regarding the name similarly having a random name makes it hard to document procedures and explain the setup. With these 2 changes now you can quickly setup a very reliable shared instance of nexus in a matter of minutes, that is very clearly documented.

I was able to change the version back to "2".

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.

2 participants