Add an ansible-lint action. #3
Triggered via pull request
September 25, 2023 13:56
Status
Failure
Total duration
4m 43s
Artifacts
–
ansible-lint.yml
on: pull_request
check-ansible-syntax
4m 33s
Annotations
10 errors and 11 warnings
yaml[line-length]:
ansible/configure-haproxy-local.yml#L224
Line too long (174 > 160 characters)
|
yaml[line-length]:
ansible/nomad-client.yml#L129
Line too long (170 > 160 characters)
|
jinja[invalid]:
ansible/roles/chromedriver/tasks/main.yml#L14
You need to install "jmespath" prior to running json_query filter
|
yaml[new-line-at-end-of-file]:
ansible/roles/google-chrome/defaults/main.yml#L2
No new line character at the end of file
|
fqcn[action-core]:
ansible/roles/google-chrome/tasks/main.yml#L2
Use FQCN for builtin module actions (file).
|
fqcn[action-core]:
ansible/roles/google-chrome/tasks/main.yml#L5
Use FQCN for builtin module actions (file).
|
fqcn[action-core]:
ansible/roles/google-chrome/tasks/main.yml#L8
Use FQCN for builtin module actions (apt_key).
|
fqcn[action-core]:
ansible/roles/google-chrome/tasks/main.yml#L11
Use FQCN for builtin module actions (apt_repository).
|
fqcn[action-core]:
ansible/roles/google-chrome/tasks/main.yml#L14
Use FQCN for builtin module actions (apt).
|
package-latest:
ansible/roles/google-chrome/tasks/main.yml#L14
Package installs should not use latest.
|
check-ansible-syntax
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
ignore-errors:
ansible/roles/consul-haproxy/handlers/main.yml#L2
Use failed_when and specify error conditions instead of using ignore_errors.
|
ignore-errors:
ansible/roles/consul-haproxy/tasks/main.yml#L10
Use failed_when and specify error conditions instead of using ignore_errors.
|
jinja[spacing]:
ansible/roles/consul-server/defaults/main.yml#L27
Jinja2 spacing could be improved: [{%- for wan_region in consul_wan_regions_oracle -%} "{{ consul_environment }}-{{ wan_region }}-consul-a.oracle.infra.jitsi.net", "{{ consul_environment }}-{{ wan_region }}-consul-b.oracle.infra.jitsi.net", "{{ consul_environment }}-{{ wan_region }}-consul-c.oracle.infra.jitsi.net", {%- endfor -%}] -> [{%- for wan_region in consul_wan_regions_oracle -%} "{{ consul_environment }}-{{ wan_region }}-consul-a.oracle.infra.jitsi.net", "{{ consul_environment }}-{{ wan_region }}-consul-b.oracle.infra.jitsi.net", "{{ consul_environment }}-{{ wan_region }}-consul-c.oracle.infra.jitsi.net",{%- endfor -%}]
|
name[casing]:
ansible/roles/google-chrome/tasks/main.yml#L2
All names should start with an uppercase letter.
|
no-free-form:
ansible/roles/google-chrome/tasks/main.yml#L2
Avoid using free-form when calling module actions. (file)
|
risky-file-permissions:
ansible/roles/google-chrome/tasks/main.yml#L2
File permissions unset or incorrect.
|
name[casing]:
ansible/roles/google-chrome/tasks/main.yml#L5
All names should start with an uppercase letter.
|
no-free-form:
ansible/roles/google-chrome/tasks/main.yml#L5
Avoid using free-form when calling module actions. (file)
|
risky-file-permissions:
ansible/roles/google-chrome/tasks/main.yml#L5
File permissions unset or incorrect.
|
no-free-form:
ansible/roles/google-chrome/tasks/main.yml#L8
Avoid using free-form when calling module actions. (apt_key)
|