Skip to content
This repository has been archived by the owner on Oct 10, 2021. It is now read-only.

Update install.yml #6

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

Conversation

schnitzlein
Copy link

@schnitzlein schnitzlein commented Nov 24, 2020

Adding chown -R for newly created user to install path

Tested on Ubuntu bionic with activemq 5.16.0

GitHub Issue: (link)

  • Other Relevant Links (Google Groups discussion, related pull requests,
    Release pull requests, etc.)

What does this Pull Request do?

This little peace of code added for me in Ubuntu 18.04 bionic beaver the chown -R for an activemq installation.
And it fixed a bug which occur if I try to run the service with the newly created user, because of missing permissions to run the activemq.

What's new?

  • activemq installpath/activemq/* permissions is set to user which runs activemq

How should this be tested?

With Ubuntu 18.04 and activemq 5.16.0 and just run the playbook.

  • the service was not be able to start the application because of mission permissions.

Additional Notes:

Thanks for this great ansible playbook! :)
I it is not overloaded and I can simply embedded in any other playbook through roles/activemq/ .
It stopped me from writing a lot of code.

Interested parties

Tag (@ mention) interested parties or, if unsure, @Islandora-Devops/committers

Thanks for this great work (Adding chown -R to install path)

Tested on Ubuntu bionic with activemq 5.16.0
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.

1 participant