search

robertdebock / ansible-role-tomcat

Install and configure tomcat on your system.
https://robertdebock.nl/
Apache License 2.0
46 stars 57 forks source link

Issue #25 - introduced config_files #26

Closed javid90khan closed 3 years ago

javid90khan commented 3 years ago

name: Pull request about: As requested on feature request #25. I have done an implementation.

Describe the change Added a new element: config_files for instances to transfer files to the [instance]/conf directory or it's subdirectory that the application can use. Use case: custom catalina.properties or a log4j config.

This can also replace any other server config if the consumer playbook needs to do that.

Testing I tested it against a CentOS 8 VM. I tried to run molecule but wasn't successful.

robertdebock commented 3 years ago

Looks good to me, thanks for the time you took to create (and fix) the code! I'll merge.