Skip to content

Conversation

rhdedgar
Copy link
Collaborator

Setting FSGroup to convey the need for a more permissive scc.

Closes RHAIENG-999.

@rhdedgar
Copy link
Collaborator Author

It's possible the RunAsUser setting could even be removed in the future, as the FSGroup: 0 setting should provide the necessary volume access.

@derekhiggins
Copy link
Collaborator

So we're dropping some privileges from the init container while mounting storage in as the 'root' group so that containers can always write to volumes?

Would you mind putting a little context on why this is needed in here (as not everybody can see what RHAIENG-999 is )

@rhdedgar
Copy link
Collaborator Author

Sure! So from the ticket, setting storage to something like:

    storage:
      size: 10Gi

Can cause the permissions error:
forbidden: unable to validate against any security context constraint

I was able to replicate that error on an OpenShift 4.18 cluster.

I've updated the FSGroup to match the container's existing RunAsUser and RunAsGroup settings to address that security discrepancy.

I also dropped the additional unneeded capabilities that would normally be given to pods that aren't run as restricted-v2, so that this fix doesn't grant more than it needs to.

Copy link
Collaborator

@derekhiggins derekhiggins left a comment

Choose a reason for hiding this comment

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

Thanks

Signed-off-by: Doug Edgar <[email protected]>
@rhdedgar rhdedgar merged commit c1f55f4 into llamastack:main Sep 18, 2025
6 checks passed
VaishnaviHire pushed a commit to VaishnaviHire/llama-stack-k8s-operator that referenced this pull request Sep 25, 2025
Setting FSGroup to convey the need for a more permissive scc.

Closes RHAIENG-999.

Signed-off-by: Doug Edgar <[email protected]>

Approved by: Derek Higgins <[email protected]>

(cherry picked from commit c1f55f4)
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