-
Notifications
You must be signed in to change notification settings - Fork 857
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
Hi @fabiogermann, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile? |
replaced iteritems() with items()
jenkins test this please |
jenkins test this please |
Hi @fabiogermann, While updating log4j template for Elasticsearch 7.X is really useful, I wonder if we still need to maintain a specific log4j template into this Ansible role. After a quick review of the template I don't see any dynamic values inside except conditionals related to Elasticsearch version. Maybe we should instead keep the |
related to #577 |
This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
I'm closing this PR as we handled it with #637 |
Introduce changes for the logging property file to be compatible with ES v7