-
Notifications
You must be signed in to change notification settings - Fork 529
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
Building from dockerfiles / ghost processes #34
Comments
I'd say that the build from dockerfile functionality can be easily considered extremely useful. Consider the following scenario: |
Any new information on building a docker image for Dockerfile? |
+1 - need it too. |
+1 me, too. Current workaround is to build the dockerfile and image somewhere else and push it to the target server afterwards. |
+1 |
+1 |
+1 |
I've not had chance to look at this yet but if folks want to check out #131. I'm interested in feedback before accepting this. |
I've merged #131 which adds this functionality. The resource ordering issue has also recently been resolved. Thanks all. |
Hi,
Firstly, apologies if the following is obvious or if what I'm doing is just plain wrong. I'm quite to new to some of this stuff.
I am trying to, in one step: instantiate a vagrant box which, through puppet provisioning, builds several docker images from Dockerfiles and starts them.
I am experimenting with functionality for building images from Dockerfiles: jvaughan/garethr-docker@2e34bb2
However, it seems that docker::image is missing the requirement for docker::install, so was failing because of the lack of a docker executable to run.
I've tried adding
within the image class. This does satisfy the install dependency and means that the image gets built.
However, it seems to be triggering additional refreshes of the docker service, and this is causing my container to become ghosted in docker. The same ghosting occurs if I manually run
Do you have any suggestions of the best way to proceed?
BTW, if the 'build from dockerfile' functionality is considered useful, I will do a pull request once I have written some tests.
The text was updated successfully, but these errors were encountered: