.ansible-lint

@@ -1,7 +1,6 @@
----
 # .ansible-lint
 warn_list:
-  - '204'   # Lines should be less than 160 characters
+  - '204' # Lines should be less than 160 characters
-  - '701'   # meta/main.yml should contain relevant info
+  - '701' # meta/main.yml should contain relevant info
 skip_list:
-  - '106'   # Role name must match ^[a-z][a-z0-9_]+$ pattern
+  - '106' # Role name must match ^[a-z][a-z0-9_]+$ pattern

init-rocky-openqa-developer-host.yml

@@ -23,17 +23,16 @@
 
   # This is to try to avoid the handler issue in pre/post tasks
   handlers:
-    - name: Import handlers
+    - import_tasks: handlers/main.yml
-      ansible.builtin.import_tasks: handlers/main.yml
 
   pre_tasks:
     - name: Check if ansible cannot be run here
-      ansible.builtin.stat:
+      stat:
         path: /etc/no-ansible
       register: no_ansible
 
     - name: Verify if we can run ansible
-      ansible.builtin.assert:
+      assert:
         that:
           - "not no_ansible.stat.exists"
         success_msg: "We are able to run on this node"
@@ -41,11 +40,11 @@
 
   tasks:
     - name: Install and configure OpenQA
-      ansible.builtin.import_tasks: tasks/openqa.yml
+      import_tasks: tasks/openqa.yml
 
   post_tasks:
     - name: Touching run file that ansible has ran here
-      ansible.builtin.file:
+      file:
         path: /var/log/ansible.run
         state: touch
         mode: '0644'

init-rocky-openqa-worker-host.yml

@@ -12,17 +12,16 @@
 
   # This is to try to avoid the handler issue in pre/post tasks
   handlers:
-    - name: Import handlers
+    - import_tasks: handlers/main.yml
-      ansible.builtin.import_tasks: handlers/main.yml
 
   pre_tasks:
     - name: Check if ansible cannot be run here
-      ansible.builtin.stat:
+      stat:
         path: /etc/no-ansible
       register: no_ansible
 
     - name: Verify if we can run ansible
-      ansible.builtin.assert:
+      assert:
         that:
           - "not no_ansible.stat.exists"
         success_msg: "We are able to run on this node"
@@ -30,11 +29,11 @@
 
   tasks:
     - name: Install and configure OpenQA workers
-      ansible.builtin.import_tasks: tasks/openqa-worker.yml
+      import_tasks: tasks/openqa-worker.yml
 
   post_tasks:
     - name: Touching run file that ansible has ran here
-      ansible.builtin.file:
+      file:
         path: /var/log/ansible.run
         state: touch
         mode: '0644'

requirements.yml

@@ -1,3 +0,0 @@
----
-collections:
-  - ansible.posix

tasks/main.yml

@@ -0,0 +1,4 @@
+---
+# No tasks
+- debug: msg="No tasks are provided here. Please import the task as needed in your playbook."
+...

tasks/openqa-worker.yml

@@ -1,25 +1,25 @@
 ---
 - name: Install OpenQA worker packages
-  ansible.builtin.dnf:
+  dnf:
     name: "{{ openqa_worker_packages }}"
     state: present
   tags:
     - packages
 
 - name: Create openQA group
-  ansible.builtin.group:
+  group:
     name: "{{ openqa_group }}"
     system: true
 
 - name: Create openQA user
-  ansible.builtin.user:
+  user:
     name: "{{ openqa_user }}"
     groups: "{{ openqa_group }}"
     append: true
     system: true
 
 - name: Configure firewalld for openQA worker connections
-  ansible.builtin.template:
+  template:
     src: etc/firewalld/services/{{ item }}.xml.j2
     dest: /etc/firewalld/services/{{ item }}.xml
     owner: root
@@ -32,7 +32,7 @@
     - configure
 
 - name: Reload firewalld
-  ansible.builtin.systemd:
+  systemd:
     name: firewalld
     state: reloaded
   tags:
@@ -40,7 +40,7 @@
   ignore_errors: "{{ ansible_check_mode }}"
 
 - name: Write openQA configuration file
