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 systemd RequiresMountsFor and unit custom template #36

Merged

Conversation

guidograzioli
Copy link
Member

@guidograzioli guidograzioli commented Jan 3, 2023

Fix #35

This change adds RequiresMountsFor= to the systemd unit when shared storage is used (activemq_shared_storage=True). A new variable activemq_shared_storage_mounted (default True), allows to avoid this behaviour (ie. the shared file system is local, instances are colocated).

As an alternative it is also now possible to override the systemd configuration with a custom template, to be passed-in via the new variable activemq_service_override_template (default: ''). The file pointed by the filename should be made available in the ansible playbook template lookup directories.

Variable Description Default
activemq_shared_storage_mounted Whether the systemd unit must require a mounted path (only when using shared storage) True
activemq_service_override_template Filename of custom systemd unit template to be deployed ``

@guidograzioli guidograzioli self-assigned this Jan 3, 2023
@guidograzioli guidograzioli added the bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements label Jan 3, 2023
@guidograzioli guidograzioli changed the title Add systemd RequireMountFor and unit custom template Add systemd RequiresMountsFor and unit custom template Jan 3, 2023
@guidograzioli guidograzioli merged commit 51d62e2 into ansible-middleware:main Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make amq-broker.service optionally dependent on NFS mount in unit file
2 participants