Skip to content

Commit d1d0f66

Browse files
EmilienMajaegerJames E. Blair
committed
Disable E106 & E208 for now
E106 rejects roles with '-' in their names, which is important for collections, but these aren't collections. Also ansible-lint throws an undocumented error "E208 File permissions not mentioned" that looks too aggressive, for example it complains IMHO wrongly about: roles/add-build-sshkey/tasks/remote-linux.yaml:1: [E208] File permissions not mentioned roles/add-build-sshkey/tasks/remote-linux.yaml:14: [E208] File permissions not mentioned roles/add-build-sshkey/tasks/remote-linux.yaml:20: [E208] File permissions not mentioned roles/add-build-sshkey/tasks/remote-linux.yaml:27: [E208] File permissions not mentioned Disable it for now until ansible-lint is fixed - and the real problems in zuul-jobs are fixed. ansible/ansible-lint#949 fixes the false positives. Once that is merged and a new release is out, we should revert and fix the real problems. Inspired by I3e3978bbc9458ad79623d22969344fb46f5fee43 and I48f837e8d5bd7d0d9d708d728d93627deb4093cc Change-Id: I0fdd4a79b886d1c4875e748b08f99da809d9d767 Co-Authored-By: Andreas Jaeger <[email protected]> Co-Authored-By: James E. Blair <[email protected]>
1 parent cad6162 commit d1d0f66

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

.ansible-lint

+5
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,12 @@ exclude_paths:
66
parseable: true
77
quiet: false
88
skip_list:
9+
- '106' # Role name does not match ``^[a-z][a-z0-9_]+$`` pattern
910
- '204' # [E204] Lines should be no longer than 120 chars
11+
# TODO(AJaeger): E208 gives false positives currently, we
12+
# should enable it once all issues in zuul-jobs and
13+
# ansible-lint are fixed.
14+
- '208' # File permissions not mentioned
1015
- '301' # [E301] Commands should not change things if nothing needs doing
1116
- '306' # [E306] Shells that use pipes should set the pipefail option
1217
verbosity: 1

0 commit comments

Comments
 (0)