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

add the mailcow__docker_compose_override_file role parameter #42

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

Conversation

hmb
Copy link

@hmb hmb commented Apr 1, 2023

For big mail databases we needed to move the mailstore onto some external volume, while keeping the other docker volumes on fast NVME. Here a docker-compose.override.yml file comes in handy, to specify that the vmail volume is an external one.

In order to deploy a (any) customized docker-compose.override.yml file, we added a parameter mailcow__docker_compose_override_file to specify an external file to the mailcow.mailcow role. If the parameter is set, the file will be copied as docker-compose.override.yml into the mailcow__install_path.

If the parameter is not given, the role behaves just as before. Of course the override file can be used to implement any desired customization. To use the parameter something like {{playbook_dir}} comes in handy, e.g.:

mailcow__docker_compose_override_file: "{{playbook_dir}}/files/my.docker-compose.override.yml"

We hope, that you will find this useful and will accept the pull request.

Add a parameter to the role, to enable adding a customized
docker-compose.override.yml file. This allows for enhancing docker compose
without having to change the stock docker-compose.yml.
@thwint
Copy link

thwint commented Dec 26, 2023

I like the idea, but I think a template is much more flexible than a static yaml file copied to the target system.

@hmb
Copy link
Author

hmb commented Jan 9, 2024

I agree, good idea - I'll change it to a template and test it again.

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

Successfully merging this pull request may close these issues.

2 participants