Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sap_*: add sles python3-rpm install to resolve #649 #733

Closed
wants to merge 9 commits into from
7 changes: 7 additions & 0 deletions roles/sap_anydb_install_oracle/tasks/oracledb_install_pre.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
---

# Reasons for noqa:
# - command-instead-of-module: Use command to install package required for Ansible Module package/package_facts
# - no-changed-when: If the package exists, zypper will make no change
- name: For SLES ensure OS Package for Python Lib of rpm bindings is enabled for System Python
ansible.builtin.command: set -o pipefail && zypper install --no-confirm --auto-agree-with-licenses python3-rpm # noqa command-instead-of-module no-changed-when
when: ansible_os_family == "Suse"

# See further:
## oracle-database-preinstall-12cr1.rpm
## oracle-database-preinstall-12cr2.rpm
Expand Down
7 changes: 7 additions & 0 deletions roles/sap_general_preconfigure/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,13 @@
tags:
- always

# Reasons for noqa:
# - command-instead-of-module: Use command to install package required for Ansible Module package/package_facts
# - no-changed-when: If the package exists, zypper will make no change
- name: For SLES ensure OS Package for Python Lib of rpm bindings is enabled for System Python
ansible.builtin.command: set -o pipefail && zypper install --no-confirm --auto-agree-with-licenses python3-rpm # noqa command-instead-of-module no-changed-when
when: ansible_os_family == "Suse"

# required for installation and configuration tasks:
- name: Gather package facts
ansible.builtin.package_facts:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# For the sake of readability and maintainability, suppress cosmetical ansible-lint warnings.
- name: "SAP HA Prepare Pacemaker - Make a list of potential VIP definitions"
ansible.builtin.set_fact:
__sap_ha_pacemaker_cluster_all_vip_fact: # noqa: jinja[spacing]
__sap_ha_pacemaker_cluster_all_vip_fact: # noqa jinja[spacing]
hana_scaleup_perf: "{{
{
sap_ha_pacemaker_cluster_vip_hana_primary_resource_name:
Expand Down
7 changes: 7 additions & 0 deletions roles/sap_hana_preconfigure/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,13 @@
__sap_hana_preconfigure_fact_ansible_distribution_minor_version: '{{ ansible_distribution_version.split(".")[1] }}'
when: ansible_distribution == 'RedHat'

# Reasons for noqa:
# - command-instead-of-module: Use command to install package required for Ansible Module package/package_facts
# - no-changed-when: If the package exists, zypper will make no change
- name: For SLES ensure OS Package for Python Lib of rpm bindings is enabled for System Python
ansible.builtin.command: set -o pipefail && zypper install --no-confirm --auto-agree-with-licenses python3-rpm # noqa command-instead-of-module no-changed-when
when: ansible_os_family == "Suse"

# required for installation and configuration tasks:
- name: Gather package facts
ansible.builtin.package_facts:
Expand Down
7 changes: 7 additions & 0 deletions roles/sap_netweaver_preconfigure/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,13 @@
assert_prefix: "assert-"
when: sap_netweaver_preconfigure_assert | d(false)

# Reasons for noqa:
# - command-instead-of-module: Use command to install package required for Ansible Module package/package_facts
# - no-changed-when: If the package exists, zypper will make no change
- name: For SLES ensure OS Package for Python Lib of rpm bindings is enabled for System Python
ansible.builtin.command: set -o pipefail && zypper install --no-confirm --auto-agree-with-licenses python3-rpm # noqa command-instead-of-module no-changed-when
when: ansible_os_family == "Suse"

# required for installation and configuration tasks:
- name: Gather package facts
ansible.builtin.package_facts:
Expand Down
7 changes: 7 additions & 0 deletions roles/sap_storage_setup/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,13 @@
ansible.builtin.include_vars:
file: "{{ ansible_os_family }}.yml"

# Reasons for noqa:
# - command-instead-of-module: Use command to install package required for Ansible Module package/package_facts
# - no-changed-when: If the package exists, zypper will make no change
- name: For SLES ensure OS Package for Python Lib of rpm bindings is enabled for System Python
ansible.builtin.command: set -o pipefail && zypper install --no-confirm --auto-agree-with-licenses python3-rpm # noqa command-instead-of-module no-changed-when
when: ansible_os_family == "Suse"

- name: SAP Storage Setup - Install additional OS packages
when: __sap_storage_setup_extra_packages is defined
ansible.builtin.package:
Expand Down
Loading