diff --git a/.ansible-lint b/.ansible-lint new file mode 100644 index 00000000..bd650004 --- /dev/null +++ b/.ansible-lint @@ -0,0 +1,30 @@ +--- +# Copyright (c) Ansible Project +# GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) +# SPDX-License-Identifier: GPL-3.0-or-later + +skip_list: + # Ignore rules that make no sense: + - galaxy[tags] + - galaxy[version-incorrect] + - meta-runtime[unsupported-version] + - no-changed-when + - sanity[cannot-ignore] # some of the rules you cannot ignore actually MUST be ignored, like yamllint:unparsable-with-libyaml + - yaml # we're using yamllint ourselves + - run-once[task] # wtf??? + + # To be checked and maybe fixed: + - ignore-errors + - key-order[task] + - name[casing] + - name[missing] + - name[play] + - name[template] + - no-free-form + - no-handler + - risky-file-permissions + - risky-shell-pipe + - var-naming[no-reserved] + - var-naming[no-role-prefix] + - var-naming[pattern] + - var-naming[read-only] diff --git a/antsibull-nox.toml b/antsibull-nox.toml index d37d52b7..36fd080e 100644 --- a/antsibull-nox.toml +++ b/antsibull-nox.toml @@ -216,6 +216,12 @@ remote = [ ################################################################################################## +[sessions.ansible_lint] +ansible_lint_package = [ + "ansible-lint", + "ansible-compat < 25.8.2", +] + [[sessions.ee_check.execution_environments]] name = "devel-ubi-9" description = "ansible-core devel @ RHEL UBI 9" diff --git a/tests/integration/targets/connection/test_connection.yml b/tests/integration/targets/connection/test_connection.yml index 7f8b5697..b6327f8c 100644 --- a/tests/integration/targets/connection/test_connection.yml +++ b/tests/integration/targets/connection/test_connection.yml @@ -3,7 +3,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- hosts: "{{ target_hosts }}" +- hosts: "{{ target_hosts }}" # noqa: syntax-check[specific] gather_facts: false serial: 1 tasks: diff --git a/tests/integration/targets/connection_docker/setup.yml b/tests/integration/targets/connection_docker/setup.yml index e522a51f..e18c2554 100644 --- a/tests/integration/targets/connection_docker/setup.yml +++ b/tests/integration/targets/connection_docker/setup.yml @@ -11,4 +11,4 @@ tasks: - name: Setup docker import_role: - name: setup_docker + name: setup_docker # noqa: syntax-check[specific] diff --git a/tests/integration/targets/connection_docker_api/setup.yml b/tests/integration/targets/connection_docker_api/setup.yml index 0b073884..2de0d882 100644 --- a/tests/integration/targets/connection_docker_api/setup.yml +++ b/tests/integration/targets/connection_docker_api/setup.yml @@ -11,8 +11,8 @@ tasks: - name: Setup docker import_role: - name: setup_docker + name: setup_docker # noqa: syntax-check[specific] - name: Setup docker Python deps import_role: - name: setup_docker_python_deps + name: setup_docker_python_deps # noqa: syntax-check[specific] diff --git a/tests/integration/targets/connection_nsenter/setup.yml b/tests/integration/targets/connection_nsenter/setup.yml index e522a51f..e18c2554 100644 --- a/tests/integration/targets/connection_nsenter/setup.yml +++ b/tests/integration/targets/connection_nsenter/setup.yml @@ -11,4 +11,4 @@ tasks: - name: Setup docker import_role: - name: setup_docker + name: setup_docker # noqa: syntax-check[specific] diff --git a/tests/integration/targets/docker_config/tasks/test_docker_config.yml b/tests/integration/targets/docker_config/tasks/test_docker_config.yml index f67c0887..f22ed37f 100644 --- a/tests/integration/targets/docker_config/tasks/test_docker_config.yml +++ b/tests/integration/targets/docker_config/tasks/test_docker_config.yml @@ -20,7 +20,7 @@ advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" - name: Parameter name should be required - community.docker.docker_config: + community.docker.docker_config: # noqa: args[module] state: present ignore_errors: true register: output @@ -32,7 +32,7 @@ - 'output.msg == "missing required arguments: name"' - name: Test parameters - community.docker.docker_config: + community.docker.docker_config: # noqa: args[module] name: foo state: present ignore_errors: true @@ -261,7 +261,7 @@ community.docker.docker_config: name: db_password data: opensesame! - template_driver: "not a template driver" + template_driver: "not a template driver" # noqa: args[module] state: present ignore_errors: true register: output diff --git a/tests/integration/targets/docker_image_export/tasks/tests/basic.yml b/tests/integration/targets/docker_image_export/tasks/tests/basic.yml index 0f0e15ef..e81dd691 100644 --- a/tests/integration/targets/docker_image_export/tasks/tests/basic.yml +++ b/tests/integration/targets/docker_image_export/tasks/tests/basic.yml @@ -46,7 +46,7 @@ register: result - name: Extract manifest.json files - ansible.builtin.command: tar xvf "{{ remote_tmp_dir }}/{{ item.file }}" manifest.json --to-stdout + ansible.builtin.command: tar xvf "{{ remote_tmp_dir }}/{{ item.file }}" manifest.json --to-stdout # noqa: command-instead-of-module loop: "{{ image_tasks }}" loop_control: label: "{{ item.file }}" diff --git a/tests/integration/targets/docker_node/tasks/test_node.yml b/tests/integration/targets/docker_node/tasks/test_node.yml index e92b1c3e..dda9be34 100644 --- a/tests/integration/targets/docker_node/tasks/test_node.yml +++ b/tests/integration/targets/docker_node/tasks/test_node.yml @@ -23,7 +23,7 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" register: output - name: assert changed when create a new swarm cluster diff --git a/tests/integration/targets/docker_node_info/tasks/test_node_info.yml b/tests/integration/targets/docker_node_info/tasks/test_node_info.yml index 7798f646..8a6e5e55 100644 --- a/tests/integration/targets/docker_node_info/tasks/test_node_info.yml +++ b/tests/integration/targets/docker_node_info/tasks/test_node_info.yml @@ -23,7 +23,7 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" register: output - name: assert changed when create a new swarm cluster diff --git a/tests/integration/targets/docker_secret/tasks/test_secrets.yml b/tests/integration/targets/docker_secret/tasks/test_secrets.yml index 1107138e..6cf163c0 100644 --- a/tests/integration/targets/docker_secret/tasks/test_secrets.yml +++ b/tests/integration/targets/docker_secret/tasks/test_secrets.yml @@ -15,7 +15,7 @@ advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" - name: Parameter name should be required - community.docker.docker_secret: + community.docker.docker_secret: # noqa: args[module] state: present ignore_errors: true register: output @@ -27,7 +27,7 @@ - 'output.msg == "missing required arguments: name"' - name: Test parameters - community.docker.docker_secret: + community.docker.docker_secret: # noqa: args[module] name: foo state: present ignore_errors: true diff --git a/tests/integration/targets/docker_stack/tasks/test_stack.yml b/tests/integration/targets/docker_stack/tasks/test_stack.yml index d1938f11..7365bc4f 100644 --- a/tests/integration/targets/docker_stack/tasks/test_stack.yml +++ b/tests/integration/targets/docker_stack/tasks/test_stack.yml @@ -20,7 +20,7 @@ - name: Create a stack without name register: output - community.docker.docker_stack: + community.docker.docker_stack: # noqa: args[module] state: present ignore_errors: true @@ -51,8 +51,8 @@ - name: Template compose files ansible.builtin.template: - src: "{{item}}" - dest: "{{remote_tmp_dir}}/" + src: "{{ item }}" + dest: "{{ remote_tmp_dir }}/" with_items: - stack_compose_base.yml - stack_compose_overrides.yml @@ -63,7 +63,7 @@ state: present name: test_stack compose: - - "{{remote_tmp_dir}}/stack_compose_base.yml" + - "{{ remote_tmp_dir }}/stack_compose_base.yml" - name: assert test_stack changed on stack creation with compose file ansible.builtin.assert: @@ -77,8 +77,8 @@ # state: present # name: test_stack # compose: - # - "{{stack_compose_base}}" - # - "{{stack_compose_overrides}}" + # - "{{ stack_compose_base }}" + # - "{{ stack_compose_overrides }}" # # - name: assert test_stack correctly changed on update with yaml # assert: diff --git a/tests/integration/targets/docker_swarm/tasks/tests/basic.yml b/tests/integration/targets/docker_swarm/tasks/tests/basic.yml index ee0cc305..46e2a25f 100644 --- a/tests/integration/targets/docker_swarm/tasks/tests/basic.yml +++ b/tests/integration/targets/docker_swarm/tasks/tests/basic.yml @@ -10,7 +10,7 @@ ## Errors ########################################################## #################################################################### - name: Test parameters with state=join - community.docker.docker_swarm: + community.docker.docker_swarm: # noqa: args[module] state: join ignore_errors: true register: output @@ -22,7 +22,7 @@ - 'output.msg == "state is join but all of the following are missing: remote_addrs, join_token"' - name: Test parameters with state=remove - community.docker.docker_swarm: + community.docker.docker_swarm: # noqa: args[module] state: remove ignore_errors: true register: output @@ -40,7 +40,7 @@ - name: Create a Swarm cluster (check mode) community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" check_mode: true diff: true register: output_1 @@ -48,21 +48,21 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" diff: true register: output_2 - name: Create a Swarm cluster (idempotent) community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" diff: true register: output_3 - name: Create a Swarm cluster (idempotent, check mode) community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" check_mode: true diff: true register: output_4 @@ -70,7 +70,7 @@ - name: Create a Swarm cluster (force re-create) community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" force: true diff: true register: output_5 @@ -78,7 +78,7 @@ - name: Create a Swarm cluster (force re-create, check mode) community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" force: true check_mode: true diff: true diff --git a/tests/integration/targets/docker_swarm/tasks/tests/options-ca.yml b/tests/integration/targets/docker_swarm/tasks/tests/options-ca.yml index c90dff3f..c965162b 100644 --- a/tests/integration/targets/docker_swarm/tasks/tests/options-ca.yml +++ b/tests/integration/targets/docker_swarm/tasks/tests/options-ca.yml @@ -58,7 +58,7 @@ register: ansible_keys - name: signing_ca_cert and signing_ca_key (check mode) community.docker.docker_swarm: - advertise_addr: '{{ansible_default_ipv4.address | default("127.0.0.1")}}' + advertise_addr: '{{ ansible_default_ipv4.address | default("127.0.0.1") }}' state: present signing_ca_cert: '{{ ansible_certificates.results[0].content | b64decode }}' signing_ca_key: '{{ ansible_keys.results[0].content | b64decode }}' @@ -69,7 +69,7 @@ ignore_errors: true - name: signing_ca_cert and signing_ca_key community.docker.docker_swarm: - advertise_addr: '{{ansible_default_ipv4.address | default("127.0.0.1")}}' + advertise_addr: '{{ ansible_default_ipv4.address | default("127.0.0.1") }}' state: present signing_ca_cert: '{{ ansible_certificates.results[0].content | b64decode }}' signing_ca_key: '{{ ansible_keys.results[0].content | b64decode }}' diff --git a/tests/integration/targets/docker_swarm/tasks/tests/options.yml b/tests/integration/targets/docker_swarm/tasks/tests/options.yml index 71f9fda2..98fe693d 100644 --- a/tests/integration/targets/docker_swarm/tasks/tests/options.yml +++ b/tests/integration/targets/docker_swarm/tasks/tests/options.yml @@ -9,7 +9,7 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" name: default diff: true @@ -72,7 +72,7 @@ community.docker.docker_swarm: state: present force: true - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" autolock_managers: true diff: true register: output_7 diff --git a/tests/integration/targets/docker_swarm_info/tasks/test_swarm_info.yml b/tests/integration/targets/docker_swarm_info/tasks/test_swarm_info.yml index 910cebfc..c9bb82dd 100644 --- a/tests/integration/targets/docker_swarm_info/tasks/test_swarm_info.yml +++ b/tests/integration/targets/docker_swarm_info/tasks/test_swarm_info.yml @@ -27,7 +27,7 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" register: output - name: assert changed when create a new swarm cluster @@ -160,7 +160,7 @@ - name: Update swarm cluster to be locked community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" autolock_managers: true register: autolock_managers_update_output ignore_errors: true diff --git a/tests/integration/targets/docker_swarm_service/tasks/main.yml b/tests/integration/targets/docker_swarm_service/tasks/main.yml index cd5ef19a..ee966686 100644 --- a/tests/integration/targets/docker_swarm_service/tasks/main.yml +++ b/tests/integration/targets/docker_swarm_service/tasks/main.yml @@ -27,7 +27,7 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" - ansible.builtin.include_tasks: run-test.yml with_fileglob: diff --git a/tests/integration/targets/docker_swarm_service/tasks/tests/configs.yml b/tests/integration/targets/docker_swarm_service/tasks/tests/configs.yml index 4ebd6262..44834735 100644 --- a/tests/integration/targets/docker_swarm_service/tasks/tests/configs.yml +++ b/tests/integration/targets/docker_swarm_service/tasks/tests/configs.yml @@ -47,7 +47,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" filename: "/tmp/{{ config_name_1 }}.txt" register: configs_1 @@ -72,7 +72,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" filename: "/tmp/{{ config_name_1 }}.txt" - config_name: "{{ config_name_2 }}" @@ -89,7 +89,7 @@ configs: - config_name: "{{ config_name_1 }}" filename: "/tmp/{{ config_name_1 }}.txt" - - config_id: "{{ config_result_2.config_id|default('') }}" + - config_id: "{{ config_result_2.config_id | default('') }}" config_name: "{{ config_name_2 }}" filename: "/tmp/{{ config_name_2 }}.txt" register: configs_4 @@ -215,7 +215,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" uid: 1000 register: configs_1 @@ -228,7 +228,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" uid: 1000 register: configs_2 @@ -241,7 +241,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" uid: 1002 register: configs_3 @@ -254,7 +254,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" uid: "1001" register: configs_4 @@ -267,7 +267,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" uid: "1001" register: configs_5 @@ -305,7 +305,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" gid: 1000 register: configs_1 @@ -318,7 +318,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" gid: 1000 register: configs_2 @@ -331,7 +331,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" gid: 1002 register: configs_3 @@ -344,7 +344,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" gid: "1001" register: configs_4 @@ -357,7 +357,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" gid: "1001" register: configs_5 @@ -394,7 +394,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" mode: "0600" register: configs_1 @@ -407,7 +407,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" mode: "0600" register: configs_2 @@ -420,7 +420,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' configs: - - config_id: "{{ config_result_1.config_id|default('') }}" + - config_id: "{{ config_result_1.config_id | default('') }}" config_name: "{{ config_name_1 }}" mode: "0777" register: configs_3 diff --git a/tests/integration/targets/docker_swarm_service/tasks/tests/misc.yml b/tests/integration/targets/docker_swarm_service/tasks/tests/misc.yml index e71efbfd..548a1e95 100644 --- a/tests/integration/targets/docker_swarm_service/tasks/tests/misc.yml +++ b/tests/integration/targets/docker_swarm_service/tasks/tests/misc.yml @@ -96,7 +96,7 @@ - name: fake image key as it is not predictable ansible.builtin.set_fact: - ansible_docker_service_output: "{{ output.swarm_service|combine({'image': docker_test_image_busybox}) }}" + ansible_docker_service_output: "{{ output.swarm_service | combine({'image': docker_test_image_busybox}) }}" - name: assert service matches expectations ansible.builtin.assert: diff --git a/tests/integration/targets/docker_swarm_service/tasks/tests/secrets.yml b/tests/integration/targets/docker_swarm_service/tasks/tests/secrets.yml index 3cce577a..bd3aaa8e 100644 --- a/tests/integration/targets/docker_swarm_service/tasks/tests/secrets.yml +++ b/tests/integration/targets/docker_swarm_service/tasks/tests/secrets.yml @@ -47,7 +47,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" filename: "/run/secrets/{{ secret_name_1 }}.txt" register: secrets_1 @@ -72,7 +72,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" filename: "/run/secrets/{{ secret_name_1 }}.txt" - secret_name: "{{ secret_name_2 }}" @@ -89,7 +89,7 @@ secrets: - secret_name: "{{ secret_name_1 }}" filename: "/run/secrets/{{ secret_name_1 }}.txt" - - secret_id: "{{ secret_result_2.secret_id|default('') }}" + - secret_id: "{{ secret_result_2.secret_id | default('') }}" secret_name: "{{ secret_name_2 }}" filename: "/run/secrets/{{ secret_name_2 }}.txt" register: secrets_4 @@ -214,7 +214,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" uid: 1000 register: secrets_1 @@ -227,7 +227,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" uid: 1000 register: secrets_2 @@ -240,7 +240,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" uid: 1002 register: secrets_3 @@ -253,7 +253,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" uid: "1001" register: secrets_4 @@ -266,7 +266,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" uid: "1001" register: secrets_5 @@ -303,7 +303,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" gid: 1001 register: secrets_1 @@ -316,7 +316,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" gid: 1001 register: secrets_2 @@ -329,7 +329,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" gid: 1002 register: secrets_3 @@ -342,7 +342,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" gid: "1003" register: secrets_4 @@ -355,7 +355,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" gid: "1003" register: secrets_5 @@ -392,7 +392,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" mode: "0600" register: secrets_1 @@ -405,7 +405,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" mode: "0600" register: secrets_2 @@ -418,7 +418,7 @@ resolve_image: false command: '/bin/sh -v -c "sleep 10m"' secrets: - - secret_id: "{{ secret_result_1.secret_id|default('') }}" + - secret_id: "{{ secret_result_1.secret_id | default('') }}" secret_name: "{{ secret_name_1 }}" mode: "0777" register: secrets_3 diff --git a/tests/integration/targets/docker_swarm_service_info/tasks/test_docker_swarm_service_info.yml b/tests/integration/targets/docker_swarm_service_info/tasks/test_docker_swarm_service_info.yml index def419e2..6a69940d 100644 --- a/tests/integration/targets/docker_swarm_service_info/tasks/test_docker_swarm_service_info.yml +++ b/tests/integration/targets/docker_swarm_service_info/tasks/test_docker_swarm_service_info.yml @@ -32,7 +32,7 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" register: output - name: Create services diff --git a/tests/integration/targets/inventory_docker_containers/playbooks/docker_setup.yml b/tests/integration/targets/inventory_docker_containers/playbooks/docker_setup.yml index dab52c7a..cfe834fe 100644 --- a/tests/integration/targets/inventory_docker_containers/playbooks/docker_setup.yml +++ b/tests/integration/targets/inventory_docker_containers/playbooks/docker_setup.yml @@ -11,11 +11,11 @@ tasks: - name: Setup docker ansible.builtin.import_role: - name: setup_docker + name: setup_docker # noqa: syntax-check[specific] - name: Setup Docker Python deps ansible.builtin.import_role: - name: setup_docker_python_deps + name: setup_docker_python_deps # noqa: syntax-check[specific] - name: Start containers community.docker.docker_container: diff --git a/tests/integration/targets/inventory_docker_machine/playbooks/pre-setup.yml b/tests/integration/targets/inventory_docker_machine/playbooks/pre-setup.yml index f0e8f767..88ec9b91 100644 --- a/tests/integration/targets/inventory_docker_machine/playbooks/pre-setup.yml +++ b/tests/integration/targets/inventory_docker_machine/playbooks/pre-setup.yml @@ -8,11 +8,11 @@ tasks: - name: Setup docker ansible.builtin.include_role: - name: setup_docker + name: setup_docker # noqa: syntax-check[specific] - name: Setup Docker Python deps ansible.builtin.import_role: - name: setup_docker_python_deps + name: setup_docker_python_deps # noqa: syntax-check[specific] # There seems to be no better way to install docker-machine. At least I couldn't find any packages for RHEL7/8. - name: Download docker-machine binary diff --git a/tests/integration/targets/inventory_docker_machine/playbooks/test_inventory_1.yml b/tests/integration/targets/inventory_docker_machine/playbooks/test_inventory_1.yml index 3b326933..4de8c15e 100644 --- a/tests/integration/targets/inventory_docker_machine/playbooks/test_inventory_1.yml +++ b/tests/integration/targets/inventory_docker_machine/playbooks/test_inventory_1.yml @@ -8,7 +8,7 @@ tasks: - name: sanity check Docker Machine output vars: - dm_ls_format: !unsafe '{{.Name}} | {{.DriverName}} | {{.State}} | {{.URL}} | {{.Error}}' + dm_ls_format: !unsafe '{{.Name}} | {{.DriverName}} | {{.State}} | {{.URL}} | {{.Error}}' # noqa: jinja[invalid] success_regex: "^vm | [^|]+ | Running | tcp://.+ |$" ansible.builtin.command: docker-machine ls --format '{{ dm_ls_format }}' register: result diff --git a/tests/integration/targets/inventory_docker_swarm/playbooks/swarm_setup.yml b/tests/integration/targets/inventory_docker_swarm/playbooks/swarm_setup.yml index 24153b2f..be1216f4 100644 --- a/tests/integration/targets/inventory_docker_swarm/playbooks/swarm_setup.yml +++ b/tests/integration/targets/inventory_docker_swarm/playbooks/swarm_setup.yml @@ -11,7 +11,7 @@ tasks: - name: Setup docker ansible.builtin.import_role: - name: setup_docker + name: setup_docker # noqa: syntax-check[specific] - name: Setup Docker SDK for Python ansible.builtin.import_role: @@ -20,4 +20,4 @@ - name: Create a Swarm cluster community.docker.docker_swarm: state: present - advertise_addr: "{{ansible_default_ipv4.address | default('127.0.0.1')}}" + advertise_addr: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}" diff --git a/tests/integration/targets/setup_docker/tasks/Debian.yml b/tests/integration/targets/setup_docker/tasks/Debian.yml index b100a1f0..0f9bc958 100644 --- a/tests/integration/targets/setup_docker/tasks/Debian.yml +++ b/tests/integration/targets/setup_docker/tasks/Debian.yml @@ -4,7 +4,7 @@ # SPDX-License-Identifier: GPL-3.0-or-later - name: Get OS version - ansible.builtin.shell: uname -r + ansible.builtin.command: uname -r register: os_version - name: Install pre-reqs @@ -35,7 +35,7 @@ name: Add Docker repo on Ubuntu 20 or before, or Debian 11 or before block: - name: Add gpg key - ansible.builtin.shell: curl -fsSL https://download.docker.com/linux/{{ ansible_distribution | lower }}/gpg >key && apt-key add key + ansible.builtin.shell: curl -fsSL https://download.docker.com/linux/{{ ansible_distribution | lower }}/gpg >key && apt-key add key # noqa: command-instead-of-module - name: Add Docker repo ansible.builtin.apt_repository: diff --git a/tests/integration/targets/setup_docker/tasks/Fedora.yml b/tests/integration/targets/setup_docker/tasks/Fedora.yml index 21c8325a..738a9db1 100644 --- a/tests/integration/targets/setup_docker/tasks/Fedora.yml +++ b/tests/integration/targets/setup_docker/tasks/Fedora.yml @@ -13,7 +13,7 @@ file: docker-ce name: docker-ce-stable description: Docker CE Stable - $basearch - baseurl: https://download.docker.com/linux/fedora/{{ 31 if ansible_facts.distribution_major_version|int > 31 else '$releasever' }}/$basearch/stable + baseurl: https://download.docker.com/linux/fedora/{{ 31 if ansible_facts.distribution_major_version | int > 31 else '$releasever' }}/$basearch/stable enabled: true gpgcheck: true diff --git a/tests/integration/targets/setup_docker/tasks/RedHat-7.yml b/tests/integration/targets/setup_docker/tasks/RedHat-7.yml index 8b6ecdde..b4315b12 100644 --- a/tests/integration/targets/setup_docker/tasks/RedHat-7.yml +++ b/tests/integration/targets/setup_docker/tasks/RedHat-7.yml @@ -30,7 +30,7 @@ gpgkey: https://download.docker.com/linux/centos/gpg - name: Update cache - ansible.builtin.command: yum -y makecache fast + ansible.builtin.command: yum -y makecache fast # noqa: command-instead-of-module - name: Install docker ansible.builtin.dnf: diff --git a/tests/integration/targets/setup_docker/tasks/main.yml b/tests/integration/targets/setup_docker/tasks/main.yml index fe22759c..bef9f1a2 100644 --- a/tests/integration/targets/setup_docker/tasks/main.yml +++ b/tests/integration/targets/setup_docker/tasks/main.yml @@ -25,7 +25,7 @@ ansible.builtin.set_fact: needs_docker_daemon: '{{ not ansible_module_running_in_container }}' - - name: + - name: Print information ansible.builtin.debug: msg: |- OS family: {{ ansible_facts.os_family }} diff --git a/tests/integration/targets/setup_docker_cli_buildx/tasks/main.yml b/tests/integration/targets/setup_docker_cli_buildx/tasks/main.yml index 85512e44..4fb7eb7f 100644 --- a/tests/integration/targets/setup_docker_cli_buildx/tasks/main.yml +++ b/tests/integration/targets/setup_docker_cli_buildx/tasks/main.yml @@ -11,7 +11,7 @@ - name: Setup Docker when: ansible_facts.distribution ~ ansible_facts.distribution_major_version not in ['CentOS6', 'RedHat6'] block: - - name: + - name: Print information ansible.builtin.debug: msg: |- OS family: {{ ansible_facts.os_family }} diff --git a/tests/integration/targets/setup_docker_cli_compose/tasks/main.yml b/tests/integration/targets/setup_docker_cli_compose/tasks/main.yml index e6c05f7a..72ce2d43 100644 --- a/tests/integration/targets/setup_docker_cli_compose/tasks/main.yml +++ b/tests/integration/targets/setup_docker_cli_compose/tasks/main.yml @@ -11,7 +11,7 @@ - name: Setup Docker when: ansible_facts.distribution ~ ansible_facts.distribution_major_version not in ['CentOS6', 'RedHat6'] block: - - name: + - name: Print information ansible.builtin.debug: msg: |- OS family: {{ ansible_facts.os_family }} diff --git a/tests/integration/targets/setup_docker_python_deps/tasks/main.yml b/tests/integration/targets/setup_docker_python_deps/tasks/main.yml index ea007b04..9d69346e 100644 --- a/tests/integration/targets/setup_docker_python_deps/tasks/main.yml +++ b/tests/integration/targets/setup_docker_python_deps/tasks/main.yml @@ -28,5 +28,5 @@ - git+https://github.com/psf/requests - git+https://github.com/urllib3/urllib3 extra_args: "-c {{ remote_constraints }}" - state: latest + state: latest # noqa: package-latest when: force_docker_sdk_for_python_dev | default(false) diff --git a/tests/integration/targets/setup_pkg_mgr/tasks/main.yml b/tests/integration/targets/setup_pkg_mgr/tasks/main.yml index 45e90f92..a67e2f36 100644 --- a/tests/integration/targets/setup_pkg_mgr/tasks/main.yml +++ b/tests/integration/targets/setup_pkg_mgr/tasks/main.yml @@ -21,12 +21,14 @@ when: ansible_os_family == "Suse" - ansible.builtin.shell: + # noqa: command-instead-of-module cmd: | sed -i 's/mirrorlist/#mirrorlist/g' /etc/yum.repos.d/*.repo sed -i 's%#baseurl=http://mirror.centos.org/%baseurl=https://vault.centos.org/%g' /etc/yum.repos.d/*.repo when: ansible_distribution in 'CentOS' and ansible_distribution_major_version == '7' - ansible.builtin.shell: + # noqa: command-instead-of-module cmd: | sed -i 's/mirrorlist/#mirrorlist/g' /etc/yum.repos.d/CentOS-Linux-*.repo sed -i 's%#baseurl=http://mirror.centos.org/$contentdir/$releasever/%baseurl=https://vault.centos.org/8.4.2105/%g' /etc/yum.repos.d/CentOS-Linux-*.repo diff --git a/tests/integration/targets/setup_podman/tasks/main.yml b/tests/integration/targets/setup_podman/tasks/main.yml index 421ff69f..08913340 100644 --- a/tests/integration/targets/setup_podman/tasks/main.yml +++ b/tests/integration/targets/setup_podman/tasks/main.yml @@ -25,7 +25,7 @@ - when: not podman_setup_marker.stat.exists and not ansible_module_running_in_container block: - - name: + - name: Print information ansible.builtin.debug: msg: |- OS family: {{ ansible_facts.os_family }} diff --git a/tests/integration/targets/setup_remote_tmp_dir/tasks/main.yml b/tests/integration/targets/setup_remote_tmp_dir/tasks/main.yml index 483d034b..51e4b800 100644 --- a/tests/integration/targets/setup_remote_tmp_dir/tasks/main.yml +++ b/tests/integration/targets/setup_remote_tmp_dir/tasks/main.yml @@ -13,7 +13,7 @@ gather_subset: distribution when: ansible_facts == {} -- ansible.builtin.include_tasks: "{{ lookup('first_found', files)}}" +- ansible.builtin.include_tasks: "{{ lookup('first_found', files) }}" vars: files: - "{{ ansible_os_family | lower }}.yml"