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

Upgrade docker driver base image to v0.0.10 #7858

Merged
merged 13 commits into from
Apr 29, 2020

Conversation

priyawadhwa
Copy link

The dockerfile currently doesn't build because it can't find the specified versions of crio and podman to install.

Priya Wadhwa added 2 commits April 22, 2020 14:41
The dockerfile currently doesn't build because it can't find the specified versions of crio and podman to install.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Apr 22, 2020
@priyawadhwa
Copy link
Author

/ok-to-test

@k8s-ci-robot k8s-ci-robot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 22, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: priyawadhwa

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 22, 2020
@minikube-pr-bot
Copy link

kvm2 Driver
docker Driver

@priyawadhwa
Copy link
Author

Failing Tests

TestStartStop/group/containerd - 533.86s

	X failed to start node: startup failed: Wait failed: waiting for apps_running: checking k8s-apps to be running: timed out waiting for the condition

coredns pod:

failed to setup network for sandbox "ef187e8c7fe723cb6292b6b5c5ed43e49b1d073b37e758224516a813c4725368": failed to set bridge addr: could not add IP address to "cni0": permission denied

TestStartStop/group/crio - 1175.54s

start_stop_delete_test.go:138: ***** TestStartStop/group/crio: pod "integration-test=busybox" failed to start within 7m0s: timed out waiting for the condition ****

TestStartStop/group/embed-certs - 724.03s
coredns:

Warning  Unhealthy       7m54s              kubelet, embed-certs-20200422151018-5649  Readiness probe failed: HTTP probe failed with statuscode: 503

@afbjorklund
Copy link
Collaborator

afbjorklund commented Apr 23, 2020

The dockerfile currently doesn't build because it can't find the specified versions of crio and podman to install.

This will happen again, neither Ubuntu nor Kubic makes any guarantees to keep old packages...

If you want to keep "pinning" packages, then we need to create a cache of all the .deb used:

lz4=1.9.1-1
gnupg=2.2.12-1ubuntu3
sudo=1.8.27-1ubuntu4.1
docker.io=19.03.2-0ubuntu1
openssh-server=1:8.0p1-6build1
dnsutils=1:9.11.5.P4+dfsg-5.1ubuntu2.1
libglib2.0-0=2.62.1-1

cri-o-1.17=1.17.3~2

podman=1.9.0~2

Including all implicit dependencies, in case there are version specifics in those as well ?

Instead we should probably either use ranges, or remove the version-release pinning.

hack/images/kicbase.Dockerfile Outdated Show resolved Hide resolved
hack/images/kicbase.Dockerfile Outdated Show resolved Hide resolved
@priyawadhwa
Copy link
Author

@afbjorklund that's a really good point. I'm not sure what the best solution here is -- for now, I've added the --cache-from flag to the docker build command for kic. This way, layers from the current kic image will be reused as much as possible, so unless someone changes the versions of crio/podman manually we should have the same exact layers

This isn't a great solution because if the order of Dockerfile commands changes the cache will probably be invalidated, so I think we should definitely keep thinking about this

@minikube-pr-bot
Copy link

kvm2 Driver
Times for minikube: [65.86279565500001 65.9434595 67.561209688]
Average time for minikube: 66.45582161433335

Times for Minikube (PR 7858): [65.315666595 68.31352301499999 68.43029887]
Average time for Minikube (PR 7858): 67.35316282666666

Averages Time Per Log

+--------------------------------+-----------+--------------------+
|              LOG               | MINIKUBE  | MINIKUBE (PR 7858) |
+--------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on   |  0.064052 |                    |
| Debian 9.11                    |           |                    |
| * Using the kvm2 driver based  |  0.021197 |           0.022735 |
| on existing profile            |           |                    |
| * Starting control plane node  |  0.003225 |           0.004968 |
| minikube in cluster minikube   |           |                    |
| * Creating kvm2 VM (CPUs=2,    | 41.174427 |          41.904245 |
| Memory=3700MB, Disk=20000MB)   |           |                    |
| ...                            |           |                    |
| * Preparing Kubernetes v1.18.0 | 22.653378 |          22.986731 |
| on Docker 19.03.8 ...          |           |                    |
| * Verifying Kubernetes         |  1.397012 |           1.382365 |
| components...                  |           |                    |
| * Enabled addons:              |  1.061827 |           0.906040 |
| default-storageclass,          |           |                    |
| storage-provisioner            |           |                    |
| * Done! kubectl is now         |  0.076995 |           0.078326 |
| configured to use "minikube"   |           |                    |
|                                |  0.003708 |           0.063882 |
+--------------------------------+-----------+--------------------+

