Added node.roles in readme and created deprecation warning #772
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? |
1 similar comment
|
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? |
|
Thanks for this PR. |
There was a problem hiding this comment.
@Bernhard-Fluehmann, thanks for this PR and sorry for the delay.
I wanted to investigate if it would worth to do it another way to be consistent with how we'll handle this change in Elasticsearch Helm chart, however I think this PR is fine for Ansible Elasticsearch.
Can you also update:
ansible-elasticsearch/docs/ssl-tls-setup.md
Lines 65 to 66 in f0e4e36
The rest LGTM:+1:
|
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. |
|
This PR has been automatically closed because it has not had recent activity since being marked as stale. |
Proposal to fix #731