-
Notifications
You must be signed in to change notification settings - Fork 5
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
Replace single monitored log dir with separate server and web #1
Conversation
This is a breaking change
In general, makes sense. Are we currently not getting logs because of this? |
At the moment only the server logs are monitored https://github.com/IDR/deployment/blob/master/ansible/group_vars/omero-hosts.yml#L58 |
There are several TODOs, both due to ansible-lint and due to an unknown problem with the service stopping
I've updated travis. There's are several TODOs, but this is no worse than it was before. |
It might be the lack of a valid slack token that's causing a problem, I'll investigate |
@joshmoore I've taken the molecule test out of travis |
Once green, |
https://github.com/openmicroscopy/ansible-role-omero-server/tree/2.0.0-m1 and https://github.com/openmicroscopy/ansible-role-omero-web/tree/1.0.0-m3 change the location of OMERO.server and OMERO.web. This PR replaces the single
omero_logmonitor_logs_dir
parameter withomero_logmonitor_logs_dir_server
andomero_logmonitor_logs_dir_web
, and changes the defaults to match the default locations for OMERO.server and OMERO.web.This is a breaking change which mostly affects the default configuration. Advanced configuration will be mostly unaffected.