Skip to content

Conversation

wainersm
Copy link
Contributor

@wainersm wainersm commented Jul 4, 2025

There are workflows that rely on az aks install-cli to get kubectl installed. There is a well-known problem on install-cli, related with API usage rate limit, that has recently caused the command to fail quite often.

This is replacing install-cli with the azure/setup-kubectl github action which has no such as rate limit problem.

While here, removed the install_cli() function from gha-run-k8s-common.sh so avoid developers using it by mistake in the future.

Fixes #11463


I've tested this change here:

kubelogin, which is downloaded by install-cli but not setup-kubectl, isn't used in our workflows. At least they didn't fail because of its lack.

setup-kubectl akin to install-cli is going to find and install latest kubectl version.

@wainersm wainersm requested review from sprt and stevenhorsman July 4, 2025 17:48
@wainersm wainersm added the area/ci Issues affecting the continuous integration label Jul 4, 2025
@wainersm wainersm force-pushed the ci_setup_kubectl branch from d210d9e to 72c82ce Compare July 7, 2025 18:28
There are workflows that rely on `az aks install-cli` to get kubectl
installed. There is a well-known problem on install-cli, related with
API usage rate limit, that has recently caused the command to fail
quite often.

This is replacing install-cli with the azure/setup-kubectl github
action which has no such as rate limit problem.

While here, removed the install_cli() function from gha-run-k8s-common.sh
so avoid developers using it by mistake in the future.

Fixes kata-containers#11463
Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
@wainersm wainersm force-pushed the ci_setup_kubectl branch from 72c82ce to e4da3b8 Compare July 8, 2025 18:19
@wainersm
Copy link
Contributor Author

wainersm commented Jul 8, 2025

Updated to pin the version to sha commit. Testing in https://github.com/kata-containers/kata-containers/actions/runs/16151195067

Copy link
Contributor

@sprt sprt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM assuming green tests.

@wainersm
Copy link
Contributor Author

wainersm commented Jul 9, 2025

LGTM assuming green tests.

As far as kubectl installation is concerned, tests are green.

Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @wainersm!

@stevenhorsman
Copy link
Member

The stable tests in https://github.com/kata-containers/kata-containers/actions/runs/16151195067/job/45590114805 seem to be working, so as this is a workflow change I'll use admin rights to merge it.

@stevenhorsman stevenhorsman merged commit 990c4e6 into kata-containers:main Jul 9, 2025
389 of 433 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues affecting the continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci: Intermittent rate limit error when installing kubectl
3 participants