Fixing Ansible lint once and for all #14

Merged
otthorn merged 5 commits from fix_ansible_lint into master 2021-02-07 18:09:28 +01:00
Owner

Fixing Ansible and Yaml linting

  • removing the uneeded rules fromt the ansible lint config
  • enforcing them everywhere
  • making ansible-lint run on the whole config and not only the playbooks

We only keep [301] Commands should not change things if nothing needs doing in the skip list because it is an idempotency check. We don't always need idempotency, we something use ansible to update our infrastucture.
If we also want to get rid of 301 we would need some deeper change.

Fixing Ansible and Yaml linting - removing the uneeded rules fromt the ansible lint config - enforcing them everywhere - making ansible-lint run on the whole config and not only the playbooks We only keep `[301] Commands should not change things if nothing needs doing` in the skip list because it is an idempotency check. We don't always need idempotency, we something use ansible to update our infrastucture. If we also want to get rid of `301` we would need some deeper change.
otthorn added 5 commits 2021-02-07 17:48:02 +01:00
otthorn requested review from ynerant 2021-02-07 17:57:54 +01:00
ynerant refused to review 2021-02-07 18:02:03 +01:00
otthorn requested review from ynerant 2021-02-07 18:04:48 +01:00
ynerant approved these changes 2021-02-07 18:08:19 +01:00
ynerant left a comment
Contributor

Ça a l'air bien :)

Ça a l'air bien :)
otthorn merged commit c2d33d594b into master 2021-02-07 18:09:28 +01:00
otthorn deleted branch fix_ansible_lint 2021-02-07 18:09:36 +01:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: aurore/ansible#14
No description provided.