Merge 8eac2acd4f into f41f08e9e3

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>

.azure-pipelines/azure-pipelines.yml

@@ -98,19 +98,6 @@ stages:
               test: sanity
             - name: Units
               test: units
-  - stage: Sanity_2_14
-    displayName: Ansible 2.14 sanity
-    dependsOn: []
-    jobs:
-      - template: templates/matrix.yml
-        parameters:
-          nameFormat: "{0}"
-          testFormat: 2.14/{0}
-          targets:
-            - name: Sanity
-              test: sanity
-            - name: Units
-              test: units
   ## Docker
   - stage: Docker_devel
     displayName: Docker devel
@@ -120,12 +107,12 @@ stages:
         parameters:
           testFormat: devel/linux/{0}/1
           targets:
-            - name: Fedora 39
-              test: fedora39
-            - name: Ubuntu 20.04
-              test: ubuntu2004
+            - name: Fedora 40
+              test: fedora40
             - name: Ubuntu 22.04
               test: ubuntu2204
+            - name: Ubuntu 24.04
+              test: ubuntu2404
   - stage: Docker_2_17
     displayName: Docker 2.17
     dependsOn: []
@@ -175,24 +162,6 @@ stages:
               test: ubuntu2004
             - name: Ubuntu 22.04
               test: ubuntu2204
-  - stage: Docker_2_14
-    displayName: Docker 2.14
-    dependsOn: []
-    jobs:
-      - template: templates/matrix.yml
-        parameters:
-          testFormat: 2.14/linux/{0}/1
-          targets:
-            - name: CentOS 7
-              test: centos7
-            - name: Fedora 36
-              test: fedora36
-            - name: openSUSE 15 py3
-              test: opensuse15
-            - name: Ubuntu 20.04
-              test: ubuntu2004
-            - name: Ubuntu 22.04
-              test: ubuntu2204
 
   ## Remote
   - stage: Remote_devel
@@ -203,8 +172,8 @@ stages:
         parameters:
           testFormat: devel/{0}/1
           targets:
-            - name: RHEL 9.3
-              test: rhel/9.3
+            - name: RHEL 9.4
+              test: rhel/9.4
             - name: FreeBSD 13.3
               test: freebsd/13.3
   - stage: Remote_2_17
@@ -250,29 +219,12 @@ stages:
               test: rhel/9.1
             - name: FreeBSD 13.2
               test: freebsd/13.2
-  - stage: Remote_2_14
-    displayName: Remote 2.14
-    dependsOn: []
-    jobs:
-      - template: templates/matrix.yml
-        parameters:
-          testFormat: 2.14/{0}/1
-          targets:
-            - name: RHEL 7.9
-              test: rhel/7.9
-            - name: RHEL 8.6
-              test: rhel/8.6
-            - name: FreeBSD 13.2
-              test: freebsd/13.2
 
   ## Finally
 
   - stage: Summary
     condition: succeededOrFailed()
     dependsOn:
-      - Sanity_2_14
-      - Remote_2_14
-      - Docker_2_14
       - Sanity_2_15
       - Remote_2_15
       - Docker_2_15

changelogs/fragments/512.yml

@@ -0,0 +1,3 @@
+---
+bugfixes:
+  - selinux - use compat.selinux instead of selinux (https://github.com/ansible-collections/ansible.posix/issues/502).

plugins/modules/selinux.py

@@ -97,7 +97,7 @@ import traceback
 
 SELINUX_IMP_ERR = None
 try:
-    import selinux
+    from ansible.module_utils.compat import selinux as selinux_compat
     HAS_SELINUX = True
 except ImportError:
     SELINUX_IMP_ERR = traceback.format_exc()
@@ -181,9 +181,9 @@ def set_config_state(module, state, configfile):
 
 def set_state(module, state):
     if state == 'enforcing':
-        selinux.security_setenforce(1)
+        selinux_compat.security_setenforce(1)
     elif state == 'permissive':
-        selinux.security_setenforce(0)
+        selinux_compat.security_setenforce(0)
     elif state == 'disabled':
         pass
     else:
@@ -249,15 +249,15 @@ def main():
     policy = module.params['policy']
     state = module.params['state']
     update_kernel_param = module.params['update_kernel_param']
-    runtime_enabled = selinux.is_selinux_enabled()
-    runtime_policy = selinux.selinux_getpolicytype()[1]
+    runtime_enabled = selinux_compat.is_selinux_enabled()
+    runtime_policy = selinux_compat.selinux_getpolicytype()[1]
     runtime_state = 'disabled'
     kernel_enabled = None
     reboot_required = False
 
     if runtime_enabled:
         # enabled means 'enforcing' or 'permissive'
-        if selinux.security_getenforce():
+        if selinux_compat.security_getenforce():
             runtime_state = 'enforcing'
         else:
             runtime_state = 'permissive'