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

Add runtimeClass support to modelmesh deployment #531

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bpradipt
Copy link

This allows the deployment to use different container runtimes or configuration exposed by K8s runtimeClass.

Motivation

Many Kubernetes (K8s) clusters uses different container runtimes for security and isolation reasons. For example in some environment the pods will be executed using Kata containers runtime to take advantage of the additional isolation provided by Kata or to leverage confidential computing technologies.
Further, a cluster may have different configurations for the default container runtime. Kubernetes provides RuntimeClass to select a different container runtime or configuration.
ModelMesh currently doesn't provide native capability to execute the serving runtime pods using different container runtimes. This PR is to add relevant support in ModelMesh for K8s RuntimeClass to enable serving runtime pods to be executed using different container runtimes or configuration for the default container runtime.

Modifications

The modifications are in the config and modelmesh deployment transformation to add the runtimeClassName to the deployment spec.

Result

The model serving runtime deployments will be using the runtimeClassName configured in the configMap (model-serving-config-defaults)

This allows the deployment to use different container runtimes
or configuration exposed by K8s runtimeClass.

Signed-off-by: Pradipta Banerjee <[email protected]>
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.

1 participant