Skip to content
This repository was archived by the owner on May 5, 2022. It is now read-only.

Conversation

@MiniCodeMonkey
Copy link
Contributor

This adds docker support to machine, allowing you to easily run a local copy without having to worry about dependencies.

My use case is being able to test changes to conform files directly on my Mac before PR'ing them.

Let me know if you guys think it would be useful to a broader audience or if you have any questions/suggestions for this.

@migurski
Copy link
Member

Neat! Any way you might modify the Dockerfile slightly to use the chef scripts? This new version has some duplication in it that feels like it could fall out of synch, since Docker isn’t a tool I use with any regularity while working.

@MiniCodeMonkey
Copy link
Contributor Author

Hey,

I see the issue with duplication, unfortunately there's not really any good way to use Chef with Docker currently -- it would greatly complicate things and would be against best practices.

I would argue that having this duplication could be considered acceptable due to the above and due to the fact that it's very minimal.

What do you think?

Mathias

@migurski
Copy link
Member

Can you say more about the best practices?

@geobrando
Copy link
Contributor

@migurski
Copy link
Member

I think I might be asking the opposite question: not how to manage containers with Chef, but how to use existing Chef cookbooks within a container.

@geobrando
Copy link
Contributor

@migurski Ahh. Didn't really read the links, but thought they may have some useful info. Makes sense though

@davidchiles
Copy link

I'm a novice to docker and have no experience with chef... so ... I tried some things.

I tried using the existing chef script from the Dockerfile and it failed at a step where it was tring to setup some database.

After fumbling around I got this to work davidchiles@131a157

I'm able to run openaddr-process-one but not able to render a preview.

Just some breadcrumbs for anyone else trying this out.

@migurski migurski mentioned this pull request Feb 23, 2017
@migurski migurski mentioned this pull request Mar 24, 2017
5 tasks
@migurski
Copy link
Member

We have some more complete Docker work in #587, so I'm going to close this PR. @MiniCodeMonkey, thank you for your help! I know it’s taken a while, but we’re finally in a place where Docker is something we can support fully.

@migurski migurski closed this Apr 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants