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

Bump component versions #4981

Merged
merged 6 commits into from
Apr 3, 2025
Merged

Bump component versions #4981

merged 6 commits into from
Apr 3, 2025

Conversation

petrutlucian94
Copy link
Contributor

@petrutlucian94 petrutlucian94 commented Mar 28, 2025

We'll bump the following versions:

  • CNI plugins 1.6.0 -> 1.6.2
  • runc 1.1.15 -> 1.2.6
  • containerd 1.6.36 -> 1.7.27
  • helm 3.16.3 -> 3.17.2
  • calico 3.28.1 -> v3.29.3

Note that we're dropping custom patch disable warnings for kubeconfig permissions as it's no longer needed since this check was also dropped upstream: helm/helm@49cb14a

@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch from 24a7b63 to 93c894b Compare March 28, 2025 13:46
@petrutlucian94 petrutlucian94 added the run-all-tests Run all the CI tests. label Apr 1, 2025
@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch from 942bf2a to b93967b Compare April 1, 2025 08:37
@petrutlucian94
Copy link
Contributor Author

petrutlucian94 commented Apr 1, 2025

There's a microk8s community addon that seems to be abandoned. Their registry is down (so is their website and there is no GH activity), which is why the microk8s CI fails:

+ /snap/microk8s/x1/microk8s-helm3.wrapper repo add sosivio https://helm.sosiv.io
Error: looks like "https://helm.sosiv.io" is not a valid chart repository or cannot be reached: Get "https://helm.sosiv.io/index.yaml"

This PR removes it: canonical/microk8s-community-addons#255

@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch from f374c02 to f392216 Compare April 1, 2025 13:44
@petrutlucian94
Copy link
Contributor Author

petrutlucian94 commented Apr 1, 2025

There's something wrong with Github Actions, they keep getting canceled and all we have is this generic error:

GitHub Actions has encountered an internal error when running your job.

I'll issue another retry later.

LE: nvm, it's caused by the runner image (20.04), which is becoming unsupported. Need to switch to a newer image.

This is a scheduled Ubuntu 20.04 brownout. Ubuntu 20.04 LTS runner will be removed on 2025-04-15. For more details, see https://github.com/actions/runner-images/issues/11101

@petrutlucian94
Copy link
Contributor Author

Test core addons upgrade seems to hang while validating the ingress feature after bumping the runner image to 22.04. This needs further investigation.

@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch 4 times, most recently from dd4094d to ed8deea Compare April 2, 2025 09:52
@petrutlucian94
Copy link
Contributor Author

We're getting a 504 from microbot, need to use tmate to see what's happening.

DEBUG    urllib3.connectionpool:connectionpool.py:228 Starting new HTTP connection (1): microbot.127.0.0.1.nip.io:80
DEBUG    urllib3.connectionpool:connectionpool.py:453 [http://microbot.127.0.0.1.nip.io:80](http://microbot.127.0.0.1.nip.io/) "GET / HTTP/1.1" 504 160
INFO     validators:validators.py:110 GET http://microbot.127.0.0.1.nip.io/ -> <Response [504]>

@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch 4 times, most recently from 980075e to e60d9da Compare April 2, 2025 13:27
@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch from a58f07e to 06ddab5 Compare April 2, 2025 15:35
Copy link
Contributor

@louiseschmidtgen louiseschmidtgen left a comment

Choose a reason for hiding this comment

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

Nice work Lucian!

Could we please break this PR up in two? One to update the workflow and then the other to bump the component version?

@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch from 06ddab5 to 77455a2 Compare April 3, 2025 07:51
@petrutlucian94
Copy link
Contributor Author

Moved the CI fixes to a separate PR, which will have to merge first: #5008

We'll bump the following versions:

* CNI plugins 1.6.0 -> 1.6.2
* runc 1.1.15 -> 1.2.6
* containerd 1.6.36 -> 1.7.27
* helm 3.16.3 -> 3.17.2
* calico 3.28.1 -> v3.29.3
The perm checks were dropped upstream as well.

helm/helm@49cb14a
The linter throws an error while parsing the calico yaml:
./upgrade-scripts/000-switch-to-calico/resources/calico.yaml:5618:
  NotIn ==> noting, not in, notion

We'll define an exception for the "NotIn" word.
@petrutlucian94 petrutlucian94 force-pushed the lpetrut/bump-versions branch from 77455a2 to b7dce6e Compare April 3, 2025 13:10
Copy link
Contributor

@louiseschmidtgen louiseschmidtgen left a comment

Choose a reason for hiding this comment

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

LGTM, please update the PR description and remove the workflow related bits that we moved to the separate PR.

@petrutlucian94 petrutlucian94 merged commit 28fa66c into master Apr 3, 2025
12 checks passed
@petrutlucian94 petrutlucian94 deleted the lpetrut/bump-versions branch April 3, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-all-tests Run all the CI tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants