Skip to content

collection: Update sample playbooks and remove deprecated variables - #1255

Open
marcelmamula wants to merge 2 commits into
sap-linuxlab:devfrom
marcelmamula:playbook-samples
Open

marcelmamula wants to merge 2 commits into
sap-linuxlab:devfrom
marcelmamula:playbook-samples

Conversation

@marcelmamula

Copy link
Copy Markdown
Contributor

Premise

Sample playbooks contain lot of variables that were deprecated and we should not be advertising them.

Changes

  • Remove non-default modes of sap_swpm and add dedicated playbooks for execution scenarios that will be properly defined in upcoming sap_swpm PR.
  • Create clear separation between roles to show how roles can be run independently.
  • Update README and create clear point towards ansible.playbooks_for_sap as source for end to end playbooks, not here.

NOTE for sap_hana_preconfigure_exec.yml

The playbook sap_hana_preconfigure_exec.yml was removed without any replacement, because its implementation had following issues:

  • Example with ansible.builtin.import_playbook does not work on 2.18 - 2.21 versions
  • Actual purpose is limited as it expects no changes in this file (since it runs from collection home), yet it needs vars to be defined.
    @berndfinger I think having good samples has more value, but maybe you can say if you added it for some specific reason. Then I can re-add it, but only if it is working properly.

@marcelmamula marcelmamula self-assigned this Oct 20, 2026
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.

1 participant