Set scripting/coding/style standards about infrastructure HOT 10 OPEN

I guess this is a good place to bring this up, as it is not part of our current ansible structure.

How do we want to handle collections? Specifically levering Galaxy collections https://galaxy.ansible.com/ so we are duplicating efforts that are already well maintained?

Our ansible structure emulates loosely the Fedora and CentOS ansible structure - We will be using ansible collections that are pulled as defined by the requirements.yml. For example, we are already defining the freeipa collection.

from infrastructure.

As someone who helps to maintain some of the following, and writes a TON of Ansible code...

• ansible-lint is THE linter for Ansible code and should definitely be applied
• yamllint is THE linter for YAML in general and should also be applied
• Molecule is a tester for Ansible playbooks and roles and we should think about wrapping this around important bits
• ansible-test is the official test tool for Ansible modules and, likewise, should be wrapped around any custom modules we write
• tox-ansible is a plugin for tox that brings together running most of the above on Ansible code

Putting all of them into pre-commit is not too difficult, either.

from infrastructure.

What are the automatic tools that could help enforce this? For shell scripts, running at least ShellCheck is probably a good idea.

from infrastructure.

Agreed we need to get something formalised before we get too many contributions.

Highly recommend we get ansible-lint set up and make it a requirement on the Ansible side.

Generally pre-commit is not a bad idea as well but not sure how restrictive we want this to be.

from infrastructure.

+1 for ansible-lint

from infrastructure.

ansible-lint GH Action: https://github.com/ansible/ansible-lint-action (PR #24 merged)
shellcheck GH Action: https://github.com/ludeeus/action-shellcheck
yamllint GH Action: https://github.com/ibiqlik/action-yamllint (PR #13 merged)

from infrastructure.

I guess this is a good place to bring this up, as it is not part of our current ansible structure.

How do we want to handle collections? Specifically levering Galaxy collections https://galaxy.ansible.com/ so we are duplicating efforts that are already well maintained?

from infrastructure.

+1 for ShellCheck. Standardising the way things are developed is crucial.

from infrastructure.

I'd just like to point out that something like shellcheck and having a style guide are not necessarily the same thing - for example, shellcheck won't care if you comment your functions correctly, or whether you put do on the same line as a forloop or not, etc. but those sorts of things are still important for consistency in a large project - arguably, the correct placement of braces or whether then goes on the same line as if or the next is less important than tabs vs. spaces, but it's still important to define these things in the beginning rather than later when you realize it causes issues and then have to refactor old code.

from infrastructure.

+1 for molecule and testinfra

from infrastructure.