-
Notifications
You must be signed in to change notification settings - Fork 790
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
Bump component versions #4981
Conversation
24a7b63
to
93c894b
Compare
942bf2a
to
b93967b
Compare
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:
This PR removes it: canonical/microk8s-community-addons#255 |
f374c02
to
f392216
Compare
There's something wrong with Github Actions, they keep getting canceled and all we have is this generic error:
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.
|
|
dd4094d
to
ed8deea
Compare
We're getting a 504 from microbot, need to use tmate to see what's happening.
|
980075e
to
e60d9da
Compare
a58f07e
to
06ddab5
Compare
There was a problem hiding this 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?
...ripts/components/helm/patches/default/0001-disable-warnings-for-kubeconfig-permissions.patch
Show resolved
Hide resolved
06ddab5
to
77455a2
Compare
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.
77455a2
to
b7dce6e
Compare
There was a problem hiding this 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.
We'll bump the following versions:
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