Apply suggestions from code review

Co-authored-by: Matt Crees <[email protected]>

Author: Alex-Welsh

.ansible-lint-ignore

@@ -1,4 +1,4 @@
-# This file contains ignores rule violations for ansible-lint
+# This file contains ignores to rule violations for ansible-lint
 etc/kayobe/ansible/vault-deploy-barbican.yml fqcn[action-core]
 etc/kayobe/ansible/vault-generate-backend-tls.yml fqcn[action-core]
 etc/kayobe/ansible/vault-generate-internal-tls.yml fqcn[action-core]

etc/kayobe/ansible/cephadm-commands-post.yml

@@ -7,7 +7,7 @@
     - cephadm
     - cephadm-commands
   tasks:
-    - name: Apply Cephadm role
+    - name: Apply Cephadm commands role
       ansible.builtin.import_role:
         name: stackhpc.cephadm.commands
       vars:

etc/kayobe/ansible/cephadm-commands-pre.yml

@@ -7,7 +7,7 @@
     - cephadm
     - cephadm-commands
   tasks:
-    - name: Apply Cephadm role
+    - name: Apply Cephadm commands role
       ansible.builtin.import_role:
         name: stackhpc.cephadm.commands
       vars:

etc/kayobe/ansible/cephadm.yml

@@ -6,7 +6,7 @@
 - name: Import Cephadm commands pre playbook
   import_playbook: cephadm-commands-pre.yml
 
-- name: Import Cephadm ec profiles playbook
+- name: Import Cephadm EC profiles playbook
   import_playbook: cephadm-ec-profiles.yml
 
 - name: Import Cephadm crush rules playbook

etc/kayobe/ansible/fix-houston.yml

@@ -20,6 +20,7 @@
   tasks:
     - name: Include kolla-ansible host vars
       ansible.builtin.include_vars: "{{ kolla_config_path }}/inventory/overcloud/host_vars/{{ inventory_hostname }}"
+
     - name: Create systemd service for -ovs network interface
       ansible.builtin.template:
         src: fix-houston-interface.service.j2

etc/kayobe/ansible/pulp-repo-promote-production.yml

@@ -3,7 +3,7 @@
   hosts: localhost
   gather_facts: true
   tasks:
-    - name: Ensure repository publications exist
+    - name: Ensure repository distributions exist
       ansible.builtin.import_role:
         name: stackhpc.pulp.pulp_distribution
       vars:

etc/kayobe/ansible/rabbitmq-reset.yml

@@ -31,7 +31,7 @@
       register: inspection
 
     - name: Ensure the {{ container_name }} container is running
-      ansible.builtin.command: systemctl start kolla-{{ container_name }}-container.service
+      ansible.builtin.command: systemctl start kolla-{{ container_name }}-container.service  # noqa command-instead-of-module
       when: inspection.stdout == 'false'
 
     - name: Wait for the {{ container_name }} container to reach state 'Running'

etc/kayobe/ansible/vault-deploy-overcloud.yml

@@ -83,7 +83,7 @@
         file: "{{ kayobe_env_config_path }}/vault/overcloud-vault-keys.json"
         name: vault_keys
 
-    - name: Import vault unseal role
+    - name: Apply vault unseal role
       ansible.builtin.import_role:
         name: stackhpc.hashicorp.vault_unseal
       vars:
@@ -99,7 +99,7 @@
   gather_facts: true
   hosts: controllers[0]
   tasks:
-    - name: Import vault pki role
+    - name: Apply vault pki role
       ansible.builtin.import_role:
         name: stackhpc.hashicorp.vault_pki
       vars:

etc/kayobe/ansible/wazuh-manager.yml

@@ -34,7 +34,8 @@
           ansible.builtin.assert:
             that: local_custom_certs_path is not defined
             fail_msg: The variable, `local_custom_certs_path`, is no longer used. Please remove this variable.
-- name: Apply wazuh indexer role
+
+- name: Apply Wazuh indexer role
   hosts: localhost
   roles:
     - role: wazuh-ansible/wazuh-ansible/roles/wazuh/wazuh-indexer