docker Driver
Times for minikube: [28.373994216999996 26.799234050000003 25.401658065000003]
Average time for minikube: 26.858295444000003

Times for Minikube (PR 7858): [27.131350208999997 26.741041884999994 35.446674972]
Average time for Minikube (PR 7858): 29.773022355333335

Averages Time Per Log

+----------------------------------------+-----------+--------------------+
|                  LOG                   | MINIKUBE  | MINIKUBE (PR 7858) |
+----------------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on           |  0.079614 |                    |
| Debian 9.11                            |           |                    |
| * Using the docker driver              |  0.002621 |           0.003207 |
| based on existing profile              |           |                    |
| * Starting control plane node          |  0.060316 |           0.064725 |
| minikube in cluster minikube           |           |                    |
| * Creating docker container            |  7.976167 |           7.900005 |
| (CPUs=2, Memory=3700MB) ...            |           |                    |
| * Preparing Kubernetes v1.18.0         |  0.000232 |           0.000223 |
| on Docker 19.03.2 ...                  |           |                    |
|   -                                    | 17.458066 |          17.911402 |
| kubeadm.pod-network-cidr=10.244.0.0/16 |           |                    |
| * Verifying Kubernetes                 |  1.155136 |           1.150069 |
| components...                          |           |                    |
| * Enabled addons:                      |  0.059566 |           0.083717 |
| default-storageclass,                  |           |                    |
| storage-provisioner                    |           |                    |
| * Done! kubectl is now                 |  0.063441 |           0.061706 |
| configured to use "minikube"           |           |                    |
|                                        |  0.003136 |           1.179010 |
+----------------------------------------+-----------+--------------------+

@minikube-pr-bot
Copy link

kvm2 Driver
Times for minikube: [66.6963137 66.99843022499999 67.45004367499999]
Average time for minikube: 67.04826253333333

Times for Minikube (PR 7858): [66.09855355 68.881941169 66.587829424]
Average time for Minikube (PR 7858): 67.18944138100001

Averages Time Per Log

+--------------------------------+-----------+--------------------+
|              LOG               | MINIKUBE  | MINIKUBE (PR 7858) |
+--------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on   |  0.063802 |           0.060724 |
| Debian 9.11                    |           |                    |
| * Using the kvm2 driver based  |  0.023918 |           0.020936 |
| on existing profile            |           |                    |
| * Starting control plane node  |  0.003544 |           0.003165 |
| minikube in cluster minikube   |           |                    |
| * Creating kvm2 VM (CPUs=2,    | 41.435417 |          42.840763 |
| Memory=3700MB, Disk=20000MB)   |           |                    |
| ...                            |           |                    |
| * Preparing Kubernetes v1.18.0 | 23.478080 |          22.215345 |
| on Docker 19.03.8 ...          |           |                    |
| * Verifying Kubernetes         |  1.500151 |           1.495433 |
| components...                  |           |                    |
| * Enabled addons:              |  0.466946 |           0.476072 |
| default-storageclass,          |           |                    |
| storage-provisioner            |           |                    |
| * Done! kubectl is now         |  0.072403 |           0.073202 |
| configured to use "minikube"   |           |                    |
|                                |  0.004001 |           0.003802 |
+--------------------------------+-----------+--------------------+

docker Driver
Times for minikube: [28.452765258 29.129603313 28.201564209]
Average time for minikube: 28.59464426

Times for Minikube (PR 7858): [26.344396718999995 35.891610876 26.795944348999996]
Average time for Minikube (PR 7858): 29.677317314666666

Averages Time Per Log

+----------------------------------------+-----------+--------------------+
|                  LOG                   | MINIKUBE  | MINIKUBE (PR 7858) |
+----------------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on           |  0.080499 |           0.078972 |
| Debian 9.11                            |           |                    |
| * Using the docker driver              |  0.002687 |           0.003393 |
| based on existing profile              |           |                    |
| * Starting control plane node          |  0.061420 |           0.061942 |
| minikube in cluster minikube           |           |                    |
| * Creating docker container            |  7.871868 |           7.853694 |
| (CPUs=2, Memory=3700MB) ...            |           |                    |
| * Preparing Kubernetes v1.18.0         |  0.000264 |           0.000196 |
| on Docker 19.03.2 ...                  |           |                    |
|   -                                    | 18.750044 |          17.971934 |
| kubeadm.pod-network-cidr=10.244.0.0/16 |           |                    |
| * Verifying Kubernetes                 |  1.747583 |           3.619738 |
| components...                          |           |                    |
| * Enabled addons:                      |  0.002816 |           0.003110 |
| default-storageclass,                  |           |                    |
| storage-provisioner                    |           |                    |
| * Done! kubectl is now                 |  0.074180 |           0.079739 |
| configured to use "minikube"           |           |                    |
|                                        |  0.003281 |           0.004597 |
+----------------------------------------+-----------+--------------------+

@minikube-pr-bot
Copy link

kvm2 Driver
Times for minikube: [65.876213851 66.11080409099998 64.33038265100001]
Average time for minikube: 65.439133531

Times for Minikube (PR 7858): [65.06875039799999 64.236856385 66.72048402499999]
Average time for Minikube (PR 7858): 65.34203026933334

Averages Time Per Log

+--------------------------------+-----------+--------------------+
|              LOG               | MINIKUBE  | MINIKUBE (PR 7858) |
+--------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on   |  0.061451 |           0.060305 |
| Debian 9.11                    |           |                    |
| * Using the kvm2 driver based  |  0.022346 |           0.020459 |
| on existing profile            |           |                    |
| * Starting control plane node  |  0.003607 |           0.003341 |
| minikube in cluster minikube   |           |                    |
| * Creating kvm2 VM (CPUs=2,    | 40.854838 |          40.624484 |
| Memory=3700MB, Disk=20000MB)   |           |                    |
| ...                            |           |                    |
| * Preparing Kubernetes v1.18.0 | 22.204904 |          22.226766 |
| on Docker 19.03.8 ...          |           |                    |
| * Verifying Kubernetes         |  1.762355 |           1.428962 |
| components...                  |           |                    |
| * Enabled addons:              |  0.452408 |           0.892187 |
| default-storageclass,          |           |                    |
| storage-provisioner            |           |                    |
| * Done! kubectl is now         |  0.073518 |           0.081538 |
| configured to use "minikube"   |           |                    |
|                                |  0.003707 |           0.003987 |
+--------------------------------+-----------+--------------------+

docker Driver
Times for minikube: [26.562768672000004 28.957590213 27.556494309999994]
Average time for minikube: 27.69228439833333

Times for Minikube (PR 7858): [27.110970461 26.565326333000005 25.801958952000003]
Average time for Minikube (PR 7858): 26.492751915333333

Averages Time Per Log

+----------------------------------------+-----------+--------------------+
|                  LOG                   | MINIKUBE  | MINIKUBE (PR 7858) |
+----------------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on           |  0.078270 |           0.078346 |
| Debian 9.11                            |           |                    |
| * Using the docker driver              |  0.002651 |           0.002651 |
| based on existing profile              |           |                    |
| * Starting control plane node          |  0.059341 |           0.061122 |
| minikube in cluster minikube           |           |                    |
| * Creating docker container            |  7.832683 |           7.765162 |
| (CPUs=2, Memory=3700MB) ...            |           |                    |
| * Preparing Kubernetes v1.18.0         |  0.000179 |           0.000248 |
| on Docker 19.03.2 ...                  |           |                    |
|   -                                    | 18.620701 |          17.398715 |
| kubeadm.pod-network-cidr=10.244.0.0/16 |           |                    |
| * Verifying Kubernetes                 |  0.916196 |           1.113475 |
| components...                          |           |                    |
| * Enabled addons:                      |  0.110368 |           0.003196 |
| default-storageclass,                  |           |                    |
| storage-provisioner                    |           |                    |
| * Done! kubectl is now                 |  0.068660 |           0.066108 |
| configured to use "minikube"           |           |                    |
|                                        |  0.003236 |           0.003730 |
+----------------------------------------+-----------+--------------------+

@minikube-pr-bot
Copy link

kvm2 Driver
Times for minikube: [70.873346909 65.540911159 64.745231539]
Average time for minikube: 67.05316320233332

Times for Minikube (PR 7858): [65.92526135600002 66.66427920199999 69.339560821]
Average time for Minikube (PR 7858): 67.30970045966667

Averages Time Per Log

+--------------------------------+-----------+--------------------+
|              LOG               | MINIKUBE  | MINIKUBE (PR 7858) |
+--------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on   |  0.061947 |           0.060851 |
| Debian 9.11                    |           |                    |
| * Using the kvm2 driver based  |  0.021586 |           0.023386 |
| on existing profile            |           |                    |
| * Starting control plane node  |  0.012415 |           0.003117 |
| minikube in cluster minikube   |           |                    |
| * Creating kvm2 VM (CPUs=2,    | 42.302204 |          41.412457 |
| Memory=3700MB, Disk=20000MB)   |           |                    |
| ...                            |           |                    |
| * Preparing Kubernetes v1.18.0 | 22.240389 |          23.573089 |
| on Docker 19.03.8 ...          |           |                    |
| * Verifying Kubernetes         |  1.538237 |           1.445672 |
| components...                  |           |                    |
| * Enabled addons:              |  0.789343 |           0.683769 |
| default-storageclass,          |           |                    |
| storage-provisioner            |           |                    |
| * Done! kubectl is now         |  0.083492 |           0.102273 |
| configured to use "minikube"   |           |                    |
|                                |  0.003549 |           0.005087 |
+--------------------------------+-----------+--------------------+

docker Driver
Times for minikube: [26.172229208999997 25.329741194000004 27.880618539999997]
Average time for minikube: 26.460862981000002

Times for Minikube (PR 7858): [26.693685580000004 26.903694577 26.982727319]
Average time for Minikube (PR 7858): 26.860035825333338

Averages Time Per Log

+----------------------------------------+-----------+--------------------+
|                  LOG                   | MINIKUBE  | MINIKUBE (PR 7858) |
+----------------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on           |  0.074041 |           0.077370 |
| Debian 9.11                            |           |                    |
| * Using the docker driver              |  0.002655 |           0.002574 |
| based on existing profile              |           |                    |
| * Starting control plane node          |  0.061512 |           0.059942 |
| minikube in cluster minikube           |           |                    |
| * Creating docker container            |  7.768590 |           7.802309 |
| (CPUs=2, Memory=3700MB) ...            |           |                    |
| * Preparing Kubernetes v1.18.0         |  0.000234 |           0.000185 |
| on Docker 19.03.2 ...                  |           |                    |
|   -                                    | 17.573147 |          17.880076 |
| kubeadm.pod-network-cidr=10.244.0.0/16 |           |                    |
| * Verifying Kubernetes                 |  0.814227 |           0.911994 |
| components...                          |           |                    |
| * Enabled addons:                      |  0.093550 |           0.050743 |
| default-storageclass,                  |           |                    |
| storage-provisioner                    |           |                    |
| * Done! kubectl is now                 |  0.069968 |           0.068185 |
| configured to use "minikube"           |           |                    |
|                                        |  0.002940 |           0.006658 |
+----------------------------------------+-----------+--------------------+

@afbjorklund
Copy link
Collaborator

@priyawadhwa @medyagh : we need to add the "varlink" package to the kicbase, for podman-env

Whatever version is available from the Kubic repository is fine, so just add it after "podman" ?

@priyawadhwa priyawadhwa added this to the v1.10.0 milestone Apr 27, 2020
@priyawadhwa
Copy link
Author

@afbjorklund thanks for the heads up, added.

@codecov-io
Copy link

Codecov Report

Merging #7858 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7858      +/-   ##
==========================================
- Coverage   35.60%   35.50%   -0.10%     
==========================================
  Files         148      148              
  Lines        9305     9330      +25     
==========================================
  Hits         3313     3313              
- Misses       5595     5620      +25     
  Partials      397      397              
Impacted Files Coverage Δ
cmd/minikube/cmd/config/prompt.go 13.41% <0.00%> (-2.08%) ⬇️
cmd/minikube/cmd/config/configure.go 1.44% <0.00%> (-0.17%) ⬇️
pkg/minikube/out/style.go 91.66% <0.00%> (ø)
cmd/minikube/cmd/service.go 10.44% <0.00%> (ø)

@minikube-pr-bot
Copy link

kvm2 Driver
Times for minikube: [66.756245557 64.442119741 66.726366556]
Average time for minikube: 65.97491061800001

Times for Minikube (PR 7858): [64.53194879499999 64.5584059 64.082664652]
Average time for Minikube (PR 7858): 64.391006449

Averages Time Per Log

+--------------------------------+-----------+--------------------+
|              LOG               | MINIKUBE  | MINIKUBE (PR 7858) |
+--------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on   |  0.061997 |           0.062116 |
| Debian 9.11                    |           |                    |
| * Using the kvm2 driver based  |  0.022733 |           0.019791 |
| on existing profile            |           |                    |
| * Starting control plane node  |  0.003165 |           0.003263 |
| minikube in cluster minikube   |           |                    |
| * Creating kvm2 VM (CPUs=2,    | 40.746157 |          40.591287 |
| Memory=3700MB, Disk=20000MB)   |           |                    |
| ...                            |           |                    |
| * Preparing Kubernetes v1.18.0 | 22.754661 |          21.368118 |
| on Docker 19.03.8 ...          |           |                    |
| * Verifying Kubernetes         |  1.586171 |           1.333179 |
| components...                  |           |                    |
| * Enabled addons:              |  0.705779 |           0.935628 |
| default-storageclass,          |           |                    |
| storage-provisioner            |           |                    |
| * Done! kubectl is now         |  0.090307 |           0.072831 |
| configured to use "minikube"   |           |                    |
|                                |  0.003941 |           0.004792 |
+--------------------------------+-----------+--------------------+

docker Driver
Times for minikube: [25.90725547 24.754539547000004 25.36849174]
Average time for minikube: 25.343428919

Times for Minikube (PR 7858): [31.001312109 26.130894519 27.839061300999997]
Average time for Minikube (PR 7858): 28.323755976333334

Averages Time Per Log

+----------------------------------------+-----------+--------------------+
|                  LOG                   | MINIKUBE  | MINIKUBE (PR 7858) |
+----------------------------------------+-----------+--------------------+
| * minikube v1.10.0-beta.1 on           |  0.081622 |           0.081269 |
| Debian 9.11                            |           |                    |
| * Using the docker driver              |  0.003411 |           0.003432 |
| based on existing profile              |           |                    |
| * Starting control plane node          |  0.060164 |           0.060155 |
| minikube in cluster minikube           |           |                    |
| * Creating docker container            |  7.619904 |           8.193053 |
| (CPUs=2, Memory=3700MB) ...            |           |                    |
| * Preparing Kubernetes v1.18.0         |  0.000225 |           0.000207 |
| on Docker 19.03.2 ...                  |           |                    |
|   -                                    | 16.572537 |          18.640506 |
| kubeadm.pod-network-cidr=10.244.0.0/16 |           |                    |
| * Verifying Kubernetes                 |  0.926694 |           1.271098 |
| components...                          |           |                    |
| * Enabled addons:                      |  0.002893 |           0.002640 |
| default-storageclass,                  |           |                    |
| storage-provisioner                    |           |                    |
| * Done! kubectl is now                 |  0.070983 |           0.068161 |
| configured to use "minikube"           |           |                    |
|                                        |  0.004996 |           0.003234 |
+----------------------------------------+-----------+--------------------+

@tstromberg tstromberg merged commit 2f78bff into kubernetes:master Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants