Skip to content

Enhance AzureDataParser to support additional mount point formats #145

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.

Sign up for GitHub

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

Merged
merged 13 commits into from
Oct 28, 2025
Merged

Enhance AzureDataParser to support additional mount point formats #145

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
495411a
Enhance AzureDataParser to support additional mount point formats and…
devanshjainms Oct 27, 2025
5e11170
Update disk performance thresholds for HANA data and log checks
devanshjainms Oct 27, 2025
79ff8b2
Refactor command execution logic in CommandCollector and update DB2 c…
devanshjainms Oct 27, 2025
1ee7bd5
Update DB2 checks for kernel parameters and adjust user roles for com…
devanshjainms Oct 28, 2025
8f8b6aa
Update DB2 command validation logic and adjust kernel panic validator…
devanshjainms Oct 28, 2025
a12d7fb
Add no_log option to enhance security and reduce sensitive data expos…
devanshjainms Oct 28, 2025
2adff95
Add no_log option to suppress sensitive data exposure when loading VM…
devanshjainms Oct 28, 2025
26a2b6f
Update logging and variable names in configuration checks for clarity…
devanshjainms Oct 28, 2025
8ddf10d
Refactor NFS IP address handling to clarify storage account extractio…
devanshjainms Oct 28, 2025
f2d5706
Enhance NFS correlation logic to include FQDN and account-based match…
devanshjainms Oct 28, 2025
8fa7b9d
Fix NFS address matching logic to use source variable for storage acc…
devanshjainms Oct 28, 2025
311b3b4
Validate user parameter in CommandCollector to prevent invalid input
devanshjainms Oct 28, 2025
1825fdc
Refactor NFS matching logic to simplify FQDN checks and improve reada…
devanshjainms Oct 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/module_utils/collector.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,11 @@ def parse_disks_vars(self, check, context) -> str:

fs_entry = None
for fs in filesystem_data:
if fs.get("target") == mount_point:
if fs.get("target") in (
mount_point,
f"{mount_point}/{context.get('database_sid', '').upper()}",
f"{mount_point}/{context.get('sap_sid', '').upper()}",
):
fs_entry = fs
break

Expand Down
34 changes: 17 additions & 17 deletions src/playbook_00_configuration_checks.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -258,12 +258,12 @@
'check_type': 'vm_info',
'metadata': hostvars[item].vm_info_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
loop: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) +
groups[sap_sid | upper + '_DB']|default([]) +
groups[sap_sid | upper + '_APP']|default([]) +
groups[sap_sid | upper + '_WEB']|default([]) +
groups[sap_sid | upper + '_PAS']|default([]) }}"
groups[sap_sid | upper + '_PAS']|default([])) | unique }}"
when: hostvars[item].vm_info_results is defined

- name: "Collect package info check results"
Expand All @@ -275,12 +275,12 @@
'check_type': 'package_info',
'metadata': hostvars[item].package_info_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
loop: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) +
groups[sap_sid | upper + '_DB']|default([]) +
groups[sap_sid | upper + '_APP']|default([]) +
groups[sap_sid | upper + '_WEB']|default([]) +
groups[sap_sid | upper + '_PAS']|default([]) }}"
groups[sap_sid | upper + '_PAS']|default([])) | unique }}"
when: hostvars[item].package_info_results is defined

- name: "Collect common SAP check results"
Expand All @@ -292,12 +292,12 @@
'check_type': 'common_sap',
'metadata': hostvars[item].common_sap_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
loop: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) +
groups[sap_sid | upper + '_DB']|default([]) +
groups[sap_sid | upper + '_APP']|default([]) +
groups[sap_sid | upper + '_WEB']|default([]) +
groups[sap_sid | upper + '_PAS']|default([]) }}"
groups[sap_sid | upper + '_PAS']|default([])) | unique }}"
when: hostvars[item].common_sap_results is defined

- name: "Collect networking check results"
Expand All @@ -309,12 +309,12 @@
'check_type': 'networking',
'metadata': hostvars[item].networking_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
loop: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) +
groups[sap_sid | upper + '_DB']|default([]) +
groups[sap_sid | upper + '_APP']|default([]) +
groups[sap_sid | upper + '_WEB']|default([]) +
groups[sap_sid | upper + '_PAS']|default([]) }}"
groups[sap_sid | upper + '_PAS']|default([])) | unique }}"
when: hostvars[item].networking_results is defined

- name: "Collect DB (HANA) check results"
Expand Down Expand Up @@ -362,8 +362,8 @@
'check_type': 'ascs_scs',
'metadata': hostvars[item].ascs_scs_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) }}"
loop: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([])) | unique }}"
when: hostvars[item].ascs_scs_results is defined

- name: "Collect SCS/ERS HA configuration check results"
Expand All @@ -375,8 +375,8 @@
'check_type': 'scs_ha_config',
'metadata': hostvars[item].scs_ha_config_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) }}"
loop: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([])) | unique }}"
when: hostvars[item].scs_ha_config_results is defined

- name: "Collect Application Server check results"
Expand All @@ -388,9 +388,9 @@
'check_type': 'app_server',
'metadata': hostvars[item].app_server_results_metadata
| default({})}] }}"
loop: "{{ groups[sap_sid | upper + '_APP']|default([]) +
loop: "{{ (groups[sap_sid | upper + '_APP']|default([]) +
groups[sap_sid | upper + '_PAS']|default([]) +
groups[sap_sid | upper + '_WEB']|default([]) }}"
groups[sap_sid | upper + '_WEB']|default([])) | unique }}"
when: hostvars[item].app_server_results is defined

- name: "Debug execution metadata"
Expand Down Expand Up @@ -427,12 +427,12 @@
if scs_high_availability
| default(false) | bool
else 'N/A' }}"
hostnames: "{{ groups[sap_sid | upper + '_SCS']|default([]) +
hostnames: "{{ (groups[sap_sid | upper + '_SCS']|default([]) +
groups[sap_sid | upper + '_ERS']|default([]) +
groups[sap_sid | upper + '_DB']|default([]) +
groups[sap_sid | upper + '_APP']|default([]) +
groups[sap_sid | upper + '_WEB']|default([]) +
groups[sap_sid | upper + '_PAS']|default([]) }}"
groups[sap_sid | upper + '_PAS']|default([])) | unique }}"
passed_count: "{{ all_results | selectattr('status', 'equalto', 'PASSED') | list | length }}"
error_count: "{{ all_results | selectattr('status', 'equalto', 'FAILED') | list | length }}"
warning_count: "{{ all_results | selectattr('status', 'equalto', 'WARNING') | list | length }}"
Expand All @@ -447,4 +447,4 @@

- name: "Debug the file name of the report generated"
ansible.builtin.debug:
msg: "Report file CONFIG_{{ sap_sid | upper }}_{{ platform | upper }} generated."
msg: "Report file CONFIG_{{ sap_sid | upper }}_{{ platform | upper }}_{{ test_group_invocation_id }} generated."