Merge pull request '1.0.29' (#16) from testing into main

Reviewed-on: #16

galaxy.yml

@@ -8,7 +8,7 @@ namespace: jchristianh
 name: baseos
 
 # The version of the collection. Must be compatible with semantic versioning
-version: 1.0.22
+version: 1.0.29
 
 # 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/base_packages/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/bashrc/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/default_services/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/etc_hosts/meta/main.yml

@@ -6,7 +6,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/hostname/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/hostname/tasks/main.yml

@@ -1,6 +1,8 @@
 ---
 
 - name: Use hostname module to set hostname, or failback to command module
+  when:
+    - not ansible_virtualization_type | regex_search("docker|podman|container")
   block:
     - name: Set hostname for host using hostname module
       ansible.builtin.hostname:
@@ -9,32 +11,8 @@
         - "Restart_syslog_{{ ansible_distribution | lower }}_{{ ansible_distribution_major_version }}"
 
   rescue:
-    - name: Ensure hostname package is present
-      ansible.builtin.package:
-        name: hostname
-        state: present
-
-    - name: Set hostname file if running in container
-      ansible.builtin.set_fact:
-        hostname_hostname_file: "{{ hostname_hostname_file }}.container"
-      when:
-        - ansible_virtualization_type | regex_search("docker|podman|container")
-
-    - name: Add entry into hostname file
-      ansible.builtin.lineinfile:
-        path: "{{ hostname_hostname_file }}"
-        line: "{{ inventory_hostname }}"
-        state: present
-        create: true
-        mode: '0644'
-
-    - name: Update hostname from /etc/hostname
-      ansible.builtin.command:
-        cmd: "hostname -F {{ hostname_hostname_file }}"
-      notify:
-        - "Restart_syslog_{{ ansible_distribution | lower }}_{{ ansible_distribution_major_version }}"
-      when:
-        - not ansible_virtualization_type | regex_search("docker|podman|container")
-
+    - name: Unable to set hostname
+      ansible.builtin.debug:
+        msg: "Unable to set hostname for {{ ansible_hostname }} to {{ inventory_hostname }}."
 
 ...

roles/motd/defaults/main.yml

@@ -1,5 +1,13 @@
 ---
 motd_motd_file: /etc/motd
 
+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_day_filter    : "(.* day[s]?),\\s+(\\d+):(\\d+):(\\d+)"
+motd_nonday_filter : "(\\d+):(\\d+):(\\d+)"
+
+motd_day_filter_repl    : "\\1 \\2 hours \\3 minutes"
+motd_nonday_filter_repl : "\\1 hours \\2 minutes \\3 seconds"
 
 ...

roles/motd/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/motd/scripts/get_uptime.pl

@@ -1,21 +0,0 @@
-#!/usr/bin/perl
-
-use strict;
-
-my $seconds = shift;
-my $string  = sprintf "%02d::%02d::%02d::%02d", (gmtime($seconds))[7,2,1,0];
-
-# days [0] :: hours [1] :: mins [2] :: seconds [3]
-my @ups = split(/::/, $string);
-
-my $return_uptime = "";
-
-my $i = 0;
-for (@ups) { $ups[$i] =~ s/^0//g; $i++; }
-
-$return_uptime .= "$ups[0] days "    if $ups[0] != 0;
-$return_uptime .= "$ups[1] hours "   if $ups[1] != 0;
-$return_uptime .= "$ups[2] minutes " if $ups[2] != 0;
-# $return_uptime .= "$ups[3] seconds"  if $ups[3] != 0;
-
-print "$return_uptime\n";

roles/motd/tasks/main.yml

@@ -1,20 +1,16 @@
 ---
-- name: Set host IP address or set a default
+- name: Format uptime containing days
   ansible.builtin.set_fact:
-    host_ip : "{{ ansible_default_ipv4.address | default('127.0.0.1') }}"
+    uptime_formatted: "{{ motd_host_uptime | regex_replace(motd_day_filter, motd_day_filter_repl) }}"
+  when:
+    - "'day' in motd_host_uptime"
 
 
-- name: Install Perl if needed
-  ansible.builtin.package:
-    name  : perl
-    state : present
-
-
-- name: Get system uptime from script
-  ansible.builtin.script:
-    cmd        : "scripts/get_uptime.pl {{ ansible_uptime_seconds }}"
-  register     : node_uptime
-  changed_when : false
+- name: Format uptime not containing days
+  ansible.builtin.set_fact:
+    uptime_formatted: "{{ motd_host_uptime | regex_replace(motd_nonday_filter, motd_nonday_filter_repl) }}"
+  when:
+    - "'day' not in motd_host_uptime"
 
 
 - name: Update MOTD
@@ -26,4 +22,3 @@
 
 
 ...
-

roles/motd/templates/motd.j2

@@ -14,11 +14,11 @@
                   |___/
 
 
-Host     : {{ ansible_fqdn }}
-IP       : {{ host_ip }}
+Host     : {{ inventory_hostname }}
+IP       : {{ motd_host_ip }}
 CPU      : {{ ansible_processor_vcpus }} x {{ ansible_processor[2] }}
 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       : {{ node_uptime.stdout }}
+Up       : {{ uptime_formatted }}
 
 

roles/node_status/defaults/main.yml

@@ -4,4 +4,4 @@ node_status_user  : chris
 node_status_group : chris
 node_status_dir   : /home/chris/node_status
 node_status_file  : status.csv
-node_status_count : 3
+node_status_lc    : 3

roles/node_status/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/node_status/tasks/main.yml

@@ -29,4 +29,5 @@
         owner     : "{{ node_status_user }}"
         group     : "{{ node_status_group }}"
       delegate_to : "{{ node_status_host }}"
-      loop: "{{ range(0, node_status_count) | list }}"
+      throttle    : 1
+

roles/resolv_conf/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/snmpd_conf/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora

roles/update_all_packages/meta/main.yml

@@ -4,7 +4,7 @@ galaxy_info:
 
   license: GPL-2.0-or-later
 
-  min_ansible_version: "2.14"
+  min_ansible_version: "2.14.0"
 
   platforms:
     - name: Fedora