-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Check if Docker installed via Snap Package Manager #11088
Conversation
Can one of the admins verify this patch? |
/ok-to-test |
kvm2 Driver Times for minikube ingress: 36.3s 34.2s 37.4s 34.4s 34.3s Times for minikube (PR 11088) start: 52.4s 47.1s 46.9s 47.3s 49.7s Times for minikube (PR 11088) ingress: 36.3s 37.3s 33.8s 34.2s 36.7s Averages Time Per Log
docker Driver Times for minikube ingress: 30.0s 29.0s 37.0s 29.5s 29.0s Times for minikube (PR 11088) ingress: 29.5s 28.0s 29.5s 31.5s 30.5s Times for minikube (PR 11088) start: 21.8s 21.4s 21.4s 22.0s 21.7s Averages Time Per Log
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: medyagh, spowelljr The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
kvm2 Driver
Times for minikube start: 49.7s 48.0s 49.2s 48.4s 49.6s Times for minikube ingress: 43.3s 43.3s 42.9s 43.9s 43.8s docker Driver
Times for minikube start: 24.2s 22.5s 21.4s 21.2s 22.0s Times for minikube ingress: 36.0s 38.0s 33.5s 37.0s 37.5s |
Closes #10772
The root of the issue is that packages installed via Snap have their own tmp folder and can't actually access
/tmp
. minikube writes a memory asset to the/tmp
folder that Docker later copies, but if installed via Snap fails due to having different tmp dirs. We added code to check if minikube is installed via Snap, and if it is, we write the memory asset to the users home dir instead.However, the issue is if Docker is installed via Snap and minikube is not, because we only check if minikube is installed via Snap, we write the memory asset to the
/tmp
dir, but Docker can't access/tmp
so it fails.I added a check to see if Docker is installed via Snap, and if minikube or Docker is installed via Snap, writes the memory asset to the users home dir instead.