Skip to content

Conversation

@berndfinger
Copy link
Member

Fixes #1137. In addition, the tag sap_hana_install_check_hana_exists has been implemented, for checking if an HANA instance with the desired SID and number is already present.

If more than one tag is provided, the role will abort. Allowing more than one tag would have resulted in additional development and testing effort. As the main purpose of these tags is to assist with support and troubleshooting, one tag at a time should be sufficient.

Solves issue sap-linuxlab#1137. Also adds tag sap_hana_install_check_hana_exists.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
Relates to sap-linuxlab#1137

Signed-off-by: Bernd Finger <bfinger@redhat.com>
Relates to sap-linuxlab#1137.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
The tag sap_hana_install_check_installation has been removed in this
commit, for reducing complexity and testing efforts.

Relates to sap-linuxlab#1137.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
Relates to sap-linuxlab#1137.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
sap_hana_install_hdblcm_commandline, from tasks/main.yml.

Signed-off-by: Bernd Finger <bfinger@redhat.com>
Copy link
Contributor

@marcelmamula marcelmamula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ja9fuchs ja9fuchs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, partially tested the tags. As discussed the check-mode of the role needs fixing.

@berndfinger berndfinger merged commit de748dc into sap-linuxlab:dev Dec 11, 2025
18 checks passed
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.

3 participants