Skip to content
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

sap_swpm: add functionality to set specific DDIC and SAP* passwords during new installation #921

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

Conversation

rob0d
Copy link
Contributor

@rob0d rob0d commented Jan 3, 2025

This is a replacement for PR #874 as I had to rebase and redo the changes due to too many changes in dev branch since I did the original PR.

marcelmamula pushed a commit to marcelmamula/community.sap_install that referenced this pull request Jan 6, 2025
Fixes sap-linuxlab#921.

The SAP Solution Manager JAVA Export file (e.g. 51054655_4.zip) contains files with the search pattern
for 'sap_export_nwas_java' before files with the search pattern for 'sap_export_solman_java'. So we
fall through (= do no longer exit) for the search pattern of the SAP Netweaver Java files (e.g. 51055106.zip).
This will work as long as such files do not have content which are matched by any other search pattern.

Tested with:
51050829_3 (NW 7.5 Installation Export)
51054655_1 (SAP Solution Manager 7.2 SR2 Installation Export I)
51054655_2 (SAP Solution Manager 7.2 SR2 Installation Export II)
51054655_4 (SAP Solution Manager 7.2 SR2 - Java)
51055106   (SAP Netweaver 7.5 SP22 Java)

$ sapfile *.zip
51050829_3.zip:                          sap_export_nwas_abap
51054655_1.zip:                          sap_export_solman_abap
51054655_2.zip:                          sap_export_solman_abap
51054655_4.zip:                          sap_export_solman_java
51055106.zip:                            sap_export_nwas_java

Signed-off-by: Bernd Finger <[email protected]>
@sean-freeman
Copy link
Member

@rob0d Accidental removal of keyword "backup from line Restore - DDIC 000 password from backup. Otherwise this is lgtm 👍

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.

2 participants