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

cmctl version reports only the old CRD version if I upgrade cert-manager without including the CRDs #127

Open
wallrj opened this issue Oct 2, 2024 · 0 comments
Labels
priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.

Comments

@wallrj
Copy link
Member

wallrj commented Oct 2, 2024

I upgraded cert-manager using Helm but forgot to use the crds.enabled value, so the deployments were upgraded but not the CRDs.

helm upgrade -n cert-manager cert-manager cert-manager --repo https://charts.jetstack.io  --devel
Release "cert-manager" has been upgraded. Happy Helming!
NAME: cert-manager
LAST DEPLOYED: Wed Oct  2 13:46:35 2024
NAMESPACE: cert-manager
STATUS: deployed
REVISION: 2
TEST SUITE: None
NOTES:
cert-manager v1.16.0-beta.0 has been deployed successfully!

I was confused by the output of cmctl version, which only reported the old CRD version and did not alert me to the mismatch with the deployment and image versions.

$ cmctl version --short
Client Version: v2.1.1
Server Version: v1.15.3

$ cmctl version -o yaml -v 7
I1002 13:58:57.933697  170011 loader.go:395] Config loaded from file:  /home/richard/.kube/config
I1002 13:58:57.934433  170011 round_trippers.go:463] GET https://8663C9323E9D091F75D47483636E8A5A.gr7.us-west-2.eks.amazonaws.com/apis/apiextensions.k8s.io/v1
I1002 13:58:57.934462  170011 round_trippers.go:469] Request Headers:
I1002 13:58:57.934468  170011 round_trippers.go:473]     User-Agent: cmctl/v0.0.0 (linux/amd64) kubernetes/$Format
I1002 13:58:57.934484  170011 round_trippers.go:473]     Accept: application/json, */*
I1002 13:58:59.203450  170011 round_trippers.go:574] Response Status: 200 OK in 1268 milliseconds
I1002 13:58:59.204064  170011 round_trippers.go:463] GET https://8663C9323E9D091F75D47483636E8A5A.gr7.us-west-2.eks.amazonaws.com/apis/apiextensions.k8s.io/v1/customresourcedefinitions/certificates.cert-manager.io
I1002 13:58:59.204123  170011 round_trippers.go:469] Request Headers:
I1002 13:58:59.204139  170011 round_trippers.go:473]     Accept: application/json, */*
I1002 13:58:59.204146  170011 round_trippers.go:473]     User-Agent: cmctl/v0.0.0 (linux/amd64) kubernetes/$Format
I1002 13:58:59.371767  170011 round_trippers.go:574] Response Status: 200 OK in 167 milliseconds
clientVersion:
  compiler: gc
  gitCommit: 479a118e3dfe3d29d9a5b23abe04ca46327735a1
  gitTreeState: ""
  gitVersion: v2.1.1
  goVersion: go1.23.1
  platform: linux/amd64
serverVersion:
  detected: v1.15.3
  sources:
    crdLabelVersion: v1.15.3
$ kubectl describe pod -n cert-manager | grep v1\.1
                  app.kubernetes.io/version=v1.16.0-beta.0
                  helm.sh/chart=cert-manager-v1.16.0-beta.0
    Image:         quay.io/jetstack/cert-manager-controller:v1.16.0-beta.0
      --acme-http01-solver-image=quay.io/jetstack/cert-manager-acmesolver:v1.16.0-beta.0
  Normal  Pulling    3m46s  kubelet            Pulling image "quay.io/jetstack/cert-manager-controller:v1.16.0-beta.0"
  Normal  Pulled     3m40s  kubelet            Successfully pulled image "quay.io/jetstack/cert-manager-controller:v1.16.0-beta.0" in 5.41s (5.41s including waiting). Image size: 21184860 bytes.
                  app.kubernetes.io/version=v1.16.0-beta.0
                  helm.sh/chart=cert-manager-v1.16.0-beta.0
    Image:         quay.io/jetstack/cert-manager-cainjector:v1.16.0-beta.0
  Normal  Pulling    3m46s  kubelet            Pulling image "quay.io/jetstack/cert-manager-cainjector:v1.16.0-beta.0"
  Normal  Pulled     3m42s  kubelet            Successfully pulled image "quay.io/jetstack/cert-manager-cainjector:v1.16.0-beta.0" in 3.972s (3.972s including waiting). Image size: 15428294 bytes.
                  app.kubernetes.io/version=v1.16.0-beta.0
                  helm.sh/chart=cert-manager-v1.16.0-beta.0
    Image:         quay.io/jetstack/cert-manager-webhook:v1.16.0-beta.0
  Normal  Pulling    3m45s  kubelet            Pulling image "quay.io/jetstack/cert-manager-webhook:v1.16.0-beta.0"
  Normal  Pulled     3m41s  kubelet            Successfully pulled image "quay.io/jetstack/cert-manager-webhook:v1.16.0-beta.0" in 4.453s (4.454s including waiting). Image size: 18194505 bytes.
@inteon inteon added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Projects
None yet
Development

No branches or pull requests

2 participants