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

integrate vagrant module with cm4 #89

Merged
merged 1 commit into from
Dec 7, 2018
Merged

Conversation

kimballXD
Copy link
Contributor

No description provided.

@kimballXD kimballXD merged commit 4219535 into cloudmesh:master Dec 7, 2018
Copy link
Member

@laszewsk laszewsk left a comment

Choose a reason for hiding this comment

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

please avoid screenshots and use

text

instead

@kimballXD
Copy link
Contributor Author

now vagrant module do not work with mongoDB, working toward integrating with mongoDB

@laszewsk
Copy link
Member

laszewsk commented Dec 7, 2018

can you comment on how you manage the vms

e.g. are you doing it like i did, e.g. a dir for each vm with its vagrant file or something else?

@kimballXD
Copy link
Contributor Author

I believe from the very beginning there is always one Vagrnatfile to handle All vms (which Vagrantfile should be saved at ~/.cloudmesh/vagrant_workspace/Vagrnatfile), since Vagrant can only handle one Vagrantfile at a time. If you use multiple Vagrantfile and one Vagrantfile for a single VM, then when you issue cm4 vagrant list, you can only get infos for one VM instead of all vms.

@laszewsk
Copy link
Member

laszewsk commented Dec 7, 2018

good analysis, this means we go with my approach to place it for each vm in ~/.cloudmesh/cagrant/NAMEOFVM

As we can than store that info in mongo we just cycle through either the vagrant files or simply use virtualboxmanage directly. We get the state form mongo, but have a refersh that loads the state into mongo

@laszewsk
Copy link
Member

laszewsk commented Dec 7, 2018

lets revisit https://github.com/cloudmesh/vagrant in addition to what you did

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