Merge pull request #151 from sap-linuxlab/dev

Merge dev to main for release 1.3.1

Author: marcelmamula

.ansible-lint

@@ -44,3 +44,5 @@ skip_list:
   - yaml[line-length]
   - no-tabs
   # - var-naming[no-jinja]
+  # Rule for variable naming is required because of stricter linting requirements.
+  - var-naming[no-role-prefix]

CHANGELOG.rst

@@ -4,6 +4,33 @@ community.sap\_infrastructure Release Notes
 
 .. contents:: Topics
 
+v1.3.1
+======
+
+Release Summary
+---------------
+
+Various minor enhancements and compatibility fixes for ansible-core 2.20.
+
+Minor Changes
+--------
+
+- collection - Update workflows with dedicated github actions (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/127)
+- collection - Add contributors list and sanity ignore files for future-proofing (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/130)
+- collection - Add linting rule to ignore stricter var naming rule in 2.20 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/136)
+- collection - Add kubernetes.core 5.3.0 to requirements (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/138)
+- sap_vm_provision - Remove ansible.posix dependency for issue 126 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/135)
+- sap_vm_provision - Add SLES 16 images, fix sshd_config and add python311 workaround (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/149)
+- sap_vm_provision/kubevirt_vm - Make usage of hugepages configurable (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/147)
+- sap_hypervisor_node_preconfigure - Update until conditional for ansible-core 2.19 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/133)
+- sap_hypervisor_node_preconfigure - Add check for webhook endpoint availability (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/142)
+
+Bugfixes
+--------
+
+- sap_hypervisor_node_preconfigure/kubevirt_vm - Fix installation of CNV operator (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/148)
+
+
 v1.3.0
 ======
 

CONTRIBUTORS.md

@@ -25,10 +25,10 @@ For specific role maintainers, see the `README.md` file in the corresponding rol
 | Name | Commits | Lines Changed | Last Commit |
 | ---- | ------- | ------------- | ----------- |
 | [Sean Freeman](https://github.com/sean-freeman) | 100 | 27358 | 2025-06-20 |
-| [Marcel Mamula](https://github.com/marcelmamula) | 53 | 5728 | 2025-09-30 |
-| [Nils Koenig](https://github.com/newkit) | 41 | 3753 | 2025-09-22 |
+| [Marcel Mamula](https://github.com/marcelmamula) | 60 | 6100 | 2026-01-07 |
+| [Nils Koenig](https://github.com/newkit) | 59 | 3971 | 2026-01-12 |
 | [Bernd Finger](https://github.com/berndfinger) | 7 | 285 | 2025-02-25 |
-| [Geetika Kapoor](https://github.com/geetikakay) | 5 | 116 | 2024-12-06 |
+| [Geetika Kapoor](https://github.com/geetikakay) | 6 | 124 | 2025-11-14 |
 | [Janine Fuchs](https://github.com/ja9fuchs) | 1 | 2 | 2024-09-03 |
 | [Markus Koch](https://github.com/rhmk) | 1 | 35 | 2024-08-27 |
 
@@ -53,8 +53,8 @@ For specific role maintainers, see the `README.md` file in the corresponding rol
 | Name | Commits | Lines Changed | Last Commit |
 | ---- | ------- | ------------- | ----------- |
 | [Sean Freeman](https://github.com/sean-freeman) | 85 | 22396 | 2025-05-26 |
-| [Marcel Mamula](https://github.com/marcelmamula) | 30 | 3264 | 2025-09-01 |
-| [Nils Koenig](https://github.com/newkit) | 14 | 1045 | 2025-09-22 |
+| [Marcel Mamula](https://github.com/marcelmamula) | 34 | 3496 | 2026-01-07 |
+| [Nils Koenig](https://github.com/newkit) | 23 | 1136 | 2026-01-12 |
 | [Bernd Finger](https://github.com/berndfinger) | 3 | 5 | 2025-02-19 |
 | [Janine Fuchs](https://github.com/ja9fuchs) | 1 | 2 | 2024-09-03 |
 | [Markus Koch](https://github.com/rhmk) | 1 | 35 | 2024-08-27 |
@@ -63,8 +63,8 @@ For specific role maintainers, see the `README.md` file in the corresponding rol
 
 | Name | Commits | Lines Changed | Last Commit |
 | ---- | ------- | ------------- | ----------- |
-| [Nils Koenig](https://github.com/newkit) | 26 | 1741 | 2025-09-22 |
-| [Geetika Kapoor](https://github.com/geetikakay) | 5 | 115 | 2024-12-06 |
+| [Nils Koenig](https://github.com/newkit) | 31 | 1847 | 2025-12-23 |
+| [Geetika Kapoor](https://github.com/geetikakay) | 6 | 123 | 2025-11-14 |
 | [Marcel Mamula](https://github.com/marcelmamula) | 3 | 455 | 2025-08-20 |
 | [Sean Freeman](https://github.com/sean-freeman) | 2 | 1870 | 2024-05-07 |
 

changelogs/changelog.yaml

@@ -115,3 +115,20 @@ releases:
       bugfixes:
         - sap_hypervisor_node_preconfigure - Fix ansible-lint errors from issue 101 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/116)
         - sap_vm_provision - Removed update-ca-certificates for issue 119 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/121)
+
+  1.3.1:
+    release_date: '2026-01-12'
+    changes:
+      release_summary: Various minor enhancements and compatibility fixes for ansible-core 2.20
+      minor_changes:
+        - collection - Update workflows with dedicated github actions (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/127)
+        - collection - Add contributors list and sanity ignore files for future-proofing (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/130)
+        - collection - Add linting rule to ignore stricter var naming rule in 2.20 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/136)
+        - collection - Add kubernetes.core 5.3.0 to requirements (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/138)
+        - sap_vm_provision - Remove ansible.posix dependency for issue 126 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/135)
+        - sap_vm_provision - Add SLES 16 images, fix sshd_config and add python311 workaround (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/149)
+        - sap_vm_provision/kubevirt_vm - Make usage of hugepages configurable (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/147)
+        - sap_hypervisor_node_preconfigure - Update until conditional for ansible-core 2.19 (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/133)
+        - sap_hypervisor_node_preconfigure - Add check for webhook endpoint availability (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/142)
+      bugfixes:
+        - sap_hypervisor_node_preconfigure/kubevirt_vm - Fix installation of CNV operator (https://github.com/sap-linuxlab/community.sap_infrastructure/pull/148)

galaxy.yml

@@ -10,7 +10,7 @@ namespace: community
 name: sap_infrastructure
 
 # The version of the collection. Must be compatible with semantic versioning
-version: 1.3.0
+version: 1.3.1
 
 # The path to the Markdown (.md) readme file. This path is relative to the root of the collection
 readme: README.md

requirements.yml

@@ -35,6 +35,9 @@ collections:
   - name: kubevirt.core
     type: galaxy
     version: 1.5.0
+  - name: kubernetes.core
+    type: galaxy
+    version: ">=5.2.0,<7.0.0"
   # For Red Hat OpenShift
   - name: community.okd
     type: galaxy

roles/sap_hypervisor_node_preconfigure/tasks/platform/redhat_ocp_virt/operators/install-cnv-operator.yml

@@ -95,15 +95,11 @@
   delay: 60
   until: __sap_hypervisor_node_preconfigure_register_hco_webhook_pod.resources | selectattr('status.phase', 'equalto', 'Running') | list | length == __sap_hypervisor_node_preconfigure_register_hco_webhook_pod.resources | length
 
-- name: Wait and check if hco-webhook-service exists in the namespace "{{ sap_hypervisor_node_preconfigure_ocpv_namespace }}"
-  kubernetes.core.k8s_info:
-    kind: Service
-    namespace: "{{ sap_hypervisor_node_preconfigure_ocpv_namespace }}"
-    name: hco-webhook-service
-  register: webhook_service
-  retries: 2
-  delay: 60
-  until: webhook_service.resources
+- name: Check if hco-webhook-service webhook endpoint is available
+  ansible.builtin.include_tasks: wait_for_webhook_endpoints.yml
+  vars:
+    sap_hypervisor_node_preconfigure_webhook_service_name: hco-webhook-service
+    sap_hypervisor_node_preconfigure_webhook_namespace: "{{ sap_hypervisor_node_preconfigure_ocpv_namespace }}"
 
 - name: Create CNV HyperConverged
   kubernetes.core.k8s:

roles/sap_hypervisor_node_preconfigure/tasks/platform/redhat_ocp_virt/operators/install-nmstate-operator.yml

@@ -39,7 +39,7 @@
   register: operatorgroup_status
   retries: 30
   delay: 10
-  until: operatorgroup_status.resources | length > 0
+  until: operatorgroup_status.resources | default([]) | length > 0
 
 - name: Subscribe to the nmstate Operator
   kubernetes.core.k8s:
@@ -98,7 +98,7 @@
   register: nmstate_status
   retries: 30
   delay: 10
-  until: nmstate_status.resources | length > 0
+  until: nmstate_status.resources | default([]) | length > 0
 
 - name: Wait and check for pod with label name nmstate-webhook under the namespace "{{ sap_hypervisor_node_preconfigure_nmstate_namespace }}"
   kubernetes.core.k8s_info:
@@ -110,3 +110,9 @@
   retries: 5
   delay: 60
   until: sap_hypervisor_node_preconfigure_register_webhook_pod.resources | selectattr('status.phase', 'equalto', 'Running') | list | length == sap_hypervisor_node_preconfigure_register_webhook_pod.resources | length
+
+- name: Check if nmstate webhook endpoint is available
+  ansible.builtin.include_tasks: wait_for_webhook_endpoints.yml
+  vars:
+    sap_hypervisor_node_preconfigure_webhook_service_name: nmstate-webhook
+    sap_hypervisor_node_preconfigure_webhook_namespace: openshift-nmstate

roles/sap_hypervisor_node_preconfigure/tasks/platform/redhat_ocp_virt/operators/install-sriov-operator.yml

@@ -68,7 +68,7 @@
   register: operatorgroup_status
   retries: 30
   delay: 10
-  until: operatorgroup_status.resources
+  until: operatorgroup_status.resources | default([]) | length > 0
 
 - name: Check if SriovOperatorConfig exists  # noqa: ignore-errors
   kubernetes.core.k8s_info:

roles/sap_hypervisor_node_preconfigure/tasks/platform/redhat_ocp_virt/operators/wait_for_webhook_endpoints.yml

@@ -0,0 +1,24 @@
+---
+- name: Wait for webhook endpoints
+  kubernetes.core.k8s_info:
+    api_version: v1
+    kind: Endpoints
+    name: "{{ sap_hypervisor_node_preconfigure_webhook_service_name }}"
+    namespace: "{{ sap_hypervisor_node_preconfigure_webhook_namespace }}"
+  register: __sap_hypervisor_node_preconfigure_register_webhook_check
+  until:
+    - __sap_hypervisor_node_preconfigure_register_webhook_check is defined
+    - __sap_hypervisor_node_preconfigure_register_webhook_check.resources | length > 0
+    - __sap_hypervisor_node_preconfigure_register_webhook_check.resources[0].subsets is defined
+    - __sap_hypervisor_node_preconfigure_register_webhook_check.resources[0].subsets | length > 0
+    - __sap_hypervisor_node_preconfigure_register_webhook_check.resources[0].subsets[0].addresses is defined
+    - __sap_hypervisor_node_preconfigure_register_webhook_check.resources[0].subsets[0].addresses | length > 0
+  retries: "{{ sap_hypervisor_node_preconfigure_webhook_retries | default(30) }}"
+  delay: "{{ sap_hypervisor_node_preconfigure_webhook_delay | default(10) }}"
+
+- name: Fail if webhook endpoints never became available
+  ansible.builtin.fail:
+    msg: >
+      Webhook {{ sap_hypervisor_node_preconfigure_webhook_service_name }} in namespace
+      {{ sap_hypervisor_node_preconfigure_webhook_namespace }} did not become available in time.
+  when: __sap_hypervisor_node_preconfigure_register_webhook_check is failed