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

v1.23.0-rc.0 breaks crictl images for Docker container-runtime (works fine on 1.23.0-beta.0) #106807

Closed
medyagh opened this issue Dec 3, 2021 · 7 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/node Categorizes an issue or PR as relevant to SIG Node.
Milestone

Comments

@medyagh
Copy link
Member

medyagh commented Dec 3, 2021

What happened?

On minikube HEAD we tried to bump the "newest" Kubernetes version from k8s 1.23.0-beta.0 to k8s v1.23.0-rc.0

however this breaks "crictl images" command

$ minikube ssh
docker@minikube:~$ sudo crictl images
FATA[0000] listing images: rpc error: code = Unimplemented desc = unknown service runtime.v1alpha2.ImageService 
https://github.com/kubernetes/minikube/issues/13077

however if we start minikube with 1.23.0-beta.0 crictl works fine

$ make;mk delete --all;mk start --kubernetes-version=v1.23.0-beta.0
$ mk ssh
docker@minikube:~$ sudo crictl images
IMAGE                                     TAG                 IMAGE ID            SIZE
gcr.io/k8s-minikube/storage-provisioner   v5                  ba04bb24b9575       29MB
k8s.gcr.io/coredns/coredns                v1.8.4              6d3ffc2696ac2       44.4MB
k8s.gcr.io/coredns/coredns                v1.8.6              edaa71f2aee88       46.8MB
k8s.gcr.io/etcd                           3.5.0-0             2252d5eb703b0       364MB
k8s.gcr.io/etcd                           3.5.1-0             1040f7790951c       132MB
k8s.gcr.io/kube-apiserver                 v1.23.0-beta.0      70f3ee870024e       132MB
k8s.gcr.io/kube-controller-manager        v1.23.0-beta.0      ba6fcb43638a3       122MB
k8s.gcr.io/kube-proxy                     v1.23.0-beta.0      e223498b094cf       109MB
k8s.gcr.io/kube-scheduler                 v1.23.0-beta.0      916a7b78e5641       52.8MB
k8s.gcr.io/pause                          3.6                 7d46a07936af9       484kB
kubernetesui/dashboard                    v2.3.1              5bb89698273d8       217MB
kubernetesui/metrics-scraper              v1.0.7              5717d272af6d4       32.5MB

in changelog I see there are API changes https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.23.md#api-change
but I dont see any messages on how the end users would need to adopt to these changes. or the crictl users

I tried with crictl version v1.21.0 and crictl version v1.22.0 and for both I got the same error,

(mind that this happens for Docker container runtime and it works fine if the container runtime is changes for containerd)

What did you expect to happen?

I expected "sudo crictl images" to reuturn list of images insted of error

How can we reproduce it (as minimally and precisely as possible)?

on minikube head

try these two

$ make./out/minikube delete --all;./out/minikube start --kubernetes-version=v1.23.0-beta.0
$ ./ou/minikube ssh sudo crictl images

$ make./out/minikube delete --all;./out/minikube start --kubernetes-version=v1.23.0-rc.0

$ ./ou/minikube ssh sudo crictl images

Anything else we need to know?

No response

Kubernetes version

v1.23.0-rc.0

Cloud provider

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

Related Issue kubernetes/minikube#13077
Related PR kubernetes/minikube#13073

@medyagh medyagh added the kind/bug Categorizes issue or PR as related to a bug. label Dec 3, 2021
@k8s-ci-robot k8s-ci-robot added needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Dec 3, 2021
@k8s-ci-robot
Copy link
Contributor

@medyagh: This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@medyagh
Copy link
Member Author

medyagh commented Dec 3, 2021

@saschagrunert
@ehashman any insight on how we can solve this issue as users of crictl ? I suspect it is related to this PR #106501

@medyagh medyagh changed the title v1.23.0-rc.0 breaks crictl images (works fine on 1.23.0-beta.0) v1.23.0-rc.0 breaks crictl images for Docker container-runtime (works fine on 1.23.0-beta.0) Dec 3, 2021
@liggitt
Copy link
Member

liggitt commented Dec 5, 2021

I think this is a duplicate of #106798, fixed in #106803 and in 1.23.0-rc.1.

Can you verify this is resolved in rc.1 for you?

@liggitt liggitt added this to the v1.23 milestone Dec 5, 2021
@neolit123
Copy link
Member

/sig node

@k8s-ci-robot k8s-ci-robot added sig/node Categorizes an issue or PR as relevant to SIG Node. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Dec 6, 2021
@medyagh
Copy link
Member Author

medyagh commented Dec 6, 2021

I think this is a duplicate of #106798, fixed in #106803 and in 1.23.0-rc.1.

Can you verify this is resolved in rc.1 for you?

thanks it is a dupe of that issue, will reopen if the new patch hasnt fixed it.

It would be amazing if kubernetes releases get tested with minikube on kubernetes side. it has happend before that minikube integration tests found a failure in kubernetes after release

@liggitt
Copy link
Member

liggitt commented Dec 6, 2021

I think this is a duplicate of #106798, fixed in #106803 and in 1.23.0-rc.1.
Can you verify this is resolved in rc.1 for you?

thanks it is a dupe of that issue, will reopen if the new patch hasnt fixed it.

Sounds good, thanks.

/close

@k8s-ci-robot
Copy link
Contributor

@liggitt: Closing this issue.

In response to this:

I think this is a duplicate of #106798, fixed in #106803 and in 1.23.0-rc.1.
Can you verify this is resolved in rc.1 for you?

thanks it is a dupe of that issue, will reopen if the new patch hasnt fixed it.

Sounds good, thanks.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/node Categorizes an issue or PR as relevant to SIG Node.
Projects
None yet
Development

No branches or pull requests

4 participants