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

Run docker in vagrant #1943

Merged
merged 2 commits into from
Jan 27, 2016
Merged

Run docker in vagrant #1943

merged 2 commits into from
Jan 27, 2016

Conversation

wizardbyron
Copy link
Contributor

Just one Vagrantfile that can help you run docker in vagrant without errors.
❗ : You have to install vagrant and virtualbox first.

usage:

git clone git@github.com:wizardbyron/swagger-codegen.git
cd swagger-codegen
vagrant up

❗ You need to run docker by sudo
Then, you can run ./run-in-docker.sh in vagrant. And the directory /swagger-codegen was mapped as /vagrant

vagrant ssh
cd /vagrant
sudo ./run-in-docker.sh mvn package

@wing328
Copy link
Contributor

wing328 commented Jan 22, 2016

@wizardbyron thanks for the PR. Are you using the latest master when creating this PR?

Please kindly rebase to resolve the merge conflicts and let us know if you need help with the rebase.

@wizardbyron
Copy link
Contributor Author

I've resolved the conflicts.
Just had a glance at the latest .gitignore file, it's mess.
I'll open another issue and PR to reorganize the ignore entries.

@wing328 wing328 added this to the v2.1.6 milestone Jan 27, 2016
@wing328
Copy link
Contributor

wing328 commented Jan 27, 2016

@wizardbyron thanks for the PR. Testing looks good:

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] swagger-codegen-project ............................ SUCCESS [  2.167 s]
[INFO] swagger-codegen (core library) ..................... SUCCESS [01:05 min]
[INFO] swagger-codegen (executable) ....................... SUCCESS [01:07 min]
[INFO] swagger-codegen (maven-plugin) ..................... SUCCESS [02:22 min]
[INFO] swagger-generator .................................. SUCCESS [04:32 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 09:09 min
[INFO] Finished at: 2016-01-27T08:29:45+00:00
[INFO] Final Memory: 46M/311M
[INFO] ------------------------------------------------------------------------
vagrant@vagrant-ubuntu-trusty-64:/vagrant$ 

wing328 added a commit that referenced this pull request Jan 27, 2016
@wing328 wing328 merged commit a856e65 into swagger-api:master Jan 27, 2016
@wizardbyron
Copy link
Contributor Author

That's great! Do I need create a new PR to add instructions in README.md ?

@wizardbyron wizardbyron deleted the vagrant_new branch January 27, 2016 08:37
@wing328
Copy link
Contributor

wing328 commented Jan 27, 2016

I'm updating it :)

@wing328
Copy link
Contributor

wing328 commented Jan 27, 2016

@wizardbyron
Copy link
Contributor Author

Cool!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants