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

collection: merge dev to main for release 1.4.1 #771

Merged
merged 178 commits into from
Jun 24, 2024
Merged

collection: merge dev to main for release 1.4.1 #771

merged 178 commits into from
Jun 24, 2024

Conversation

berndfinger
Copy link
Member

No description provided.

berndfinger and others added 30 commits February 2, 2024 19:13
Update dev from latest main for version 1.4.0
sap_*_preconfigure: disable and stop sapconf when saptune run
Solves issue #641.

Signed-off-by: Bernd Finger <[email protected]>
Partially solves issue #641.

Signed-off-by: Bernd Finger <[email protected]>
Partially solves issue #641.

Signed-off-by: Bernd Finger <[email protected]>
Solves issue #641.

Signed-off-by: Bernd Finger <[email protected]>
- remove a trailing space
- add a `noqa no-changed-when` in order to prevent a false-positive of
  the linting
sap_hana_preconfigure: Catch SELinux disabled
... for limiting the role scope.

Relates to #342.

Sample playbook calls:

$ ansible-playbook sap.yml --tags \
sap_general_preconfigure_configuration,sap_general_preconfigure_configuration_all_steps

$ ansible-playbook sap.yml --tags \
sap_general_preconfigure_configuration,sap_general_preconfigure_2772999_02

$ ansible-playbook sap.yml --tags \
sap_general_preconfigure_configuration,sap_general_preconfigure_selinux

$ ansible-playbook sap.yml --tags \
sap_general_preconfigure_configuration,sap_general_preconfigure_configuration_all_steps \
--skip-tags sap_general_preconfigure_2772999_02

$ ansible-playbook sap.yml --tags \
sap_general_preconfigure_configuration,sap_general_preconfigure_configuration_all_steps \
--skip-tags sap_general_preconfigure_selinux

Signed-off-by: Bernd Finger <[email protected]>
collection: Fix ansible-test sanity errors
Avoids the violation of the 'var-naming[no-role-prefix]' ansible-lint
rule.

Relates to #658.

Signed-off-by: Bernd Finger <[email protected]>
sap_anydb_install_oracle: Fix temp removal task
fix(ansible-test-sanity): update workflow to run in every pr
…ion steps

Examples for using tags: See README.md.

Relates to: #342.

Signed-off-by: Bernd Finger <[email protected]>
Relates to issue #342.

Signed-off-by: Bernd Finger <[email protected]>
relates to issue #342

Signed-off-by: Bernd Finger <[email protected]>
berndfinger and others added 23 commits June 4, 2024 13:46
sap_install_media_detect: Allow detection of SAPEXE only
sap_ha_pacemaker_cluster: ASCS ERS Simple Mount
- We use sap_hana_install_shared_path in favor of sap_hana_install_install_path
- If present (e.g. in playbooks or inventories), we use the first directory component
  of sap_hana_install_install_path for sap_hana_install_root_path and we assign
  sap_hana_install_install_path to sap_hana_install_shared_path.

Examples:
If sap_hana_install_install_path is defined as '/hana_01/shared', the following variables
will be set:

sap_hana_install_root_path.: '/hana_01'
sap_hana_install_shared_path: '/hana_01/shared'

If sap_hana_install_install_path is not defined, the following variables will be set:

sap_hana_install_root_path.: '/hana'
sap_hana_install_shared_path: '/hana/shared'

Signed-off-by: Bernd Finger <[email protected]>
sap_ha_pacemaker_cluster: Add python3-pip and NFS fix for Azure
sap_hana_install: Add support for fapolicyd
All the fapolicyd tasks are now inside blocks which are only active when
sap_hana_install_use_fapolicyd is true.

Signed-off-by: Bernd Finger <[email protected]>
Also fix some fapolicyd task names.

Signed-off-by: Bernd Finger <[email protected]>
... for installing packages, except for RHEL 8.1.

Solves issue #755.

Signed-off-by: Bernd Finger <[email protected]>
…module-except-rh81

sap_general_preconfigure: Use the package module in most cases
Solves issue #767.

Signed-off-by: Bernd Finger <[email protected]>
sap_ha_pacemaker_cluster: fix pcs resource restart
…or-workflow-files

Workflows: Add missing '---'
Copy link
Member

@sean-freeman sean-freeman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

Copy link
Contributor

@marcelmamula marcelmamula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ja9fuchs ja9fuchs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested 2 main setups and 👍

@berndfinger berndfinger merged commit c1b479e into main Jun 24, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants