Skip to content

Change default group for consistency and openshift compatability#2766

Merged
Alex-Tideman merged 1 commit intotemporalio:mainfrom
ajzawawi:main
Jun 18, 2025
Merged

Change default group for consistency and openshift compatability#2766
Alex-Tideman merged 1 commit intotemporalio:mainfrom
ajzawawi:main

Conversation

@ajzawawi
Copy link
Copy Markdown
Contributor

Description & motivation 💭

This PR updates the Dockerfile used to build the Temporal UI server image to make it compatible with OpenShift's default security constraints. Specifically:

  • Changed the user and group ID from 5000 to 1000, a more common non-root default

These changes:

  • Enable the image to run in OpenShift environments without requiring privileged SCCs or custom UID assignments

  • Make the Dockerfile consistent with other Temporal repositories that follow a similar non-root UID convention for container compatibility

Issue(s) closed

@ajzawawi ajzawawi requested a review from a team as a code owner June 16, 2025 17:30
@ajzawawi ajzawawi requested review from andrewzamojc and removed request for a team June 16, 2025 17:30
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
holocene ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2025 5:31pm

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 16, 2025

CLA assistant check
All committers have signed the CLA.

@ajzawawi
Copy link
Copy Markdown
Contributor Author

ajzawawi commented Jun 16, 2025

@andrewzamojc @rossedfort @Alex-Tideman

Would appreciate a review. This issue is currently blocking me from deploying temporal in Openshift at my org.

@Alex-Tideman Alex-Tideman self-requested a review June 18, 2025 13:37
@Alex-Tideman Alex-Tideman merged commit 2267695 into temporalio:main Jun 18, 2025
13 checks passed
hferentschik pushed a commit to DataDog/temporalio-ui that referenced this pull request Mar 12, 2026
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.

web is unable to create ./config/docker.yaml

3 participants