sap_ha_pacemaker_cluster: JAVA HA scenarios and complete refactor#882
Merged
Conversation
Added SCS/ERS Classic and Simple Mount Refactor of nwas_abap to work with java Refactor of private variables Backwards compatibility for nwas_abap
ja9fuchs
reviewed
Nov 20, 2024
Contributor
ja9fuchs
left a comment
There was a problem hiding this comment.
Just for visibility: the review is ongoing.
ja9fuchs
approved these changes
Nov 26, 2024
Contributor
ja9fuchs
left a comment
There was a problem hiding this comment.
Tested installation and compared results:
- HANA cluster using deprecated vars
- ASCS/ERS cluster using deprecated vars
- ASCS/ERS cluster using new vars with custom values
No surprises, everything looked fine. 👍
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Changes
Added SCS/ERS ENSA1 and ENSA2 Classic and Simple Mount (same code as ASCS since there are no differences in config itself)
Refactor of all variables with
*nwas_abap_*to*nwas_*to work with java and to make more sense in regards to SAP naming.*nwas_abap_*is retained only as part ofsap_ha_pacemaker_cluster_host_typeRefactor of private variables to remove all calculations from defaults and vars. All of them are moved to include_vars tasks separately so we can do
New asserts to cover mandatory variables, that were mandatory before, but never validated with assert.
Added variables for future proofing PAS/AAS, but not actual cluster setup code.
argument_spec and readme updated with variables that were not listed before (health checks, VIP, etc.)
Added
DEPRECATED_VARIABLES.mdwith list of changed variables and their status.Tested and validated on
Operating system:
Platforms:
Tested scenarios: