Merge pull request '1.0.33' (#20) from testing into main

Reviewed-on: #20

.ansible-lint

@@ -4,5 +4,3 @@ skip_list:
   - yaml[line-length]
   - no-changed-when
   - galaxy[no-changelog]
-  - no-free-form
-  - package-latest

.drone.yml

@@ -0,0 +1,111 @@
+---
+kind: pipeline
+type: docker
+name: CentOS_9-Ansible-Pipeline
+
+trigger:
+  branch:
+    exclude:
+      - main
+  event: [push]
+
+steps:
+  - name: ansible_environment_verify
+    image: gitea.thezengarden.net/podman/ansible-dev/centos9:release
+    commands:
+      - . ./.drone.env
+      - env
+      - git log -1
+      - ansible --version
+      - ansible-lint --version
+
+  - name: ansible_lint
+    image: gitea.thezengarden.net/podman/ansible-dev/centos9:release
+    commands:
+      - . ./.drone.env
+      - ansible-lint --offline --nocolor $ANSIBLE_LINT_EXCLUSION
+
+
+---
+kind: pipeline
+type: docker
+name: Fedora_39-Ansible-Pipeline
+
+trigger:
+  branch:
+    exclude:
+      - main
+  event: [push]
+
+steps:
+  - name: ansible_environment_verify
+    image: gitea.thezengarden.net/podman/ansible-dev/fedora39:release
+    commands:
+      - . ./.drone.env
+      - env
+      - git log -1
+      - ansible --version
+      - ansible-lint --version
+
+  - name: ansible_lint
+    image: gitea.thezengarden.net/podman/ansible-dev/fedora39:release
+    commands:
+      - . ./.drone.env
+      - ansible-lint --offline --nocolor $ANSIBLE_LINT_EXCLUSION
+
+
+---
+kind: pipeline
+type: docker
+name: Debian_11-Ansible-Pipeline
+
+trigger:
+  branch:
+    exclude:
+      - main
+  event: [push]
+
+steps:
+  - name: ansible_environment_verify
+    image: gitea.thezengarden.net/podman/ansible-dev/debian11:release
+    commands:
+      - . ./.drone.env
+      - env
+      - git log -1
+      - ansible --version
+      - ansible-lint --version
+
+  - name: ansible_lint
+    image: gitea.thezengarden.net/podman/ansible-dev/debian11:release
+    commands:
+      - . ./.drone.env
+      - ansible-lint --offline --nocolor $ANSIBLE_LINT_EXCLUSION
+
+
+---
+kind: pipeline
+type: docker
+name: Debian_12-Ansible-Pipeline
+
+
+trigger:
+  branch:
+    exclude:
+      - main
+  event: [push]
+
+steps:
+  - name: ansible_environment_verify
+    image: gitea.thezengarden.net/podman/ansible-dev/debian12:release
+    commands:
+      - . ./.drone.env
+      - env
+      - git log -1
+      - ansible --version
+      - ansible-lint --version
+
+  - name: ansible_lint
+    image: gitea.thezengarden.net/podman/ansible-dev/debian12:release
+    commands:
+      - . ./.drone.env
+      - ansible-lint --offline --nocolor $ANSIBLE_LINT_EXCLUSION

.gitea/workflows/ansible-test.yml

@@ -7,7 +7,10 @@ on:
 
 jobs:
   Ansible-Development-Pipeline:
-    runs-on: ansible-dev-centos
+    strategy:
+      matrix:
+        os: [ ansible-dev-centos9, ansible-dev-fedora39, ansible-dev-debian11, ansible-dev-debian12 ]
+    runs-on: ${{ matrix.os }}
 
     steps:
       - name: Clone repository
@@ -15,18 +18,13 @@ jobs:
 
       - name: Ansible Environment Verify
         run: |
-          echo "BUILD HOST: $(cat /etc/hostname)" && echo
+          . ./.drone.env
-          echo "BUILD USER: $(whoami)" && echo
+          env
-          echo "PWD: $(pwd)" && echo
+          git log -1
-          lscpu && echo
+          ansible --version
-          . ./.ci.env && echo
+          ansible-lint --version
-          env && echo
-          git log -1 && echo
-          ansible --version && echo
-          ansible-lint --version --offline && echo
-
 
       - name: Run Ansible-Lint
         run: |
-          . ./.ci.env
+          . ./.drone.env
           ansible-lint --offline --nocolor $ANSIBLE_LINT_EXCLUSION

galaxy.yml

@@ -8,7 +8,7 @@ namespace: jchristianh
 name: baseos
 
 # The version of the collection. Must be compatible with semantic versioning
-version: 1.0.36
+version: 1.0.33
 
 # The path to the Markdown (.md) readme file. This path is relative to the root of the collection
 readme: README.md

meta/runtime.yml

@@ -1,2 +1,2 @@
 ---
-requires_ansible: ">=2.15.0"
+requires_ansible: ">=2.14.0"

roles/etc_hosts/tasks/main.yml

@@ -3,7 +3,7 @@
   ansible.builtin.set_fact:
     etc_hosts_etc_hosts_file: "{{ etc_hosts_etc_hosts_file }}.docker"
   when:
-    - ansible_virtualization_type is search("docker|podman|container")
+    - ansible_virtualization_type | regex_search("docker|podman|container")
 
 
 - name: Deploy {{ etc_hosts_etc_hosts_file }}

roles/hostname/tasks/main.yml

@@ -2,7 +2,7 @@
 
 - name: Use hostname module to set hostname, or failback to command module
   when:
-    - not ansible_virtualization_type is search("docker|podman|container")
+    - not ansible_virtualization_type | regex_search("docker|podman|container")
   block:
     - name: Set hostname for host using hostname module
       ansible.builtin.hostname:

roles/motd/defaults/main.yml

@@ -1,13 +1,13 @@
 ---
 motd_motd_file: /etc/motd
 
-motd_host_ip: "{{ ansible_default_ipv4.address | default('127.0.0.1') }}"
+motd_host_ip       : "{{ ansible_default_ipv4.address | default('127.0.0.1') }}"
-motd_host_uptime: "{{ now().replace(microsecond=0) - now().fromtimestamp(now(fmt='%s') | int - ansible_uptime_seconds) }}"
+motd_host_uptime   : "{{ now().replace(microsecond=0) - now().fromtimestamp(now(fmt='%s') | int - ansible_uptime_seconds) }}"
 
-motd_day_filter: "(.* day[s]?),\\s+(\\d+):(\\d+):(\\d+)"
+motd_day_filter    : "(.* day[s]?),\\s+(\\d+):(\\d+):(\\d+)"
-motd_nonday_filter: "(\\d+):(\\d+):(\\d+)"
+motd_nonday_filter : "(\\d+):(\\d+):(\\d+)"
 
-motd_day_filter_repl: "\\1 \\2 hours \\3 minutes"
+motd_day_filter_repl    : "\\1 \\2 hours \\3 minutes"
 motd_nonday_filter_repl : "\\1 hours \\2 minutes \\3 seconds"
 
 ...

roles/motd/tasks/main.yml

@@ -1,16 +1,20 @@
 ---
 - name: Format uptime using Jinja
   ansible.builtin.set_fact:
-    motd_uptime_formatted: |
+    uptime_formatted: |
       {% if 'day' in motd_host_uptime %}
         {{ motd_host_uptime | regex_replace(motd_day_filter, motd_day_filter_repl) }}
       {% else %}
         {{ motd_host_uptime | regex_replace(motd_nonday_filter, motd_nonday_filter_repl) }}
       {% endif %}
 
+
 - name: Update MOTD
   ansible.builtin.template:
-    src: templates/motd.j2
+    src        : templates/motd.j2
-    dest: "{{ motd_motd_file }}"
+    dest       : "{{ motd_motd_file }}"
-    mode: "0644"
+    mode       : "0644"
-  changed_when: false
+  changed_when : false
+
+
+...

roles/motd/templates/motd.j2

@@ -19,6 +19,6 @@ IP       : {{ motd_host_ip }}
 CPU      : {{ ansible_processor_vcpus }} x {{ ansible_processor[2] | regex_replace('\\s+', ' ') }}
 Memory   : {{ "{0:0.2f}".format(ansible_memfree_mb/1024) }} GB free of {{ (ansible_memtotal_mb/1024)|round}} GB
 Platform : {{ ansible_distribution }} {{ ansible_distribution_version }} {{ ansible_kernel }}
-Up       : {{ motd_uptime_formatted | trim }}
+Up       : {{ uptime_formatted | trim }}
 
 

roles/node_status/tasks/main.yml

@@ -1,31 +1,33 @@
 ---
 - name: Update node status when not running in a container
-  when: not ansible_virtualization_type is search("docker|podman|container")
+  when: not ansible_virtualization_type | regex_search("docker|podman|container")
   block:
     - name: Create directory {{ node_status_dir }}
       ansible.builtin.file:
-        path: "{{ node_status_dir }}"
+        path      : "{{ node_status_dir }}"
-        state: directory
+        state     : directory
-        mode: '0755'
+        mode      : '0755'
-      become: false
+      become      : false
-      run_once: true
+      run_once    : true
-      delegate_to: "{{ node_status_host }}"
+      delegate_to : "{{ node_status_host }}"
+
 
     - name: Capture system uptime
       ansible.builtin.command  : uptime
-      register: node_status_sys_uptime
+      register : r_node_status_uptime
       changed_when: false
 
+
     - name: Log node status
       ansible.builtin.lineinfile:
-        path: "{{ node_status_dir }}/{{ node_status_file }}"
+        path      : "{{ node_status_dir }}/{{ node_status_file }}"
-        state: present
+        state     : present
-        regexp: "{{ inventory_hostname }}"
+        regexp    : "{{ inventory_hostname }}"
-        line: "{{ ansible_date_time.epoch }},,{{ inventory_hostname }},,{{ ansible_distribution }} {{ ansible_distribution_version }},,{{ node_status_sys_uptime.stdout }}"
+        line      : "{{ ansible_date_time.epoch }},,{{ inventory_hostname }},,{{ ansible_distribution }} {{ ansible_distribution_version }},,{{ r_node_status_uptime.stdout }}"
-        create: true
+        create    : true
-        mode: '0644'
+        mode      : '0644'
-        owner: "{{ node_status_user }}"
+        owner     : "{{ node_status_user }}"
-        group: "{{ node_status_group }}"
+        group     : "{{ node_status_group }}"
-      delegate_to: "{{ node_status_host }}"
+      delegate_to : "{{ node_status_host }}"
-      throttle: 1
+      throttle    : 1
 

roles/resolv_conf/tasks/main.yml

@@ -3,7 +3,7 @@
   ansible.builtin.set_fact:
     resolv_conf_resolv_conf_file: "{{ resolv_conf_resolv_conf_file }}.container"
   when:
-    - ansible_virtualization_type is search("docker|podman|container")
+    - ansible_virtualization_type | regex_search("docker|podman|container")
 
 
 - name: Deploy {{ resolv_conf_resolv_conf_file }}

roles/snmpd_conf/handlers/main.yml

@@ -4,4 +4,4 @@
     name : snmpd
     state: restarted
   when:
-    - not ansible_virtualization_type is search("docker|podman|container")
+    - not ansible_virtualization_type | regex_search("docker|podman|container")

roles/snmpd_conf/tasks/main.yml

@@ -1,11 +1,12 @@
 ---
 - name: Deploy and configure SNMP
   block:
-    - name: Set package name for Debian
+    - name: Set package name for Debian 12
       ansible.builtin.set_fact:
         snmpd_conf_snmp_pkg: snmpd
       when:
         - ansible_distribution | lower == 'debian'
+        - ansible_distribution_major_version == '12'
 
     - name: Install SNMP
       ansible.builtin.package:

roles/update_all_packages/tasks/main.yml

@@ -1,21 +1,29 @@
 ---
 - name: Update package cache for Debian hosts
-  ansible.builtin.apt:
+  apt:
     update_cache: true
   when:
-    - "'apt' in ansible_pkg_mgr"
+    - ansible_distribution|lower == 'debian'
+  tags:
+    - skip_ansible_lint
 
 
 - name: Update package cache for RHEL/CentOS/Fedora hosts
-  ansible.builtin.dnf:
+  dnf:
     update_cache: true
   when:
-    - "'dnf' in ansible_pkg_mgr"
+    - (ansible_distribution|lower == 'centos' or
+      ansible_distribution|lower == 'fedora' or
+      ansible_distribution|lower == 'redhat')
     - ansible_distribution_major_version|int >= 8
+  tags:
+    - skip_ansible_lint
 
 
 - name: Update all host packages
-  ansible.builtin.package: name=* state=latest
+  package: name=* state=latest
+  tags:
+    - skip_ansible_lint
 
 
 ...