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

docker vs docker-command #3

Open
laszewsk opened this issue Oct 18, 2020 · 0 comments
Open

docker vs docker-command #3

laszewsk opened this issue Oct 18, 2020 · 0 comments

Comments

@laszewsk
Copy link
Member

Currently, the only way to install cloudmesh-docker using cloudmesh-installer is by downloading the "source" bundle, which includes over 30 additional packages:

Screen Shot 2020-10-18 at 5 46 21 PM

There's also a "docker" bundle that doesn't contain cloudmesh-docker:

Screen Shot 2020-10-18 at 5 51 20 PM

This PR adds cloudmesh-docker to the "docker" bundle so that users can install cloudmesh-docker without installing 30+ additional packages.

@reillymarkowitz
add cloudmesh-docker repo to docker bundle
a5904e0
@laszewsk
laszewsk requested changes 1 hour ago

Member
laszewsk left a comment
This has to be done differently.

docker here means cmsd which is a dockerized container of cms

whet we do here is a different thing we start a docker command that uses docker from within cms not as the other package the uses docker from the outside.

We realize this can lead to confusion, but we had so far no docker command. so we need to start thinking about propper bundle names. As we use docker before it is unwise to make that change as it could impact the documentation of cmsd and lead to big confusions.

I have not yet thought about a proper bundle name for the docker command.

maybe "docker-command" would be ok

@laszewsk

Member
laszewsk commented 1 hour ago
This is a good find, but not as easy to fix ...

@laszewsk

Member
laszewsk commented 1 hour ago
I have added a bundle "docker-command" but did not yet provide any documentation for it. I think it deserves special attention and adds to the documentation in cloudmesh-docker and cloudmesh-cmsd

I think we need to add this even to the man pages.

@reillymarkowitz

Author
reillymarkowitz commented 20 minutes ago
Closing this PR because 2d2933e introduces a new "docker-command" bundle that solves the issue

@reillymarkowitz reillymarkowitz closed this 20 minutes ago
@laszewsk

Member
laszewsk commented 19 seconds ago
reopen the pull request as we need to make sure the documentation gets fixed and addresses teh existence of

docker
vs
docker-command

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

1 participant