Skip to content

Add SAP System-level Function Support with Asynchronous Status Tracki… #37

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

nbttmbrg
Copy link

This PR adds support for SAP System-level functions (StartSystem, StopSystem, RestartSystem, and UpdateSystem) to the sap_control role. These functions operate across all instances of an SAP system rather than on individual components.

Should fix #36 and #30

Any comment and advices are very welcome and I can rework things if needed.
This has been tested on Linux.

Changes

  • (defaults/main.yml) Added new functions :
restartsystem_all_nw
updatesystem_all_nw
startsystem_all_nw
stopsystem_all_nw
  • (defaults/main.yml) Added new dictionary sap_control__waitforasync that provides the necessary elements for task file sapcontrol_async.yml to handle correctly the polling
  • prepare.yml will now detect when the function name ends with system and include the according task file
  • system function only run on the ASCS or SCS, ensuring that they'll run on each system in the limit but only once per system

…ng (#1)

* Adding support for system wide functions
* Nested Jinja makes the use of dynamic variables inside sapcontrol functions complicated. Falling back to declarative commands.
* Adding steps to make sure that the system is started before attempting RKS
* Enhance debug output for SAP Control parameters
---------

Signed-off-by: Nicolas Bettembourg <nbettembourg@oxya.com>
Co-authored-by: Rainer Leber <39616583+rainerleber@users.noreply.github.com>
@nbttmbrg
Copy link
Author

I should add : I modified the Readme but not the picture. Don't know if that's mandatory.

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