Skip to content

Incorporated review comments #3841

Incorporated review comments

Incorporated review comments #3841

Triggered via push November 25, 2024 04:10
Status Success
Total duration 3m 54s
Artifacts 1

ansible-test.yml

on: push
Matrix: Build collection
Matrix: Ansible lint
sanity  /  Matrix Sanity
15s
sanity / Matrix Sanity
unit-galaxy  /  Matrix Unit
13s
unit-galaxy / Matrix Unit
Matrix: sanity / test
Matrix: unit-galaxy / test
all_green
0s
all_green
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
Build collection (stable-2.17)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-python@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Ansible lint (3.11, stable-2.17)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-python@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
jinja[spacing]: roles/idrac_reset/molecule/_get_or_update_settings.yml#L135
Jinja2 spacing could be improved: {{ oem['#DellManager.ResetToDefaults'] ['ResetType@Redfish.AllowableValues'] if '#DellManager.ResetToDefaults' in oem else [] }} -> {{ oem['#DellManager.ResetToDefaults']['ResetType@Redfish.AllowableValues'] if '#DellManager.ResetToDefaults' in oem else [] }}
args[module]: tests/integration/targets/prepare_idrac_lifecycle_controller_status/tasks/main.yaml#L12
missing required arguments: idrac_ip, idrac_password, idrac_user
args[module]: tests/integration/targets/prepare_lifecycle_contoller_job_queue/tasks/main.yml#L12
missing required arguments: idrac_ip, idrac_password, idrac_user
args[module]: tests/integration/targets/prepare_lifecycle_controller_job/tasks/main.yaml#L12
missing required arguments: idrac_ip, idrac_password, idrac_user
var-naming[no-role-prefix]: roles/idrac_attributes/defaults/main.yml#L3
Variables names from within roles should use idrac_attributes_ as a prefix. (vars: validate_certs)
var-naming[no-role-prefix]: roles/idrac_attributes/defaults/main.yml#L4
Variables names from within roles should use idrac_attributes_ as a prefix. (vars: https_timeout)
var-naming[no-role-prefix]: roles/idrac_attributes/defaults/main.yml#L5
Variables names from within roles should use idrac_attributes_ as a prefix. (vars: https_port)
var-naming[no-role-prefix]: roles/idrac_attributes/molecule/default/converge.yml#L10
Variables names from within roles should use idrac_attributes_ as a prefix. (vars: hostname)
var-naming[no-role-prefix]: roles/idrac_attributes/molecule/default/converge.yml#L10
Variables names from within roles should use idrac_attributes_ as a prefix. (vars: idrac_attributes)
var-naming[no-role-prefix]: roles/idrac_attributes/molecule/default/converge.yml#L10
Variables names from within roles should use idrac_attributes_ as a prefix. (vars: password)

Artifacts

Produced during runtime
Name Size
collection
1.11 MB