-  ansible.builtin.template:
+  template:
     src: etc/openqa/{{ item }}.j2
     dest: /etc/openqa/{{ item }}
     owner: "{{ openqa_user }}"
@@ -52,13 +52,13 @@
   tags:
     - configure
 
-- name: Start openQA workers
+- name: Start {{ openqa_worker_count }} openQA workers
   ansible.builtin.systemd:
     name: "openqa-worker@{{ item }}"
     state: started
     enabled: true
   # range "end" parameter is exclusive, so add 1
-  loop: "{{ range(1, (openqa_worker_count | int + 1)) | list }}"
+  loop: "{{ range(1, (openqa_worker_count|int + 1)) | list }}"
   tags:
     - start_workers
     - configure

tasks/openqa.yml

@@ -1,13 +1,13 @@
 ---
 - name: Install OpenQA packages
-  ansible.builtin.yum:
+  yum:
     name: "{{ openqa_packages }}"
     state: present
   tags:
     - packages
 
 - name: Copy httpd configuration files
-  ansible.builtin.copy:
+  copy:
     remote_src: true
     src: /etc/httpd/conf.d/{{ item }}.template
     dest: /etc/httpd/conf.d/{{ item }}
@@ -22,7 +22,7 @@
     - configure
 
 - name: Template OpenQA configuration files
-  ansible.builtin.template:
+  template:
     src: etc/openqa/{{ item }}.j2
     dest: /etc/openqa/{{ item }}
     owner: "{{ openqa_user }}"
@@ -35,21 +35,20 @@
     - configure
 
 - name: Get service facts
-  ansible.builtin.service_facts:
+  service_facts:
 
 - name: Check for non-empty postgres data directory
-  ansible.builtin.stat:
+  stat:
     path: /var/lib/pgsql/data/base
   register: postgres_data_dir
 
 - name: If postgresql is not already running, initialize database
-  ansible.builtin.command: postgresql-setup --initdb
+  command: postgresql-setup --initdb
   when: not ( ansible_facts.services["postgresql.service"]["state"] == "running" )
         and not postgres_data_dir.stat.exists
-  changed_when: true
 
 - name: Enable and start postgresql service
-  ansible.builtin.systemd:
+  systemd:
     name: postgresql
     state: started
     enabled: true
@@ -57,7 +56,7 @@
         and not postgres_data_dir.stat.exists
 
 - name: Configure SELinux to allow httpd connection to network
-  ansible.posix.seboolean:
+  seboolean:
     name: httpd_can_network_connect
     state: true
     persistent: true
@@ -65,7 +64,7 @@
     - configure
 
 - name: Enable and start OpenQA services
-  ansible.builtin.systemd:
+  systemd:
     name: "{{ item }}"
     state: started
     enabled: true
@@ -74,7 +73,7 @@
     - configure
 
 - name: Create openqa-vnc firewalld service
-  ansible.builtin.template:
+  template:
     src: etc/firewalld/services/openqa-vnc.xml.j2
     dest: /etc/firewalld/services/openqa-vnc.xml
     owner: root
@@ -84,13 +83,13 @@
     - configure
 
 - name: Load openqa-vnc firewalld service
-  ansible.builtin.systemd:
+  systemd:
     name: firewalld
     state: reloaded
   tags:
     - configure
 
-- name: Permit traffic for http and openqa-vnc services
+- name: Permit traffic for {{ item }} service
   ansible.posix.firewalld:
     service: "{{ item }}"
     permanent: true
@@ -102,21 +101,21 @@
     - configure
 
 - name: Reload FirewallD
-  ansible.builtin.systemd:
+  systemd:
     name: firewalld
     state: reloaded
   tags:
     - configure
 
 - name: Check for existing repository
-  ansible.builtin.stat:
+  stat:
     path: "{{ openqa_homedir }}/share/tests/rocky"
   register: rocky_testing_repo
   tags:
     - configure
 
 - name: Clone repository if it does not already exist
-  ansible.builtin.git:
+  git:
     accept_hostkey: true
     dest: "{{ openqa_homedir }}/share/tests/rocky"
     repo: "{{ openqa_rocky_testing_repo }}"
