Skip to content

fix(e2e): make postgres fixture compatible with OpenShift#2002

Merged
TaylorMutch merged 1 commit into
NVIDIA:mainfrom
sjenning:fix/postgres-e2e-openshift
Jun 25, 2026
Merged

fix(e2e): make postgres fixture compatible with OpenShift#2002
TaylorMutch merged 1 commit into
NVIDIA:mainfrom
sjenning:fix/postgres-e2e-openshift

Conversation

@sjenning

Copy link
Copy Markdown
Contributor

Summary

Make the e2e PostgreSQL deployment fixture compatible with OpenShift by addressing UID-based filesystem permission issues that occur when OpenShift assigns random UIDs to pods.

Changes

  • Add PGDATA env var pointing to a subdirectory (/var/lib/postgresql/data/pgdata) so postgres creates and owns the directory it needs to chmod, bypassing the UID mismatch
  • Add uri field to the postgres secret for connection string convenience
  • Add postgres-data and postgres-run emptyDir volume mounts for proper filesystem isolation

Testing

  • mise run pre-commit passes (rust lint, format, license headers all green)
  • E2E tests verified on OpenShift cluster

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Seth Jennings <sjenning@redhat.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@TaylorMutch

Copy link
Copy Markdown
Collaborator

/ok to test 72ce6c2

@TaylorMutch TaylorMutch enabled auto-merge (squash) June 25, 2026 14:08
@TaylorMutch TaylorMutch merged commit c636e70 into NVIDIA:main Jun 25, 2026
30 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.

2 participants