.gitignore

@@ -1,3 +1,2 @@
 SOURCES/auto-maintenance-e2a233f5a0cb68363798bf014b16552cca681bd0.tar.gz
-SOURCES/community.sap_install-60f5f40497d54c46b0bd3f79ef1fef4ada148119.tar.gz
-SOURCES/prepare-redhat.sap_install-main.tar.bz2
+SOURCES/redhat.sap_install-1.3.5.tar.gz

.rhel-system-roles-sap.metadata

@@ -1,3 +1,2 @@
 584f8d1681adf13cb7af256a7b38d94ca80dcab2 SOURCES/auto-maintenance-e2a233f5a0cb68363798bf014b16552cca681bd0.tar.gz
-7189971675a00ae0c56a2ea4776f4d9975c8b624 SOURCES/community.sap_install-60f5f40497d54c46b0bd3f79ef1fef4ada148119.tar.gz
-8aa3602249c3bef424ce51ab1d4c4b2fb686d674 SOURCES/prepare-redhat.sap_install-main.tar.bz2
+835376d0e4799989e9d4deb21ac7e1d335c941de SOURCES/redhat.sap_install-1.3.5.tar.gz

SOURCES/community.sap_install-shebang.patch

@@ -1,90 +0,0 @@
-diff -up community.sap_install/roles/sap_general_preconfigure/tests/run-sap_general_preconfigure-tests.py.me community.sap_install/roles/sap_general_preconfigure/tests/run-sap_general_preconfigure-tests.py
---- community.sap_install/roles/sap_general_preconfigure/tests/run-sap_general_preconfigure-tests.py.me	2022-07-12 16:23:48.262581739 +0200
-+++ community.sap_install/roles/sap_general_preconfigure/tests/run-sap_general_preconfigure-tests.py	2022-07-12 16:36:43.620709628 +0200
-@@ -1,4 +1,4 @@
--#!/usr/bin/env python
-+#!/usr/bin/python3
- 
- """
- Tests for role sap_general_preconfigure
-diff -up community.sap_install/roles/sap_hana_install/tests/install/run-sap_hana_install-install-tests.py.me community.sap_install/roles/sap_hana_install/tests/install/run-sap_hana_install-install-tests.py
---- community.sap_install/roles/sap_hana_install/tests/install/run-sap_hana_install-install-tests.py.me	2022-07-12 16:36:52.262900877 +0200
-+++ community.sap_install/roles/sap_hana_install/tests/install/run-sap_hana_install-install-tests.py	2022-07-12 16:36:59.246055414 +0200
-@@ -1,4 +1,4 @@
--#!/usr/bin/env python
-+#!/usr/bin/python3
- 
- import os
- import sys
-diff -up community.sap_install/roles/sap_hana_install/tests/sapcar/run-sap_hana_install-sapcar-tests.py.me community.sap_install/roles/sap_hana_install/tests/sapcar/run-sap_hana_install-sapcar-tests.py
---- community.sap_install/roles/sap_hana_install/tests/sapcar/run-sap_hana_install-sapcar-tests.py.me	2022-07-12 16:37:17.272454327 +0200
-+++ community.sap_install/roles/sap_hana_install/tests/sapcar/run-sap_hana_install-sapcar-tests.py	2022-07-12 16:37:24.142606360 +0200
-@@ -1,4 +1,4 @@
--#!/usr/bin/env python
-+#!/usr/bin/python3
- 
- import os
- import sys
-diff -up community.sap_install/roles/sap_hana_preconfigure/tests/run-sap_hana_preconfigure-tests.py.me community.sap_install/roles/sap_hana_preconfigure/tests/run-sap_hana_preconfigure-tests.py
---- community.sap_install/roles/sap_hana_preconfigure/tests/run-sap_hana_preconfigure-tests.py.me	2022-07-12 16:37:52.794240408 +0200
-+++ community.sap_install/roles/sap_hana_preconfigure/tests/run-sap_hana_preconfigure-tests.py	2022-07-12 16:37:58.553367854 +0200
-@@ -1,4 +1,4 @@
--#!/usr/bin/env python
-+#!/usr/bin/python3
- 
- """
- Tests for role sap_hana_preconfigure
-diff -up community.sap_install/roles/sap_netweaver_preconfigure/tests/run-sap_netweaver_preconfigure-tests.py.me community.sap_install/roles/sap_netweaver_preconfigure/tests/run-sap_netweaver_preconfigure-tests.py
---- community.sap_install/roles/sap_netweaver_preconfigure/tests/run-sap_netweaver_preconfigure-tests.py.me	2022-07-12 16:37:34.384833016 +0200
-+++ community.sap_install/roles/sap_netweaver_preconfigure/tests/run-sap_netweaver_preconfigure-tests.py	2022-07-12 16:37:41.495990384 +0200
-@@ -1,4 +1,4 @@
--#!/usr/bin/env python
-+#!/usr/bin/python3
- 
- """
- Tests for role sap_netweaver_preconfigure
-diff -up community.sap_install/tools/swpm2_parameters_inifile_generate.py.me community.sap_install/tools/swpm2_parameters_inifile_generate.py
---- community.sap_install/tools/swpm2_parameters_inifile_generate.py.me	2022-07-12 16:38:10.026621753 +0200
-+++ community.sap_install/tools/swpm2_parameters_inifile_generate.py	2022-07-12 16:38:29.915061875 +0200
-@@ -1,4 +1,4 @@
--#!/usr/bin/env python
-+#!/usr/bin/python3
- from __future__ import (absolute_import, division, print_function)
- __metaclass__ = type
- 
-diff -up community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_general_preconfigure/tools/beautify-assert-output.sh.me community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_general_preconfigure/tools/beautify-assert-output.sh
---- community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_general_preconfigure/tools/beautify-assert-output.sh.me	2022-07-12 17:48:10.476508351 +0200
-+++ community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_general_preconfigure/tools/beautify-assert-output.sh	2022-07-12 17:48:27.813896005 +0200
-@@ -1,4 +1,4 @@
--#!/bin/bash
-+#!/usr/bin/bash
- 
- # default font color: Light Cyan, which should be readable on both bright and dark background
- __FONT_COLOR=36m
-diff -up community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_install/files/tmp/tail-f-hdblcm-install-trc.sh.me community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_install/files/tmp/tail-f-hdblcm-install-trc.sh
---- community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_install/files/tmp/tail-f-hdblcm-install-trc.sh.me	2022-07-12 17:48:35.527068474 +0200
-+++ community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_install/files/tmp/tail-f-hdblcm-install-trc.sh	2022-07-12 17:48:43.703251292 +0200
-@@ -1,4 +1,4 @@
--#!/bin/bash
-+#!/usr/bin/bash
- 
- while true; do
- # in case hdblcm has not yet started, we assume that it is waiting for sapdsigner to complete:
-diff -up community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh.me community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh
---- community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh.me	2022-07-12 17:49:22.425117059 +0200
-+++ community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh	2022-07-12 17:49:30.241291825 +0200
-@@ -1,4 +1,4 @@
--#!/bin/bash
-+#!/usr/bin/bash
- 
- # default font color: Light Cyan, which should be readable on both bright and dark background
- __FONT_COLOR=36m
-diff -up community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh.me community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh
---- community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh.me	2022-07-12 17:48:51.584427510 +0200
-+++ community.sap_install-29b37456211c8bf825547f614652f24cf324860a/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh	2022-07-12 17:49:00.678630847 +0200
-@@ -1,4 +1,4 @@
--#!/bin/bash
-+#!/usr/bin/bash
- 
- # default font color: Light Cyan, which should be readable on both bright and dark background
- __FONT_COLOR=36m

SOURCES/redhat.sap_install-1.3.4-add_tools.patch

@@ -0,0 +1,128 @@
+commit 83db825f7f30445408efa6061631d8dd0590e891
+Author: Than Ngo <than@redhat.com>
+Date:   Thu Jan 25 16:17:21 2024 +0100
+
+    add missing tools folders
+    
+    Signed-off-by: Than Ngo <than@redhat.com>
+
+diff --git a/roles/sap_general_preconfigure/tools/beautify-assert-output.sh b/roles/sap_general_preconfigure/tools/beautify-assert-output.sh
+new file mode 100755
+index 0000000..f48087c
+--- /dev/null
++++ b/roles/sap_general_preconfigure/tools/beautify-assert-output.sh
+@@ -0,0 +1,34 @@
++#!/bin/bash
++
++# default font color: Light Cyan, which should be readable on both bright and dark background
++__FONT_COLOR=36m
++
++if [[ ${1}. == "font_light_gray". ]]; then
++   __FONT_COLOR=37m
++elif [[ ${1}. == "font_no_color". ]]; then
++   __FONT_COLOR=30m
++fi
++
++if [[ ${2}. == "reset." ]]; then
++   awk 'BEGIN{printf ("\033['${__FONT_COLOR}'Resetting font color\n")}'
++   exit
++fi
++
++awk '{sub ("    \"msg\": ", "")}
++  /TASK/{task_line=$0}
++  /fatal:/{fatal_line=$0; nfatal[host]++}
++  /...ignoring/{nfatal[host]--; if (nfatal[host]<0) nfatal[host]=0}
++  /^[a-z]/&&/: \[/{gsub ("\\[", ""); gsub ("]", ""); gsub (":", ""); host=$2}
++  /SAP note/{print "\033['${__FONT_COLOR}'[" host"] "$0}
++  /FAIL:/{nfail[host]++; print "\033[31m[" host"] "$0}
++  /WARN:/{nwarn[host]++; print "\033[33m[" host"] "$0}
++  /PASS:/{npass[host]++; print "\033[32m[" host"] "$0}
++  /INFO:/{print "\033[34m[" host"] "$0}
++  /changed/&&/unreachable/{print "\033['${__FONT_COLOR}'[" host"] "$0}
++  END{print ("---"); for (var in npass) {printf ("[%s] ", var); if (nfatal[var]>0) {
++        printf ("\033[31mFATAL ERROR!!! Playbook might have been aborted!!!\033['${__FONT_COLOR}' Last TASK and fatal output:\n"); print task_line, fatal_line
++        exit 199
++     }
++     else printf ("\033[31mFAIL: %d  \033[33mWARN: %d  \033[32mPASS: %d\033['${__FONT_COLOR}'\n", nfail[var], nwarn[var], npass[var])}
++     if (nfail[var] != 0) exit (nfail[var])
++  }'
+diff --git a/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh b/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh
+new file mode 100755
+index 0000000..f48087c
+--- /dev/null
++++ b/roles/sap_hana_preconfigure/tools/beautify-assert-output.sh
+@@ -0,0 +1,34 @@
++#!/bin/bash
++
++# default font color: Light Cyan, which should be readable on both bright and dark background
++__FONT_COLOR=36m
++
++if [[ ${1}. == "font_light_gray". ]]; then
++   __FONT_COLOR=37m
++elif [[ ${1}. == "font_no_color". ]]; then
++   __FONT_COLOR=30m
++fi
++
++if [[ ${2}. == "reset." ]]; then
++   awk 'BEGIN{printf ("\033['${__FONT_COLOR}'Resetting font color\n")}'
++   exit
++fi
++
++awk '{sub ("    \"msg\": ", "")}
++  /TASK/{task_line=$0}
++  /fatal:/{fatal_line=$0; nfatal[host]++}
++  /...ignoring/{nfatal[host]--; if (nfatal[host]<0) nfatal[host]=0}
++  /^[a-z]/&&/: \[/{gsub ("\\[", ""); gsub ("]", ""); gsub (":", ""); host=$2}
++  /SAP note/{print "\033['${__FONT_COLOR}'[" host"] "$0}
++  /FAIL:/{nfail[host]++; print "\033[31m[" host"] "$0}
++  /WARN:/{nwarn[host]++; print "\033[33m[" host"] "$0}
++  /PASS:/{npass[host]++; print "\033[32m[" host"] "$0}
++  /INFO:/{print "\033[34m[" host"] "$0}
++  /changed/&&/unreachable/{print "\033['${__FONT_COLOR}'[" host"] "$0}
++  END{print ("---"); for (var in npass) {printf ("[%s] ", var); if (nfatal[var]>0) {
++        printf ("\033[31mFATAL ERROR!!! Playbook might have been aborted!!!\033['${__FONT_COLOR}' Last TASK and fatal output:\n"); print task_line, fatal_line
++        exit 199
++     }
++     else printf ("\033[31mFAIL: %d  \033[33mWARN: %d  \033[32mPASS: %d\033['${__FONT_COLOR}'\n", nfail[var], nwarn[var], npass[var])}
++     if (nfail[var] != 0) exit (nfail[var])
++  }'
+diff --git a/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh b/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh
+new file mode 100755
+index 0000000..f48087c
+--- /dev/null
++++ b/roles/sap_netweaver_preconfigure/tools/beautify-assert-output.sh
+@@ -0,0 +1,34 @@
++#!/bin/bash
++
++# default font color: Light Cyan, which should be readable on both bright and dark background
++__FONT_COLOR=36m
++
++if [[ ${1}. == "font_light_gray". ]]; then
++   __FONT_COLOR=37m
++elif [[ ${1}. == "font_no_color". ]]; then
++   __FONT_COLOR=30m
++fi
++
++if [[ ${2}. == "reset." ]]; then
++   awk 'BEGIN{printf ("\033['${__FONT_COLOR}'Resetting font color\n")}'
++   exit
++fi
++
++awk '{sub ("    \"msg\": ", "")}
++  /TASK/{task_line=$0}
++  /fatal:/{fatal_line=$0; nfatal[host]++}
++  /...ignoring/{nfatal[host]--; if (nfatal[host]<0) nfatal[host]=0}
++  /^[a-z]/&&/: \[/{gsub ("\\[", ""); gsub ("]", ""); gsub (":", ""); host=$2}
++  /SAP note/{print "\033['${__FONT_COLOR}'[" host"] "$0}
++  /FAIL:/{nfail[host]++; print "\033[31m[" host"] "$0}
++  /WARN:/{nwarn[host]++; print "\033[33m[" host"] "$0}
++  /PASS:/{npass[host]++; print "\033[32m[" host"] "$0}
++  /INFO:/{print "\033[34m[" host"] "$0}
++  /changed/&&/unreachable/{print "\033['${__FONT_COLOR}'[" host"] "$0}
++  END{print ("---"); for (var in npass) {printf ("[%s] ", var); if (nfatal[var]>0) {
++        printf ("\033[31mFATAL ERROR!!! Playbook might have been aborted!!!\033['${__FONT_COLOR}' Last TASK and fatal output:\n"); print task_line, fatal_line
++        exit 199
++     }
++     else printf ("\033[31mFAIL: %d  \033[33mWARN: %d  \033[32mPASS: %d\033['${__FONT_COLOR}'\n", nfail[var], nwarn[var], npass[var])}
++     if (nfail[var] != 0) exit (nfail[var])
++  }'

SPECS/rhel-system-roles-sap.spec

@@ -1,11 +1,9 @@
-%if 0%{?rhel} && ! 0%{?epel}
-%bcond_with ansible
-%else
-%bcond_without ansible
-%endif
-
+# disable collection_artifact by default
 %bcond_with collection_artifact
 
+# disable collection by default since version 3.6.0 (1.3.4)
+%bcond_with collection
+
 %if 0%{?fedora} || 0%{?rhel} > 8
 %bcond_without html
 %else
@@ -26,45 +24,45 @@
 # collection_version has to be increased by every rebuild
 # otherwise, it cannot be uploaded to Automation Hub and Galaxy
 # due to version conflict
-%define collection_version 1.2.3
+%define collection_version 1.3.5
 
 %global mainid e2a233f5a0cb68363798bf014b16552cca681bd0
-%global commit_id_sap_install 60f5f40497d54c46b0bd3f79ef1fef4ada148119
+%global commit_id_sap_install %{collection_version}
 %global rolename1 sap_general_preconfigure
-%global rolename2 sap_hana_preconfigure
-%global rolename3 sap_netweaver_preconfigure
-%global rolename4 sap_hana_install
+%global rolename2 sap_ha_install_hana_hsr
+%global rolename3 sap_hana_install
+%global rolename4 sap_hana_preconfigure
 %global rolename5 sap_ha_pacemaker_cluster
-%global rolename6 sap_ha_set_netweaver
-%global rolename7 sap_hostagent
-%global rolename8 sap_install_media_detect
-%global rolename9 sap_storage
-%global rolename10 sap_swpm
-%global rolename11 sap_ha_install_hana_hsr
+%global rolename6 sap_hypervisor_node_preconfigure
+%global rolename7 sap_netweaver_preconfigure
+%global rolename8 sap_swpm
+%global rolename9 sap_vm_preconfigure
 %global src_owner linux-system-roles
-%global github_repo_sap_install community.sap_install
-%global rolename_rhel %{rolename1} %{rolename2} %{rolename3} %{rolename4} %{rolename5} %{rolename10} %{rolename11}
-%global rolename_community %{rolename_rhel} %{rolename1} %{rolename6} %{rolename7} %{rolename8} %{rolename9}
-%if 0%{?fedora}
-%global rolenames %{rolename_community}
-%else
+%global github_repo_sap_install redhat.sap_install
+%global rolename_rhel %{rolename1} %{rolename2} %{rolename3} %{rolename4} %{rolename5} %{rolename6} %{rolename7} %{rolename8} %{rolename9} 
 %global rolenames %{rolename_rhel}
-%endif
 
 Name: %{package_name}
 Summary: System Roles to configure RHEL for running SAP NetWeaver- or SAP HANA-based products
-Version: 3.5.0
-Release: 1%{?dist}
+Version: 3.6.0
+Release: 2%{?dist}
 License: GPL-3.0-or-later
-Url: https://github.com/sap-linuxlab/community.sap_install
+Url: https://github.com/redhat-sap/community.sap_install
 Source0: https://github.com/linux-system-roles/auto-maintenance/archive/%{mainid}/auto-maintenance-%{mainid}.tar.gz
-Source1: https://github.com/sap-linuxlab/%{github_repo_sap_install}/archive/%{commit_id_sap_install}/%{github_repo_sap_install}-%{commit_id_sap_install}.tar.gz
-# files/README.md, files/roles/sap_swpm/README.md from https://gitlab.cee.redhat.com/sap-linuxlab/prepare-redhat.sap_install
-Source2: prepare-redhat.sap_install-main.tar.bz2
-Patch1: community.sap_install-shebang.patch
+Source1: https://github.com/redhat-sap/redhat.sap_install/archive/%{commit_id_sap_install}/%{github_repo_sap_install}-%{commit_id_sap_install}.tar.gz
+
+# add collection files
+Patch1: redhat.sap_install-1.3.5-collections-files-324bfb8bab50d2baa1c4aacdb59e4e5c9e2ba74f.patch
+
+# add tests files
+Patch2: redhat.sap_install-1.3.4-add_tests.patch
+
+# add tools files
+Patch3: redhat.sap_install-1.3.4-add_tools.patch
 
 BuildArch: noarch
 
+Requires: rhel-system-roles
 
 %if %{with html}
 # Requirements for md2html.sh to build the documentation
@@ -89,8 +87,14 @@ Requires: python-jmespath
 Requires: python-netaddr
 %endif
 
-%if %{with ansible}
-BuildRequires: ansible-core >= 2.9.10
+# NOTE: ansible-core is in rhel-8.6 and newer, but not installable
+# in buildroot as it depended on modular Python.
+# It has been installable at buildtime in 8.8 and newer.
+%if %{with collection}
+%if 0%{?rhel} >= 8
+BuildRequires: ansible-core >= 2.11.0
+Requires: (ansible-core >= 2.11.0 or ansible >= 2.9.0)
+%endif
 %endif
 
 %if %{undefined __ansible_provides}
@@ -99,6 +103,10 @@ Provides: ansible-collection(%{collection_namespace}.%{collection_name}) = %{ver
 # be compatible with the usual Fedora Provides:
 Provides: ansible-collection-%{collection_namespace}-%{collection_name} = %{version}-%{release}
 
+# ansible-galaxy is available by ansible-core on RHEL 8.6 and newer at buildtime.
+%define ansible_collection_build() ansible-galaxy collection build
+%define ansible_collection_install() ansible-galaxy collection install -n -p %{buildroot}%{_datadir}/ansible/collections %{collection_namespace}-%{collection_name}-%{collection_version}.tar.gz
+
 %if 0%{?fedora} || 0%{?rhel} >= 8
 %{!?ansible_collection_files:%define ansible_collection_files %{_datadir}/ansible/collections/ansible_collections/%{collection_namespace}/}
 %else
@@ -107,13 +115,6 @@ Provides: ansible-collection-%{collection_namespace}-%{collection_name} = %{vers
 %endif
 %endif
 
-
-%if %{without ansible}
-%define ansible_collection_build_install() tar -cf %{_tmppath}/%{collection_namespace}-%{collection_name}-%{collection_version}.tar.gz .; mkdir -p %{buildroot}%{ansible_collection_files}%{collection_name}; (cd %{buildroot}%{ansible_collection_files}%{collection_name}; tar -xf %{_tmppath}/%{collection_namespace}-%{collection_name}-%{collection_version}.tar.gz)
-%else
-%define ansible_collection_build_install() ansible-galaxy collection build; ansible-galaxy collection install -n -p %{buildroot}%{_datadir}/ansible/collections %{collection_namespace}-%{collection_name}-%{collection_version}.tar.gz
-%endif
-
 %description
 Collection of Ansible roles which configures a RHEL system according
 to applicable SAP notes so that any SAP software can be installed.
@@ -127,58 +128,26 @@ Collection artifact for %{name}. This package contains %{collection_namespace}-%
 %endif
 
 %prep
-%setup -q -a1 -a2 -n auto-maintenance-%{mainid}
-
-# add correct version and namespace for Automation Hub and Galaxy
-cp %{github_repo_sap_install}-%{commit_id_sap_install}/galaxy.yml .
-sed -i -e "s|^namespace: .*|namespace: %{collection_namespace}|" galaxy.yml
-sed -i -e "s|^version: .*|version: %{collection_version}|" galaxy.yml
-
-# update README.md and roles/sap_swpm/README.md
-cp prepare-redhat.sap_install-main/files/README.md %{github_repo_sap_install}-%{commit_id_sap_install}/
-cp prepare-redhat.sap_install-main/files/roles/sap_swpm/README.md %{github_repo_sap_install}-%{commit_id_sap_install}/roles/sap_swpm/README.md
-
-# remove roles/sap_swpm/requirements.yml
-rm -f %{github_repo_sap_install}-%{commit_id_sap_install}/roles/sap_swpm/requirements.yml
+%setup -q -a1 -n auto-maintenance-%{mainid}
 
 pushd %{github_repo_sap_install}-%{commit_id_sap_install}/
+%patch -P1 -p1
+%patch -P2 -p1
+%patch -P3 -p1
 # workaround for md2html error
 sed -i -e "s|for POWER ||g" roles/sap_hana_preconfigure/README.md
-%patch -P1 -p1
-
-# modify collection-requirements.yml, fedora.linux_system_roles -> redhat.rhel_system_roles
-for r in %{rolenames} ; do
-  yml=roles/${r}/meta/collection-requirements.yml
-  [ -f ${yml} ] && sed -i "s,fedora.linux_system_roles,redhat.rhel_system_roles," ${yml}
-done
-
-# modify defaults/main.yml, fedora.linux_system_roles -> redhat.rhel_system_roles
-for r in %{rolenames} ; do
-  yml=roles/${r}/defaults/main.yml
-  [ -f ${yml} ] && sed -i "s,system_roles_collection: 'fedora.linux_system_roles',system_roles_collection: 'redhat.rhel_system_roles'," ${yml}
-done
 popd
 
 # remove zero file and symlinks
 find . -type f -size 0 -delete
 find . -type l -delete
 
-pushd %{github_repo_sap_install}-%{commit_id_sap_install}/roles/
-# remove all SLES stuff and RHEL 6 stuff
-rm -rf */tasks/SLES/
-rm -f  */vars/SLES_15.yml
-rm -rf sap_hana_preconfigure/tasks/sapnote/1275776 # SLES - sapconf, saptune, etc
-rm -rf sap_hana_preconfigure/tasks/sapnote/1944799 # SLES - HANA install and config guidelines
-rm -f  sap_hana_preconfigure/tasks/sapnote/1944799.yml
-rm -f  sap_hana_preconfigure/tasks/sapnote/2013638.yml # RHEL 6.5 - OS settings
-rm -f  sap_hana_preconfigure/tasks/sapnote/2136965.yml # RHEL 6.6 - OS settings
-rm -f  sap_hana_preconfigure/tasks/sapnote/2247020.yml # RHEL 6.7 - OS settings
-rm -rf sap_hana_preconfigure/tasks/sapnote/2578899 # SLES 15 - installation
-rm -rf sap_hana_preconfigure/tasks/sapnote/2684254 # SLES 15 - OS settings for HANA
-rm -f  sap_hana_preconfigure/vars/RedHat_6.5.yml
-rm -f  sap_hana_preconfigure/vars/RedHat_6.6.yml
-rm -f  sap_hana_preconfigure/vars/RedHat_6.7.yml
-popd
+# fix python and bash shebangs
+find -type f \( -iname "*.py" \) -exec sed -i '1s=^#! */usr/bin/\(python\|env python\)[23]\?=#!/usr/bin/python3=' {} +
+find -type f \( -iname "*.sh" \) -exec sed -i '1s=^#! */bin/bash=#!/usr/bin/bash=' {} +
+
+# remove json files, these are already present in the collection subdirectory
+rm -f FILES.json MANIFEST.json
 
 %build
 %if %{with html}
@@ -189,6 +158,7 @@ done
 sh md2html.sh $readmes %{github_repo_sap_install}-%{commit_id_sap_install}/README.md
 %endif
 
+%if %{with collection}
 # create dest-path
 mkdir .collections
 
@@ -206,6 +176,13 @@ cp -p galaxy.yml .collections/ansible_collections/%{collection_namespace}/%{coll
 # update README.md 
 cp %{github_repo_sap_install}-%{commit_id_sap_install}/README.md .collections/ansible_collections/%{collection_namespace}/%{collection_name}
 
+# Build the collection
+pushd .collections/ansible_collections/%{collection_namespace}/%{collection_name}/
+%ansible_collection_build
+popd
+
+%endif
+
 %install
 mkdir -p $RPM_BUILD_ROOT%{_datadir}/ansible/roles \
          $RPM_BUILD_ROOT%{_pkgdocdir}
@@ -213,6 +190,7 @@ mkdir -p $RPM_BUILD_ROOT%{_datadir}/ansible/roles \
 # create file selection for documents
 echo "%dir %{_pkgdocdir}" > files_section.txt
 cp %{github_repo_sap_install}-%{commit_id_sap_install}/README.md $RPM_BUILD_ROOT%{_pkgdocdir}/
+
 %if %{with html}
 cp %{github_repo_sap_install}-%{commit_id_sap_install}/README.html $RPM_BUILD_ROOT%{_pkgdocdir}/
 %endif
@@ -232,9 +210,13 @@ install -m 644 %{github_repo_sap_install}-%{commit_id_sap_install}/roles/$role/R
 done
 
 # install collection files
+%if %{with collection}
 pushd .collections/ansible_collections/%{collection_namespace}/%{collection_name}/
-%ansible_collection_build_install
+%ansible_collection_install
 popd
+%else
+cp -pR %{github_repo_sap_install}-%{commit_id_sap_install}/collections $RPM_BUILD_ROOT%{_datadir}/ansible/
+%endif
 
 # install collection_artifact
 %if %{with collection_artifact}
@@ -248,19 +230,36 @@ popd
 %dir %{_datadir}/ansible
 %dir %{_datadir}/ansible/roles
 %doc %{_pkgdocdir}/*/README.md
+%license %{_pkgdocdir}/*/LICENSE
+%{_datadir}/ansible/roles/*
+
 %if %{with html}
 %{_pkgdocdir}/*/README.html
 %endif
-%license %{_pkgdocdir}/*/LICENSE
-%{_datadir}/ansible/roles/*
+%if %{with collection}
 %{ansible_collection_files}
-
+%else
+%{_datadir}/ansible/collections
+%endif
 %if %{with collection_artifact}
 %files collection-artifact
 %{_datadir}/ansible/collections/%{collection_namespace}-%{collection_name}-%{collection_version}.tar.gz
 %endif
 
 %changelog
+* Mon Feb 05 2024 Than Ngo <than@redhat.com> - 3.6.0-2
+- sap_hypervisor_node_preconfigure: fix for role name and path for included tasks
+- update collection files
+Related: RHEL-16897
+
+* Thu Jan 25 2024 Than Ngo <than@redhat.com> - 3.6.0-1
+- rebase sap roles to version 1.3.4
+- add collection files
+- update README files and add missing galaxy.yml, LICENSE
+- fix syntax error for defining dependent collections in galaxy.yml
+- add tools and tests
+Resolves: RHEL-16897
+
 * Mon May 08 2023 Than Ngo <than@redhat.com> - 3.5.0-1
 - updated README.md files for sap and sap_swpm roles
 - migrated to SPDX license