@@ -126,7 +125,7 @@
     - configure
 
 - name: Set owner/group/permissions on repo contents
-  ansible.builtin.file:
+  file:
     path: "{{ openqa_homedir }}/share/tests/rocky"
     recurse: true
     owner: "{{ openqa_user }}"
@@ -137,17 +136,17 @@
 
 # fifloader.py will fail if the Demo user is not logged in
 - name: Authenticate to web UI the first time
-  ansible.builtin.uri:
+  uri:
     url: "http://{{ openqa_host }}/login"
 
 - name: Run fifloader.py
-  ansible.builtin.command: ./fifloader.py -l -c templates.fif.json templates-updates.fif.json
+  command: ./fifloader.py -l -c templates.fif.json templates-updates.fif.json
   changed_when: "1 != 1"
   args:
     chdir: "{{ openqa_homedir }}/share/tests/rocky"
 
 - name: Create ISO directory
-  ansible.builtin.file:
+  file:
     path: "{{ openqa_homedir }}/share/factory/iso/fixed"
     state: directory
     owner: "{{ openqa_user }}"
@@ -157,7 +156,7 @@
     - download_isos
 
 - name: Download ISOs
-  ansible.builtin.get_url:
+  get_url:
     dest: "{{ openqa_homedir }}/share/factory/iso/fixed/{{ item.name }}"
     url: "{{ rocky_iso_download_url }}/{{ item.name }}"
     checksum: "{{ item.checksum }}"
@@ -169,19 +168,19 @@
   tags:
     - download_isos
 
-- name: Start OpenQA workers
+- name: Start {{ openqa_worker_count }} OpenQA workers
   ansible.builtin.systemd:
     name: "openqa-worker@{{ item }}"
     state: started
     enabled: true
   # range 'end' parameter is exclusive, so add 1
-  loop: "{{ range(1, (openqa_worker_count | int + 1)) | list }}"
+  loop: "{{ range(1, (openqa_worker_count|int + 1)) | list }}"
   tags:
     - start_workers
     - configure
 
 - name: POST a job
-  ansible.builtin.command: |
+  command: |
     openqa-cli api -X POST isos \
       ISO=Rocky-{{ rocky_version }}-{{ rocky_arch }}-minimal.iso \
       ARCH={{ rocky_arch }} \

tests/test.yml

@@ -1,9 +1,5 @@
 ---
-- name: Run tests
+- hosts: localhost
-  hosts: localhost
   remote_user: root
   tasks:
-    - name: Ensure required variables are defined
+    - import_tasks: example.yml
-      ansible.builtin.assert:
-        that:
-          - openqa_host is defined

vars/openqa-worker.yml

@@ -15,7 +15,7 @@ openqa_worker_count: 2
 # The max port should be 5990 + n where n is the total
 # number of workers you want to enable on your system.
 openqa_min_vnc_port: 5991
-openqa_max_vnc_port: "{{ 5990 + openqa_worker_count | int }}"
+openqa_max_vnc_port: "{{ 5990 + openqa_worker_count|int }}"
 
 # Port range to open for socket connections from the primary host.
 openqa_min_socket_port: 20000
@@ -23,13 +23,14 @@ openqa_max_socket_port: 20089
 
 # Packages to install
 openqa_worker_packages:
-  - firewalld
   - guestfs-tools
   - libguestfs-xfs
   - libvirt-daemon-config-network
+  - virt-install
   - openqa-worker
   - perl-REST-Client
   - python3-libguestfs
   - virt-install
   - withlock
+  - firewalld
 ...

vars/openqa.yml

@@ -45,7 +45,7 @@ openqa_worker_count: 1
 # The max port should be 5990 + n where n is the total
 # number of workers you want to enable on your system.
 openqa_min_vnc_port: 5991
-openqa_max_vnc_port: "{{ 5990 + openqa_worker_count | int }}"
+openqa_max_vnc_port: "{{ 5990 + openqa_worker_count|int }}"
 
 # Packages to install
 openqa_packages: