NETWAYS / ansible-collection-elasticstack

A collection to install and manage the Elastic Stack
GNU General Public License v3.0
9 stars 8 forks source link

Variable naming conventions #159

Closed widhalmt closed 1 year ago

widhalmt commented 1 year ago

The newest version of ansible-lint says, variables within roles should use the role name as a prefix.

For the most part we already stick to that rule. But we have a bunch of variables that are global to the whole collection. They are mostly used for interaction between the roles in the collection. These are prefixed with elastic_.

How should we deal with this new rule?

widhalmt commented 1 year ago

I opened a discussion upstream ( https://github.com/ansible/ansible-lint/discussions/3451 ) to get a viable solution. Thanks @dgoetz for suggesting.