Added node.roles in readme and created deprecation warning

[Bernhard-Fluehmann]

Proposal to fix #731

[elasticmachine]

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?

[elasticmachine]

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?

[jmlrt]

Thanks for this PR.
This will need a bit more investigation in our side to ensure this is the way we want it to be implemented.

[jmlrt]

@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

	node.data: false
	node.master: true

The rest LGTM:+1:

[botelastic]

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.
To track this PR (even if closed), please open a corresponding issue if one does not already exist.

[botelastic]

This PR has been automatically closed because it has not had recent activity since being marked as stale.
Please reopen when work resumes.