Select mount module by managed node python version

Gate the mount tasks on ansible_facts.python_version instead of
ansible_version, rename the vendored module to mount_python_27, and gather
the python_version fact subset.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: 2 people

library/mount_ansible_29.py library/mount_python_27.pylibrary/mount_ansible_29.py renamed to library/mount_python_27.py

@@ -19,7 +19,7 @@
 
 DOCUMENTATION = r'''
 ---
-module: mount_ansible_29
+module: mount_python_27
 short_description: Control active and configured mount points
 description:
   - This module controls active and configured mount points in C(/etc/fstab).
@@ -149,57 +149,57 @@
 EXAMPLES = r'''
 # Before 2.3, option 'name' was used instead of 'path'
 - name: Mount DVD read-only
-  mount_ansible_29:
+  mount_python_27:
     path: /mnt/dvd
     src: /dev/sr0
     fstype: iso9660
     opts: ro,noauto
     state: present
 
 - name: Mount up device by label
-  mount_ansible_29:
+  mount_python_27:
     path: /srv/disk
     src: LABEL=SOME_LABEL
     fstype: ext4
     state: present
 
 - name: Mount up device by UUID
-  mount_ansible_29:
+  mount_python_27:
     path: /home
     src: UUID=b3e48f45-f933-4c8e-a700-22a159ec9077
     fstype: xfs
     opts: noatime
     state: present
 
 - name: Unmount a mounted volume
-  mount_ansible_29:
+  mount_python_27:
     path: /tmp/mnt-pnt
     state: unmounted
 
 - name: Remount a mounted volume
-  mount_ansible_29:
+  mount_python_27:
     path: /tmp/mnt-pnt
     state: remounted
 
 # The following will not save changes to fstab, and only be temporary until
 # a reboot, or until calling "state: unmounted" followed by "state: mounted"
 # on the same "path"
 - name: Remount a mounted volume and append exec to the existing options
-  mount_ansible_29:
+  mount_python_27:
     path: /tmp
     state: remounted
     opts: exec
 
 - name: Mount and bind a volume
-  mount_ansible_29:
+  mount_python_27:
     path: /system/new_volume/boot
     src: /boot
     opts: bind
     state: mounted
     fstype: none
 
 - name: Mount an NFS volume
-  mount_ansible_29:
+  mount_python_27:
     src: 192.168.1.100:/nfs/ssd/shared_data
     path: /mnt/shared_data
     opts: rw,sync,hard

tasks/main-blivet.yml

@@ -190,11 +190,11 @@
 #     update its view, then set up the new mounts. Otherwise,
 #     systemd will forcibly prevent mounting a new volume to an
 #     existing mount point.
-# ansible 2.9 (ansible-engine) uses the vendored mount_ansible_29 module,
-# since ansible.posix cannot be installed there; other systems use the bare
-# mount name, which redirects to the latest ansible.posix.mount.
-- name: Remove obsolete mounts - ansible 2.9
-  mount_ansible_29:
+# On managed nodes with python 2 (EL7 with ansible 2.9, where the
+# ansible.posix collection cannot be installed), use the dependency-free
+# vendored mount_python_27 module, which retains python 2 support.
+- name: Remove obsolete mounts - python 2
+  mount_python_27:
     src: "{{ mount_info['src'] | default(omit) }}"
     path: "{{ mount_info['path'] }}"
     fstype: "{{ mount_info['fstype'] | default(omit) }}"
@@ -204,12 +204,12 @@
             selectattr('state', 'match', '^absent$') | list }}"
   loop_control:
     loop_var: mount_info
-  when: ansible_version.full is version('2.10', '<')
+  when: ansible_facts['python_version'] is version('3', '<')
 
-# Bare module name (not FQCN): on ansible 2.9 the collection is not
-# installable and an unresolvable FQCN aborts play parsing even for a task
-# skipped by "when". The bare name resolves to the ansible 2.9 builtin
-# (skipped here) and redirects to the latest collection module elsewhere.
+# Bare module name (not FQCN): an unresolvable FQCN aborts play parsing on
+# ansible 2.9 even for a task skipped by "when", so the bare name is used.
+# This runs only on managed nodes with python 3, where the bare name resolves
+# to the latest ansible.posix.mount.
 - name: Remove obsolete mounts
   mount:  # noqa fqcn
     src: "{{ mount_info['src'] | default(omit) }}"
@@ -221,18 +221,18 @@
             selectattr('state', 'match', '^absent$') | list }}"
   loop_control:
     loop_var: mount_info
-  when: ansible_version.full is version('2.10', '>=')
+  when: ansible_facts['python_version'] is version('3', '>=')
 
 - name: Tell systemd to refresh its view of /etc/fstab
   systemd:
     daemon_reload: true
   when: blivet_output['mounts'] | length > 0
 
-# ansible 2.9 (ansible-engine) uses the vendored mount_ansible_29 module,
-# since ansible.posix cannot be installed there; other systems use the bare
-# mount name, which redirects to the latest ansible.posix.mount.
-- name: Set up new/current mounts - ansible 2.9
-  mount_ansible_29:
+# On managed nodes with python 2 (EL7 with ansible 2.9, where the
+# ansible.posix collection cannot be installed), use the dependency-free
+# vendored mount_python_27 module, which retains python 2 support.
+- name: Set up new/current mounts - python 2
+  mount_python_27:
     src: "{{ mount_info['src'] | default(omit) }}"
     path: "{{ mount_info['path'] }}"
     fstype: "{{ mount_info['fstype'] | default(omit) }}"
@@ -242,12 +242,12 @@
             rejectattr('state', 'match', '^absent$') | list }}"
   loop_control:
     loop_var: mount_info
-  when: ansible_version.full is version('2.10', '<')
+  when: ansible_facts['python_version'] is version('3', '<')
 
-# Bare module name (not FQCN): on ansible 2.9 the collection is not
-# installable and an unresolvable FQCN aborts play parsing even for a task
-# skipped by "when". The bare name resolves to the ansible 2.9 builtin
-# (skipped here) and redirects to the latest collection module elsewhere.
+# Bare module name (not FQCN): an unresolvable FQCN aborts play parsing on
+# ansible 2.9 even for a task skipped by "when", so the bare name is used.
+# This runs only on managed nodes with python 3, where the bare name resolves
+# to the latest ansible.posix.mount.
 - name: Set up new/current mounts
   mount:  # noqa fqcn
     src: "{{ mount_info['src'] | default(omit) }}"
@@ -259,7 +259,7 @@
             rejectattr('state', 'match', '^absent$') | list }}"
   loop_control:
     loop_var: mount_info
-  when: ansible_version.full is version('2.10', '>=')
+  when: ansible_facts['python_version'] is version('3', '>=')
 
 - name: Manage mount ownership/permissions
   file:

vars/main.yml

@@ -7,6 +7,8 @@ __storage_required_facts:
   - distribution_version
   - os_family
   - pkg_mgr
+  # provides ansible_facts.python_version, used to pick the mount module
+  - python_version
 
 # the subsets of ansible_facts that need to be gathered in case any of the
 # facts in required_facts is missing; see the documentation of