-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Closed
Labels
co/runtime/dockerIssues specific to a docker runtimeIssues specific to a docker runtimekind/featureCategorizes issue or PR as related to a new feature.Categorizes issue or PR as related to a new feature.priority/backlogHigher priority than priority/awaiting-more-evidence.Higher priority than priority/awaiting-more-evidence.
Description
Just like the "dockershim" in Kubernetes, the cruntime uses direct commands for Docker.
https://github.com/kubernetes/minikube/blob/master/pkg/minikube/cruntime/docker.go
It would be nice if it would use the CRI (where possible), like the other runtimes are doing.
https://github.com/kubernetes/minikube/blob/master/pkg/minikube/cruntime/cri.go
We do have the crictl all configured, so just a matter of "flipping a switch" basically.
The initial approach might be to introduce a feature toggle, to make it opt-in to start with...
dmpe and ariescheurwater
Metadata
Metadata
Assignees
Labels
co/runtime/dockerIssues specific to a docker runtimeIssues specific to a docker runtimekind/featureCategorizes issue or PR as related to a new feature.Categorizes issue or PR as related to a new feature.priority/backlogHigher priority than priority/awaiting-more-evidence.Higher priority than priority/awaiting-more-evidence.