Skip to content

fix(#2810): PSE privileges are TRUST ADMIN + CERTIFICATE ADMIN, not CREATE PSE - #2847

Merged
jung-thomas merged 1 commit into
DEVfrom
fix/2810-pse-trust-admin
Oct 19, 2026
Merged

jung-thomas merged 1 commit into
DEVfrom
fix/2810-pse-trust-admin

Conversation

@jung-thomas

Copy link
Copy Markdown
Contributor

Follow-up to #2846. The default_access_role.hdbrole mechanism was correct, but the privilege name was wrong. Second DEV deploy (tutorials-db-deployer, run 20261019-115443) failed:

com.sap.hana.di.role: "CREATE PSE": invalid system privilege

CREATE PSE is the SQL statement; it is not a system-privilege name. The system privilege that authorizes CREATE PSE is TRUST ADMIN, and ALTER PSE … SET OWN CERTIFICATE needs CERTIFICATE ADMIN.

Change

  • db/src/defaults/default_access_role.hdbrole → system_privileges: ["TRUST ADMIN", "CERTIFICATE ADMIN"]
  • runbook B1 corrected; both prior failed attempts recorded; verify query + tier table updated.

The HDI special-case path (src/defaults/default_access_role.hdbrole auto-granted by #OO) is unchanged and confirmed working — the artifact deployed and reached the grant; only the privilege names were rejected.

Verify after deploy

SELECT PRIVILEGE FROM EFFECTIVE_PRIVILEGES
  WHERE USER_NAME = CURRENT_USER AND PRIVILEGE IN ('TRUST ADMIN','CERTIFICATE ADMIN');  -- expect 2 rows

Then B2 (PSE + remote source) and B3 (VECTOR_EMBEDDING via TUTORIALS_AICORE).

Note: DEV main/DEV currently carries the broken ["CREATE PSE"] role from #2846 and will fail every db-deployer run until this merges.

Second DEV deploy (20261019-115443) failed:
  com.sap.hana.di.role: "CREATE PSE": invalid system privilege

CREATE PSE is the SQL STATEMENT, not a system-privilege name. The
privilege that authorizes CREATE PSE is TRUST ADMIN; ALTER PSE ... SET
OWN CERTIFICATE needs CERTIFICATE ADMIN. The default_access_role
mechanism itself was correct (the artifact deployed and reached the
grant) — only the privilege names were wrong.

- default_access_role.hdbrole: system_privileges now
  [TRUST ADMIN, CERTIFICATE ADMIN].
- runbook: B1 corrected; both failed attempts recorded; verify query and
  tier table updated.
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.

1 participant