-
Notifications
You must be signed in to change notification settings - Fork 14
Added support for SAP HANA SR Angi, offline validation of cluster resource parameters, support for SLES and RHEL management server #102
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
Conversation
…r SUSE and REDHAT as management server
…ame dynamically (#95)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request introduces comprehensive improvements to the SAP high availability testing framework by adding support for SAP HANA SR Angi, offline validation capabilities, extended operating system support, and improved modularity. The changes enhance testing flexibility and accuracy across different SAP configurations.
Key changes:
- Added support for SAP HANA SR Angi provider alongside traditional SAPHanaSR
- Introduced offline validation functionality for testing cluster configurations using pre-collected CIB XML files
- Refactored parameter handling to use enums and improve type safety across modules
- Enhanced credential management and documentation for multiple authentication methods
Reviewed Changes
Copilot reviewed 91 out of 93 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
tests/modules/ |
Updated test modules to use enum-based operating system handling and removed deprecated ansible_os_family parameters |
src/roles/misc/tasks/ |
Added new offline validation tasks and updated existing tasks to support HANA SR Angi provider detection |
src/modules/ |
Refactored modules to use centralized enums and removed hardcoded string comparisons for OS families |
src/vars/input-api.yaml |
Added new commands and test cases for offline validation and HANA SR Angi support |
src/playbook_01_ha_offline_tests.yml |
New playbook specifically for offline validation testing scenarios |
DhruvAggarwal98
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG!
hdamecharla
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
dhruvmicrosoft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG!
This pull request introduces several improvements and updates across documentation, workflow configuration, and the main test automation script to enhance SAP high availability testing and validation. The most significant changes include expanded OS and cluster configuration support, new documentation for offline validation, improved credential management, and refactoring of the main test script for better usability and extensibility.
Documentation Improvements:
OFFLINE_VALIDATION.mdguide detailing how to perform offline validation of SAP HANA and Central Services clusters using previously collected CIB XML files, including setup instructions, directory structure, and usage examples.Test Automation Script Enhancements (
scripts/sap_automation_qa.sh):Workflow Configuration Updates:
github/codeql-actionin GitHub workflow files (.github/workflows/codeql.yml,ossf-scoreboard.yml,trivy.yml) to use versionv3.28.20for improved security and compatibility. [1] [2] [3]These changes collectively improve the flexibility, maintainability, and clarity of the SAP automation QA framework, making it easier to validate high availability configurations both online and offline, and to run targeted test scenarios across supported platforms.