forked from seapath/ci
-
Notifications
You must be signed in to change notification settings - Fork 0
/
ansible-lint.conf
44 lines (40 loc) · 1.78 KB
/
ansible-lint.conf
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
# Copyright (C) 2023, RTE (http://www.rte-france.com)
# SPDX-License-Identifier: Apache-2.0
#
# This file handle the configuration of ansible-lint
exclude_paths:
- ceph-ansible
- roles/systemd_networkd
- roles/corosync
# ceph-ansible, systemd_networkd and corosync are submodules and seapath don't
# have control of them.
skip_list:
- yaml # yaml syntax warnings
- unnamed-task # All tasks should be named
- role-name # All role names should match "^[a-z_][a-z0-9_]*$"
- risky-file-permissions # All file creation must specify permissions
- no-tabs # Most files should not contain tabs
- no-handler # "when: result.changed" should trigger a handler instead
- no-changed-when # Commands should not change things if nothing needs doing
- no-relative-paths # Doesn't need a relative path in role
### Why skip these warnings :
#
# - yaml errors are mostly "line >80 chars" and comment formatting. A yaml auto
# formatter could be run on the repository and would correct some problems.
# Reducing the length size requires a lot of reformatting work and isn't a big
# deal for now.
#
# - unnamed-task are all debugging task and thus not so important to name.
#
# - role-name doesn't seem too important to us.
#
# - risky-file-permissions concern a global cyber-security question : Specifying
# permissions on files must be done on the overall SEAPATH project in order to
# be effective. This is a much bigger task.
#
# - no-tabs is raised by a patch task. The concerned tool should soon merge the
# patch and this will not be required anymore.
#
# - no-handler, no-changed-when and no-relative-paths should not be skipped. The
# raised warnings should be corrected as soon as possible for these three rules to
# run on the CI.