Skip to content

sap_swpm: Replace SAP detection fail with warning for wrong GID and UID - #1231

Merged
marcelmamula merged 1 commit into
sap-linuxlab:devfrom
marcelmamula:swpm-uid-gid
Aug 17, 2026
Merged

sap_swpm: Replace SAP detection fail with warning for wrong GID and UID#1231
marcelmamula merged 1 commit into
sap-linuxlab:devfrom
marcelmamula:swpm-uid-gid

Conversation

@marcelmamula

Copy link
Copy Markdown
Contributor

Changes

#1213 brought a lot of improvements including much stricter validations.
One of them was bit too strict as re-testing has showed that sapinst has some leniency when handling existing users and groups.

  • Change fail modules to debug to show WARN instead.

Test results

Summary: Very lenient in accepting IDs from inifile and raises issue only for shared filesystems.

Accepts:

  • Existing users sidadm and sapadm with different IDs.
  • Existing group sapsys with different ID.

Rejects:

  • Existing files under user with different group ID. Example: ASCS with ae1adm GID: 3333 and ERS with ae1adm GID: 3334
    NOTE: This is specific only to distributed systems with shared filesystems!
Group sapsys exists with GID 3334, while the default profile /sapmnt/AE1/profile/DEFAULT.PFL is owned by GID 3333. <br>SOLUTION: Change the GID of group sapsys to 3333 or remove the group account.

HANA Details

  • No changes to strict validation of sap_hana_install, because of its own strict behavior documented in test section below.

Test results

Summary: Very strict validation that allows only wrong UID.

Accepts:

  • Wrong UID with correct GID and correct sidadm password provided in parameter file under password=

Rejects:

  • Wrong UID with wrong GID and correct sidadm password provided in parameter file under password=
The defined user 'h01adm' already exists on the system. Neither the password, nor any other attribute of the user will be changed.
Verify that the user is correct.
Running in batch mode
  Group ID '2001' for 'sapsys' is invalid because 'sapsys' already exists with ID '3335' on the local host
  • Wrong UID with wrong GID and missing sidadm password provided in parameter file under password=
The defined user 'h01adm' already exists on the system. Neither the password, nor any other attribute of the user will be changed.
Verify that the user is correct.
Running in batch mode
  Mandatory parameter 'password' (Password) is missing or invalid

@berndfinger berndfinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@marcelmamula
marcelmamula merged commit df26916 into sap-linuxlab:dev Aug 17, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants