
We need to use noqa for gather_facts_delegated_delegate_hosts because of the Python array syntax. Change-Id: Id2f298d22bc0f0031aded62bb1fb69dd08d64245
38 lines
1.2 KiB
Plaintext
38 lines
1.2 KiB
Plaintext
---
|
|
# NOTE(priteau): Rule file imported from kolla-ansible
|
|
strict: true
|
|
use_default_rules: true
|
|
skip_list:
|
|
# [E301] Commands should not change things if nothing needs doing
|
|
# TODO(mnasiadka): Fix tasks that fail this check in a later iteration
|
|
- no-changed-when
|
|
# [E503] Tasks that run when changed should likely be handlers
|
|
- no-handler
|
|
# [unnamed-task] All tasks should be named
|
|
# FIXME(mgoddard): Add names to all tasks
|
|
- unnamed-task
|
|
# Package installs should not use latest
|
|
- package-latest
|
|
# NOTE(frickler): Agreed at Zed PTG not to use FQCN for builtin actions for now, due to
|
|
# conflicts with open patches and backports.
|
|
- fqcn-builtins
|
|
# Allow Jinja templating inside task and play names
|
|
- name[template]
|
|
# FQCNs again, now for module actions
|
|
- fqcn[action]
|
|
# role name check matching ^*$
|
|
- role-name
|
|
# Allow long lines
|
|
- yaml[line-length]
|
|
# TODO(frickler): Discuss these in detail, skipping for now to unblock things
|
|
- command-instead-of-module
|
|
- command-instead-of-shell
|
|
- deprecated-local-action
|
|
- key-order[task]
|
|
- name[play]
|
|
- risky-file-permissions
|
|
- risky-shell-pipe
|
|
- run-once[task]
|
|
- var-naming[no-reserved]
|
|
- var-naming[no-role-prefix]
|