Fixing Ansible lint once and for all #14

Merged
otthorn merged 5 commits from fix_ansible_lint into master 3 years ago
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 3 years ago
otthorn requested review from ynerant 3 years ago
ynerant refused to review 3 years ago
otthorn requested review from ynerant 3 years ago
ynerant approved these changes 3 years ago
ynerant left a comment
Owner

Ça a l'air bien :)

Ça a l'air bien :)
otthorn merged commit c2d33d594b into master 3 years ago
otthorn deleted branch fix_ansible_lint 3 years ago

Reviewers

ynerant approved these changes 3 years ago
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
The pull request has been merged as c2d33d594b.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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
Loading…
There is no content yet.