fix(update): fix boolean list evaluation in display summary conditional

- Change 'dnf_update_result.results' check to 'dnf_update_result.results | length > 0'
  to prevent Jinja boolean evaluation failure on lists in Ansible 2.15+.

Co-developed-by: Gemini AI <renich+gemini@woralelandia.com>
Signed-off-by: Rénich Bon Ćirić <renich@woralelandia.com>

base.centos.yaml

@@ -1,8 +1,9 @@
 ---
 - name: Base server configuration and tools
   hosts:
-    - web
     - git
+    - id
+    - web
 
   vars_files:
     - vars/centos/dnf.yaml

inventories/production/all.yaml

@@ -1,13 +1,18 @@
 ---
-web:
-  hosts:
-    web0.mx-os.mx:
-
 git:
   hosts:
     web0.mx-os.mx:
 
+id:
+  hosts:
+    id0.mx-os.mx:
+
 runners:
   hosts:
     0.fedora.runners.mx-os.mx:
     1.fedora.runners.mx-os.mx:
+
+web:
+  hosts:
+    web0.mx-os.mx:
+

roles/common/defaults/main.yaml

@@ -1,3 +1,3 @@
-#SPDX-License-Identifier: MIT-0
+# SPDX-License-Identifier: MIT-0
 ---
 # defaults file for roles/common

roles/common/handlers/main.yaml

@@ -1,4 +1,4 @@
-#SPDX-License-Identifier: MIT-0
+# SPDX-License-Identifier: MIT-0
 ---
 # handlers file for roles/common
 

roles/common/meta/main.yaml

@@ -1,8 +1,8 @@
-#SPDX-License-Identifier: MIT-0
+# SPDX-License-Identifier: MIT-0
 galaxy_info:
-  author: your name
+  author: Rénich Bon Ćirić
-  description: your role description
+  description: Base server configuration and common utilities
-  company: your company (optional)
+  company: MxOS
 
   # If the issue tracker for your role is not on github, uncomment the
   # next line and provide a value
@@ -15,9 +15,9 @@ galaxy_info:
   # - GPL-3.0-only
   # - Apache-2.0
   # - CC-BY-4.0
-  license: license (GPL-2.0-or-later, MIT, etc)
+  license: MIT
 
-  min_ansible_version: 2.1
+  min_ansible_version: "2.1"
 
   # If this a Container Enabled role, provide the minimum Ansible Container version.
   # min_ansible_container_version:

roles/common/tasks/main.yaml

@@ -1,7 +1,7 @@
-#SPDX-License-Identifier: MIT-0
+# SPDX-License-Identifier: MIT-0
 ---
 - name: Configure dnf settings
-  ansible.builtin.ini_file:
+  community.general.ini_file:
     path: /etc/dnf/dnf.conf
     section: main
     option: "{{ item.option }}"
@@ -160,7 +160,7 @@
       set -g history-limit 20000
 
       # status color
-      set -g status-bg purple
+      set -g status-bg colour60
 
       # resize
       setw -g aggressive-resize on
@@ -176,4 +176,3 @@
     owner: root
     group: root
     mode: '0644'
-

roles/common/vars/main.yaml

@@ -1,3 +1,3 @@
-#SPDX-License-Identifier: MIT-0
+# SPDX-License-Identifier: MIT-0
 ---
 # vars file for roles/common

roles/users/meta/main.yaml

@@ -1,7 +1,7 @@
 galaxy_info:
-  author: your name
+  author: Rénich Bon Ćirić
-  description: your role description
+  description: User and group management with SSH keys
-  company: your company (optional)
+  company: MxOS
 
   # If the issue tracker for your role is not on github, uncomment the
   # next line and provide a value
@@ -14,9 +14,9 @@ galaxy_info:
   # - GPL-3.0-only
   # - Apache-2.0
   # - CC-BY-4.0
-  license: license (GPL-2.0-or-later, MIT, etc)
+  license: MIT
 
-  min_ansible_version: 2.1
+  min_ansible_version: "2.1"
 
   # If this a Container Enabled role, provide the minimum Ansible Container version.
   # min_ansible_container_version:

roles/users/tasks/main.yaml

@@ -24,11 +24,11 @@
 - name: Set infinite history
   ansible.builtin.copy:
     content: |
-      export HISTSIZE=-1
-      export HISTFILESIZE=-1
       export HISTCONTROL=ignoredups
+      export HISTFILESIZE=-1
+      export HISTSIZE=-1
+      export HISTTIMEFORMAT="%F %T "
     dest: /etc/profile.d/history.sh
     owner: root
     group: root
     mode: '0644'
-

update.yaml

@@ -20,7 +20,10 @@
     - name: Display summary of updated packages
       ansible.builtin.debug:
         msg: "System packages updated. {{ dnf_update_result.results | length }} package(s) affected."
-      when: dnf_update_result.changed and dnf_update_result.results is defined and dnf_update_result.results
+      when:
+        - dnf_update_result.changed
+        - dnf_update_result.results is defined
+        - dnf_update_result.results | length > 0
 
     - name: Remove unused dependencies (autoremove)
       ansible.builtin.dnf:
@@ -45,8 +48,8 @@
       ansible.builtin.reboot:
         msg: "Rebooting server to apply system updates."
         connect_timeout: 5
-        reboot_timeout: 30
+        reboot_timeout: 300
-        pre_reboot_delay: 0
+        pre_reboot_delay: 5
         post_reboot_delay: 30
         test_command: uptime
       when: reboot_is_needed and auto_reboot | bool

vars/centos/packages.yaml

@@ -12,6 +12,7 @@ packages:
   - htop
   - rpmconf
   - rsync
+  - systemd-container
   - tmux
   - tmux-powerline
   - vim
@@ -19,6 +20,8 @@ packages:
   - wireshark-cli
 
 unwanted_packages:
+  - cloud-init
+  - cloud-utils-growpart
   - cockpit
   - cockpit-bridge
   - cockpit-packagekit
@@ -26,4 +29,3 @@ unwanted_packages:
   - cockpit-ws
   - sos
 
-

vars/fedora/packages.yaml

@@ -10,6 +10,7 @@ packages:
   - htop
   - rpmconf
   - rsync
+  - systemd-container
   - tmux
   - tmux-powerline
   - vim