locale_gen: fix: use glibc_ubuntu mode only if files for that exists

changelogs/fragments/9735-locale_gen-use_glibc_ubuntu_mode_only_if_file_exists.yml

@@ -0,0 +1,2 @@
+bugfixes:
+  - "locale_gen - use ``glibc`` mode only if files for that exists. In other case use ``other_glibc`` mode (https://github.com/ansible-collections/community.general/pull/9735)."

plugins/modules/locale_gen.py

@@ -70,6 +70,7 @@
   choices:
     - glibc
     - ubuntu_legacy
+    - other_glibc
   returned: success
   sample: glibc
   version_added: 10.2.0
@@ -123,11 +124,18 @@ def __init_module__(self):
                 available=SUPPORTED_LOCALES,
                 apply_change=self.apply_change_glibc,
             ),
+            other_glibc=dict(
+                available=ETC_LOCALE_GEN,
+                apply_change=self.apply_change_glibc,
+            ),
         )
 
-        if os.path.exists(ETC_LOCALE_GEN):
+        if os.path.exists(ETC_LOCALE_GEN) and os.path.exists(SUPPORTED_LOCALES):
             self.vars.ubuntu_mode = False
             self.vars.mechanism = "glibc"
+        elif os.path.exists(ETC_LOCALE_GEN):
+            self.vars.ubuntu_mode = False
+            self.vars.mechanism = "other_glibc"
         elif os.path.exists(VAR_LIB_LOCALES):
             self.vars.ubuntu_mode = True
             self.vars.mechanism = "ubuntu_legacy"

tests/integration/targets/locale_gen/tasks/main.yml

@@ -10,10 +10,19 @@
 
 - name: Bail out if not supported
   ansible.builtin.meta: end_play
-  when: ansible_distribution not in ('Ubuntu', 'Debian')
+  when:
+    - "ansible_os_family not in ('Debian', 'Archlinux', 'RedHat')"
 
-- name: Run tests auto-detecting mechanism
+- name: Run tests auto-detecting mechanism [Debian]
   ansible.builtin.include_tasks: basic.yml
   loop: "{{ locale_list_basic }}"
   loop_control:
     loop_var: locale_basic
+  when:
+    - ansible_os_family == 'Debian'
+
+- name: Run tests auto-detecting mechanism [Archlinux, RedHat]
+  ansible.builtin.include_tasks:
+    file: "non_debian.yml"
+  when:
+    - "ansible_os_family in ('RedHat', 'Archlinux')"

tests/integration/targets/locale_gen/tasks/non_debian.yml

@@ -0,0 +1,44 @@
+---
+# Copyright (c) Ansible Project
+# GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt)
+# SPDX-License-Identifier: GPL-3.0-or-later
+
+- name: "Non dnf RedHat system are not supported"
+  ansible.builtin.meta: "end_play"
+  when:
+    - ansible_pkg_mgr == 'yum'
+
+- name: "Install locale packages for RedHat"
+  ansible.builtin.dnf:
+    name:
+      - "glibc-langpack-pl"
+      - "glibc-langpack-en"
+    state: "present"
+  when:
+    - "ansible_os_family == 'RedHat'"
+
+- name: "Prepare locale.gen file for RedHat"
+  ansible.builtin.template:
+    src: "locale_gen.j2"
+    dest: "/etc/locale.gen"
+    owner: "root"
+    group: "root"
+    mode: "0644"
+    force: true
+  when:
+    - "ansible_os_family == 'RedHat'"
+
+- name: "Check for locale that pack are not installed"
+  locale_gen:
+    name: "ansible.UTF-8"
+    state: "present"
+  register: "locale_gen_locale_missed_result"
+  ignore_errors: true
+
+- name: "Deploy locales"
+  locale_gen:
+    name: "{{ item['locale'] + '.' + item['charset'] }}"
+    state: "present"
+  loop: "{{ vars['locale_list_simple'] | flatten(levels=1) }}"
+  loop_control:
+    label: "{{ 'Deploy locale: ' + item['locale'] }}"

tests/integration/targets/locale_gen/templates/locale_gen.j2

@@ -0,0 +1,9 @@
+{#
+Copyright (c) Ansible Project
+GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt)
+SPDX-License-Identifier: GPL-3.0-or-later
+#}
+
+{% for l in vars['locale_list_simple'] %}
+{{ l['locale'] + '.' + l['charset'] + ' ' + l['charset'] }}
+{% endfor %}

tests/integration/targets/locale_gen/vars/main.yml

@@ -27,3 +27,11 @@ locale_list_basic:
   - localegen: de_CH.UTF-8
     locales: [de_CH.utf8]
     skip_removal: false
+
+# Simple locales list for non Debian distro
+
+locale_list_simple:
+  - locale: 'en_US'
+    charset: 'UTF-8'
+  - locale: 'pl_PL'
+    charset: 'UTF-8'