From 109a25c84523d208a15a101948040a557ff0aab2 Mon Sep 17 00:00:00 2001 From: Rich Megginson Date: Mon, 3 Jun 2024 10:58:56 -0600 Subject: [PATCH] fix ansible-lint octal mode issues --- tasks/shell_pcs/cluster-setup-corosync.yml | 2 +- tasks/shell_pcs/cluster-setup-keys.yml | 4 ++-- tasks/shell_pcs/configure-shell.yml | 6 +++--- tasks/shell_pcs/create-and-push-cib.yml | 4 ++-- tasks/shell_pcs/sbd.yml | 12 ++++++------ tests/tasks/fixture_psks.yml | 6 +++--- tests/tasks/qnetd_backup_restore.yml | 2 +- tests/tests_cluster_basic_existing_psks.yml | 12 ++++++------ tests/tests_cluster_basic_new_psks.yml | 8 ++++---- 9 files changed, 28 insertions(+), 28 deletions(-) diff --git a/tasks/shell_pcs/cluster-setup-corosync.yml b/tasks/shell_pcs/cluster-setup-corosync.yml index 8d0118d6..591b7abd 100644 --- a/tasks/shell_pcs/cluster-setup-corosync.yml +++ b/tasks/shell_pcs/cluster-setup-corosync.yml @@ -29,7 +29,7 @@ dest: /etc/corosync/corosync.conf owner: root group: root - mode: 0644 + mode: "0644" register: __ha_cluster_distribute_corosync_conf when: __ha_cluster_data_corosync_conf is defined diff --git a/tasks/shell_pcs/cluster-setup-keys.yml b/tasks/shell_pcs/cluster-setup-keys.yml index 0b86c3eb..297a2e56 100644 --- a/tasks/shell_pcs/cluster-setup-keys.yml +++ b/tasks/shell_pcs/cluster-setup-keys.yml @@ -14,7 +14,7 @@ dest: /etc/corosync/authkey owner: root group: root - mode: 0400 + mode: "0400" register: __ha_cluster_distribute_corosync_authkey - name: Get pacemaker authkey @@ -31,7 +31,7 @@ dest: /etc/pacemaker/authkey owner: hacluster group: haclient - mode: 0400 + mode: "0400" register: __ha_cluster_distribute_pacemaker_authkey - name: Remove qdevice certificates [CLI] diff --git a/tasks/shell_pcs/configure-shell.yml b/tasks/shell_pcs/configure-shell.yml index 1ba45883..a1c5523d 100644 --- a/tasks/shell_pcs/configure-shell.yml +++ b/tasks/shell_pcs/configure-shell.yml @@ -67,14 +67,14 @@ dest: /var/lib/pcsd/pcsd.key owner: root group: root - mode: 0600 + mode: "0600" - name: Distribute pcsd TLS certificate copy: src: "{{ ha_cluster_pcsd_public_key_src }}" dest: /var/lib/pcsd/pcsd.crt owner: root group: root - mode: 0600 + mode: "0600" - name: Distribute pcs_settings.conf template: @@ -82,7 +82,7 @@ dest: /var/lib/pcsd/pcs_settings.conf owner: root group: root - mode: 0644 + mode: "0644" trim_blocks: false - name: Start pcsd with updated config files and configure it to start on boot diff --git a/tasks/shell_pcs/create-and-push-cib.yml b/tasks/shell_pcs/create-and-push-cib.yml index 7db42ddb..4bd4bbf6 100644 --- a/tasks/shell_pcs/create-and-push-cib.yml +++ b/tasks/shell_pcs/create-and-push-cib.yml @@ -36,7 +36,7 @@ dest: "{{ item }}" owner: root group: root - mode: 0600 + mode: "0600" loop: - "{{ __ha_cluster_tempfile_cib_xml.path }}" - "{{ __ha_cluster_tempfile_original_cib_xml.path }}" @@ -276,7 +276,7 @@ dest: "{{ __ha_cluster_tempfile_cib_diff.path }}" owner: root group: root - mode: 0600 + mode: "0600" check_mode: false changed_when: not ansible_check_mode when: __ha_cluster_cib_diff.rc == 1 diff --git a/tasks/shell_pcs/sbd.yml b/tasks/shell_pcs/sbd.yml index 41bb95c9..611609b4 100644 --- a/tasks/shell_pcs/sbd.yml +++ b/tasks/shell_pcs/sbd.yml @@ -11,7 +11,7 @@ lineinfile: path: "/etc/modprobe.d/{{ item }}.conf" create: true - mode: 0644 + mode: "0644" # wokeignore:rule=blacklist regexp: "^(options|blacklist) {{ item }}" # wokeignore:rule=blacklist @@ -35,7 +35,7 @@ lineinfile: path: "/etc/modules-load.d/{{ item }}.conf" create: true - mode: 0644 + mode: "0644" regexp: "^{{ item }}" line: "{{ item }}" state: present @@ -95,7 +95,7 @@ dest: /etc/sysconfig/sbd owner: root group: root - mode: 0644 + mode: "0644" vars: options: "{{ ha_cluster_sbd_options | d([]) }}" node_name: "{{ __ha_cluster_node_name }}" @@ -119,7 +119,7 @@ state: directory owner: root group: root - mode: 0755 + mode: "0755" - name: Override start timeout for SBD template: @@ -127,7 +127,7 @@ dest: /etc/systemd/system/sbd.service.d/override-timeout.conf owner: root group: root - mode: 0644 + mode: "0644" vars: # Make sure the timeout is at least the default 90 seconds. # The intent is to make the timeout longer if needed, not shorter. @@ -169,7 +169,7 @@ state: file owner: hacluster group: haclient - mode: 0600 + mode: "0600" when: - not pacemaker_running diff --git a/tests/tasks/fixture_psks.yml b/tests/tasks/fixture_psks.yml index 422a9400..fde83eb0 100644 --- a/tests/tasks/fixture_psks.yml +++ b/tests/tasks/fixture_psks.yml @@ -35,16 +35,16 @@ copy: content: "{{ lookup('pipe', 'openssl rand -base64 256') | b64decode }}" dest: "{{ __test_corosync_key_path }}" - mode: 0400 + mode: "0400" - name: Generate pacemaker key copy: content: "{{ lookup('pipe', 'openssl rand -base64 256') | b64decode }}" dest: "{{ __test_pacemaker_key_path }}" - mode: 0400 + mode: "0400" - name: Generate fence_xvm key copy: content: "{{ lookup('pipe', 'openssl rand -base64 512') | b64decode }}" dest: "{{ __test_fence_xvm_key_path }}" - mode: 0400 + mode: "0400" diff --git a/tests/tasks/qnetd_backup_restore.yml b/tests/tasks/qnetd_backup_restore.yml index 4104805f..a5b8e1a6 100644 --- a/tests/tasks/qnetd_backup_restore.yml +++ b/tests/tasks/qnetd_backup_restore.yml @@ -7,7 +7,7 @@ file: path: /etc/corosync/qnetd_backup state: directory - mode: 0700 + mode: "0700" # Ansible copy doesn't support 'mode: preserve' with 'remote_src: true' - name: Back up qnetd settings diff --git a/tests/tests_cluster_basic_existing_psks.yml b/tests/tests_cluster_basic_existing_psks.yml index 0a7c61ed..f6aa5e64 100644 --- a/tests/tests_cluster_basic_existing_psks.yml +++ b/tests/tests_cluster_basic_existing_psks.yml @@ -39,7 +39,7 @@ state: directory owner: root group: root - mode: 0755 + mode: "0755" loop: - /etc/corosync - /etc/pacemaker @@ -52,7 +52,7 @@ dest: /etc/corosync/authkey owner: root group: root - mode: 0400 + mode: "0400" - name: Ensure pacemaker key is present copy: @@ -60,7 +60,7 @@ dest: /etc/pacemaker/authkey owner: root group: root - mode: 0400 + mode: "0400" - name: Ensure fence-virt key is present copy: @@ -68,7 +68,7 @@ dest: /etc/cluster/fence_xvm.key owner: root group: root - mode: 0400 + mode: "0400" when: __test_fence_virt_supported - name: Distribute pcsd TLS private key @@ -77,7 +77,7 @@ dest: /var/lib/pcsd/pcsd.key owner: root group: root - mode: 0600 + mode: "0600" - name: Distribute pcsd TLS certificate copy: @@ -85,7 +85,7 @@ dest: /var/lib/pcsd/pcsd.crt owner: root group: root - mode: 0600 + mode: "0600" - name: Get corosync key hash stat: diff --git a/tests/tests_cluster_basic_new_psks.yml b/tests/tests_cluster_basic_new_psks.yml index ec2c46e0..6f74b400 100644 --- a/tests/tests_cluster_basic_new_psks.yml +++ b/tests/tests_cluster_basic_new_psks.yml @@ -43,7 +43,7 @@ state: directory owner: root group: root - mode: 0755 + mode: "0755" loop: - /etc/pacemaker - /var/lib/pcsd @@ -54,7 +54,7 @@ dest: /etc/pacemaker/authkey owner: root group: root - mode: 0400 + mode: "0400" - name: Ensure fence-virt key is not present file: @@ -68,7 +68,7 @@ dest: /var/lib/pcsd/pcsd.key owner: root group: root - mode: 0600 + mode: "0600" - name: Distribute pcsd TLS certificate copy: @@ -76,7 +76,7 @@ dest: /var/lib/pcsd/pcsd.crt owner: root group: root - mode: 0600 + mode: "0600" - name: Get corosync key hash stat: