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

The right kubectl is missing on old kubernetes [rc1] #974

Closed
kravciak opened this issue Jan 22, 2025 · 5 comments
Closed

The right kubectl is missing on old kubernetes [rc1] #974

kravciak opened this issue Jan 22, 2025 · 5 comments

Comments

@kravciak
Copy link
Contributor

kravciak commented Jan 22, 2025

Job kubewarden-controller-delete-controller-job fails with kubectl missing error

To reproduce with e2e tests:
make clean cluster install opentelemetry-remote-collector-tests.bats K3S=1.25 VERSION=next

Error when running e2e

~ k get pods -n kubewarden
my-collector-collector-7d4986f75-9ktbb              1/1     Running   0             2m4s
kubewarden-controller-759f6d4847-sgqvd              1/1     Running   0             2m1s
policy-server-default-75479dbb6-pf54m               1/1     Running   0             94s
kubewarden-controller-delete-controller-job-fwmj7   0/1     Error     4 (57s ago)   109s

~ k logs  -n kubewarden kubewarden-controller-delete-controller-job-fwmj7
F0122 15:49:36.491508       1 main.go:81] the right kubectl is missing, binary downloads from kubernetes' upstream mirror are disabled

I also noticed we pull kuberlr-kubectl:v4.0.0 which is meant for rancher v2.11.0+
https://github.com/rancher/kuberlr-kubectl?tab=readme-ov-file#branches-releases-and-rancher

Failed release test: https://github.com/kubewarden/helm-charts/actions/runs/12910158312/job/36003339620

@kravciak kravciak added this to the 1.21 milestone Jan 22, 2025
@kravciak kravciak changed the title The right kubectl is missing on kubernetes 1.25 [1.21-rc1] The right kubectl is missing on kubernetes 1.25 Jan 22, 2025
@kravciak kravciak changed the title [1.21-rc1] The right kubectl is missing on kubernetes 1.25 The right kubectl is missing on kubernetes 1.25 [rc1] Jan 22, 2025
@kravciak
Copy link
Contributor Author

kravciak commented Jan 22, 2025

Seems it's caused by AllowDownload = false in their Dockerfile

Can be tested also with: make clean cluster install uninstall K3S=1.25 VERSION=next
Affected versions: k8s 1.25 - 1.27

@kravciak kravciak changed the title The right kubectl is missing on kubernetes 1.25 [rc1] The right kubectl is missing on old kubernetes [rc1] Jan 22, 2025
@viccuad viccuad moved this to Todo in Kubewarden Jan 23, 2025
@viccuad viccuad self-assigned this Jan 23, 2025
@viccuad viccuad moved this from Todo to In Progress in Kubewarden Jan 23, 2025
@viccuad
Copy link
Member

viccuad commented Jan 23, 2025

Opened rancher/kuberlr-kubectl#42.

@viccuad viccuad moved this from In Progress to Blocked in Kubewarden Jan 23, 2025
@viccuad
Copy link
Member

viccuad commented Jan 23, 2025

Blocked on that PR; if it doesn't move forward I will either fork or go back to our image for this release at least.

viccuad added a commit to viccuad/helm-charts that referenced this issue Jan 23, 2025
viccuad added a commit to viccuad/helm-charts that referenced this issue Jan 23, 2025
@flavio
Copy link
Member

flavio commented Jan 24, 2025

The bug has been fixed by doing this revert. We can close the issue

@flavio flavio closed this as completed Jan 24, 2025
@github-project-automation github-project-automation bot moved this from Blocked to Done in Kubewarden Jan 24, 2025
@viccuad
Copy link
Member

viccuad commented Jan 24, 2025

The follow-up of this is flavio/kuberlr#153.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants