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 module #159

Closed
BrianHicks opened this issue Aug 16, 2016 · 6 comments
Closed

Docker module #159

BrianHicks opened this issue Aug 16, 2016 · 6 comments

Comments

@BrianHicks
Copy link
Contributor

BrianHicks commented Aug 16, 2016

Docker module should:

  • get image (version or "latest")

I'm not sure if it should ensure the container is running or not. Thoughts?

@ryane
Copy link
Contributor

ryane commented Aug 16, 2016

would it make sense to split into docker image and container resources? the container resource could be responsible for ensuring the container is running

@BrianHicks
Copy link
Contributor Author

Let’s go with docker.image and docker.container.

On 16 Aug 2016, at 16:58, Ryan Eschinger wrote:

would it make sense to split into docker image and container
resources? the container resource could be responsible for ensuring
the container is running

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#159 (comment)

@rebeccaskinner
Copy link
Contributor

Just contributing random ideas but maybe they could look something like...

docker.image 'rabbitmq' {
    image = "rabbitmq"
    tag = "ctl"
}

docker.container "rabbit instance" {
    image = "rabbitmq"
    tag = "ctl"
    name = "dockerinstance"
    env = [...]
    ...
    depends = ["docker.image.rabbitmq"]
}

@BrianHicks
Copy link
Contributor Author

that's more or less exactly what I was thinking, actually.

Probably start with docker.image if you take this ticket, @ryane. Much simpler from what I remember of that API. When you get that in we can talk about how it looks before you do the other one. Smaller PRs FTW.

@BrianHicks
Copy link
Contributor Author

Exactly. But the image module is the simpler of the two to implement first.

On Aug 16, 2016, at 4:58 PM, Ryan Eschinger notifications@github.com wrote:

would it make sense to split into docker image and container resources? the container resource could be responsible for ensuring the container is running


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@ryane ryane self-assigned this Aug 17, 2016
@ryane ryane added in progress and removed ready labels Aug 18, 2016
@stevendborrelli
Copy link
Member

Closing via #178

@stevendborrelli stevendborrelli added this to the 0.2.0 milestone Sep 7, 2016
@BrianHicks BrianHicks modified the milestones: 0.2.0, Strange Loop Demo (the real 0.2.0) Sep 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants