Ansible-lint: Difference between revisions
From DWIKI
mNo edit summary |
m (→APT) |
||
| (4 intermediate revisions by the same user not shown) | |||
| Line 1: | Line 1: | ||
=FAQ= | =FAQ= | ||
==Don't compare to empty string== | |||
* Use `when: var | length > 0` instead of `when: var != ""`. | |||
* Use `when: var | length == 0` instead of `when: var == ""`. | |||
==Don't compare to literal True/False== | ==Don't compare to literal True/False== | ||
| Line 6: | Line 9: | ||
use | use | ||
when: not (foo == 1 and bar == 0) | when: not (foo == 1 and bar == 0) | ||
==Use shell only when shell functionality is required== | |||
==Commands should not change things if nothing needs doing== | ==Commands should not change things if nothing needs doing== | ||
| Line 14: | Line 20: | ||
==Lines should be no longer than 160 chars== | ==Lines should be no longer than 160 chars== | ||
ignore for now? | ignore for now? | ||
==Shells that use pipes should set the pipefail option== | |||
shell: | | |||
set -o pipefail | |||
your command | |||
args: | |||
executable: /bin/bash | |||
==Ansible and APT== | |||
===Links=== | |||
*[https://docs.ansible.com/ansible/latest/collections/ansible/builtin/apt_module.html ansible.builtin.apt module – Manages apt-packages] | |||
===HOWTO=== | |||
====Override needrestart==== | |||
-name: install something | |||
ansible.builtin.apt: | |||
... | |||
environment: | |||
- NEEDRESTART_MODE: automatically | |||
===FAQ=== | |||
====Package installs should not use latest==== | |||
?? | |||
Latest revision as of 09:54, 18 November 2024
FAQ
Don't compare to empty string
- Use `when: var | length > 0` instead of `when: var != ""`.
- Use `when: var | length == 0` instead of `when: var == ""`.
Don't compare to literal True/False
Instead of
when: (foo == 1 and bar == 0) == false
use
when: not (foo == 1 and bar == 0)
Use shell only when shell functionality is required
Commands should not change things if nothing needs doing
Sometimes fixed by adding
changed_when: false
Lines should be no longer than 160 chars
ignore for now?
Shells that use pipes should set the pipefail option
shell: | set -o pipefail your command args: executable: /bin/bash
Ansible and APT
Links
HOWTO
Override needrestart
-name: install something ansible.builtin.apt: ...
environment: - NEEDRESTART_MODE: automatically
FAQ
Package installs should not use latest
??
