rh-aiservices-bu / llm-on-openshift

Resources, demos, recipes,... to work with LLMs on OpenShift with OpenShift AI or Open Data Hub.
Apache License 2.0
74 stars 71 forks source link

Fix yq script for setting the security context in the vectordb-minio … #70

Closed pdettori closed 5 days ago

pdettori commented 1 week ago

Fixes #69

guimou commented 6 days ago

@pdettori So you apply the security context at the container level and not the Pod right? No other change? (sorry, I prefer to check if I read it right :smile:)

pdettori commented 5 days ago

@guimou correct. One yq command is used to set the security context at the container level. I found that an additional yq command is needed to unset the default security context at the pod level because runAsUser and runAsGroup are set to default values outside OpenShift ranges.

guimou commented 5 days ago

@pdettori Thanks a lot for your PR!