Skip to content
This repository has been archived by the owner on Mar 22, 2018. It is now read-only.

[WIP] Build images with ansible-container. #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] Build images with ansible-container. #68

wants to merge 1 commit into from

Conversation

mbarnes
Copy link
Contributor

@mbarnes mbarnes commented May 8, 2017

Add necessary YAML files to let ansible-container build service-specific container images. Promises to be easier than maintaining and building a suite of per-service Dockerfiles.

Related to projectatomic/commissaire#129

Add necessary YAML files to let ansible-container build service-specific
container images.  Promises to be easier than maintaining and building a
suite of per-service Dockerfiles.
@mbarnes
Copy link
Contributor Author

mbarnes commented May 9, 2017

I think this is getting close. I organized the Ansible roles similar to how the source code is organized: a base service role common to all services, and a dedicated role for each service (storage, investigator, clusterexec, etc.). That keeps the container.yml file fairly clean.

I was trying to fine-tune the dependencies for each service, and ended up hacking our own setup a bit because we only have one requirements.txt shared for all services. I didn't want, for example, Ansible being installed for the commissaire-storage-service image, which doesn't use Ansible. So for each image I override the requires.txt file in the package's "egg-info" directory with a unique file tailored to that service.

Next step is figuring out how to integrate config files into all this.

@mbarnes
Copy link
Contributor Author

mbarnes commented May 10, 2017

@mbarnes
Copy link
Contributor Author

mbarnes commented May 15, 2017

Related: projectatomic/commissaire#135

@ashcrow ashcrow mentioned this pull request Jun 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant