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

Set group permissions on whispercpp app folder #681

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

johnmcollier
Copy link
Contributor

@johnmcollier johnmcollier commented Jul 17, 2024

This PR updates the whispercpp dockerfile to mark the /app folder as owned by the GID 0, which is the group OpenShift uses.

For some context, our team has been looking at consuming some of the Podman Desktop AI lab recipes via RHDH software templates, which will include OpenShift as a deployment target for the recipes. While trying to run the whispercpp model server, we found it was running into permissions issues when using the UID that OpenShift assigns to it (on the default, restricted SCC). Setting the user ID to 1001 and the group owner on the folder to 0 resolved these issues.

We've verified both locally and on OpenShift, let me know if this change is okay to go in. We also have an image built from this branch up at quay.io/redhat-ai-dev/whispercpp:latest

Signed-off-by: John Collier <[email protected]>
Copy link
Member

@rhatdan rhatdan left a comment

Choose a reason for hiding this comment

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

LGTM

@rhatdan rhatdan merged commit 38092ab into containers:main Jul 22, 2024
7 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