Upgrade versions #9798

electrocucaracha · 2023-02-15T19:57:36Z

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test

/kind feature

/kind flake

What this PR does / why we need it:
This change upgrades the version number for Calico, Cilium, Helm, crun, Katacontainers, youki, gvisor and skopeo among others

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:
The youki container runtime has changed the naming of their release assets since v0.0.2.

Does this PR introduce a user-facing change?:

Upgrade many tool versions (Helm, crun, kata, youki, gvisor, skopeo, Calico, Cilium etc...)

JaneLiuL

/lgtm

MrFreezeex

Small comment for flannel but apart from that looks good to me, thank you very much for all the work in this PR and the other split-ed ones 🙏

MrFreezeex · 2023-06-22T18:08:42Z

roles/download/defaults/main/main.yml

@@ -114,13 +114,13 @@ typha_enabled: false
 calico_apiserver_enabled: false

 flannel_version: "v0.22.0"
-flannel_cni_version: "v1.1.2"
+flannel_cni_version: "v1.2.0"


This was intentionally downgraded somewhat recently see this #10205 (comment)

Got it, I'll change it and fix the kubelet_checksums versions

MrFreezeex · 2023-06-22T18:12:07Z

roles/download/defaults/main/checksums.yml

    v1.26.0: 5538c88b8ccde419e6158ab9c06dfcca1fa0abecf33d0a75b2d22ceddd283f0d
    v1.25.0: 1b77d1f198c67b2015104eee6fe7690465b8efa4675ea6b4b958c63d60a487e7
-
+    v1.24.2: 76e37bc9abe6a5f402594010c347d4cd4e69e204e23610bdfacad31305d5c7e2


Ah wait this as well, we do not support 1.24 anymore in master

MrFreezeex · 2023-06-22T18:43:58Z

Thanks for the additional changes 🙏
/lgtm

MrFreezeex · 2023-06-22T20:39:38Z

README.md

@@ -169,7 +169,7 @@ Note: Upstart/SysV init based OS types are not supported.
 - Network Plugin
  - [cni-plugins](https://github.com/containernetworking/plugins) v1.2.0
  - [calico](https://github.com/projectcalico/calico) v3.25.1
-  - [cilium](https://github.com/cilium/cilium) v1.13.3
+  - [cilium](https://github.com/cilium/cilium) v1.14.7


Wait this is wrong 1.14.7 doesn't exist 🤔, is it the cilium cli or something?

You're right, the latest version is 1.13.4

MrFreezeex · 2023-06-22T20:42:42Z

roles/download/defaults/main/checksums.yml

@@ -60,9 +80,13 @@ kubelet_checksums:
    v1.25.2: 995f885543fa61a08bd4f1008ba6d7417a1c45bd2a8e0f70c67a83e53b46eea5
    v1.25.1: 6fe430ad91e1ed50cf5cc396aa204fda0889c36b8a3b84619d633cd9a6a146e2
    v1.25.0: ad45ac3216aa186648fd034dec30a00c1a2d2d1187cab8aae21aa441a13b4faa
+    v1.24.15: 547147c8dd4096d975bb2669f0be500fff6f8fdd0848d25dace3d278ad1e3b88


Dunno if you are not done yet but there's still a bunch of 1.24 version in there 🙈

The number of versions included in the checksums.yml file is controlled by the third argument of the get_version function. For kubeadm and kubelet, the value is set to 25. Should I need to change it to 23?

Ohhh I see I wasn't fully aware it worked like that sorry. So I think this should be more a minimum version for crio / core kubernetes versions than "last x versions". But IMO this could very well be done later on (or not as you wish), I wouldn't want to block this PR that was already left hanging for too long :(.

MrFreezeex · 2023-06-23T08:01:24Z

/lgtm

The following applications have been upgraded: * Cilium * Helm * crun * Katacontainers * youki * gvisor * skopeo * yq Signed-off-by: Victor Morales <chipahuac@hotmail.com>

MrFreezeex · 2023-07-05T10:05:40Z

Heyy could one of you take a look at this PR stuck for quite some time that gets a conflict after most app update 🙏 @yankay @mzaian @oomichi @floryut

/lgtm

mzaian

Thanks @electrocucaracha and @MrFreezeex

k8s-ci-robot · 2023-07-05T10:29:39Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: electrocucaracha, MrFreezeex, mzaian

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

~~OWNERS~~ [mzaian]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

The following applications have been upgraded: * Cilium * Helm * crun * Katacontainers * youki * gvisor * skopeo * yq Signed-off-by: Victor Morales <chipahuac@hotmail.com>

k8s-ci-robot added kind/feature Categorizes issue or PR as related to a new feature. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Feb 15, 2023

k8s-ci-robot requested review from cristicalin and EppO February 15, 2023 19:58

electrocucaracha mentioned this pull request Feb 15, 2023

Improve method to get binary checksums #9782

Merged

JaneLiuL reviewed Feb 16, 2023

View reviewed changes

k8s-ci-robot assigned JaneLiuL Feb 16, 2023

k8s-ci-robot added lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 16, 2023

electrocucaracha force-pushed the update_checksum branch from 27c0702 to 612a5c9 Compare February 20, 2023 16:44

k8s-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 20, 2023

electrocucaracha force-pushed the update_checksum branch from 612a5c9 to b6be761 Compare March 2, 2023 01:46

k8s-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Mar 2, 2023

electrocucaracha force-pushed the update_checksum branch from b6be761 to 8cc9df1 Compare March 6, 2023 18:01

k8s-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Mar 6, 2023

electrocucaracha force-pushed the update_checksum branch from 8cc9df1 to 95eb25c Compare March 8, 2023 04:02

k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 8, 2023

electrocucaracha requested review from JaneLiuL and removed request for EppO and cristicalin March 8, 2023 16:04

k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 9, 2023

electrocucaracha force-pushed the update_checksum branch from 95eb25c to c2482bb Compare March 9, 2023 18:47

k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 9, 2023

electrocucaracha force-pushed the update_checksum branch from c2482bb to 97184c0 Compare March 9, 2023 22:45

k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 15, 2023

electrocucaracha force-pushed the update_checksum branch from 97184c0 to 2d1b566 Compare June 22, 2023 17:49

k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 22, 2023

electrocucaracha force-pushed the update_checksum branch from 2d1b566 to 7596fec Compare June 22, 2023 18:05

MrFreezeex approved these changes Jun 22, 2023

View reviewed changes

MrFreezeex reviewed Jun 22, 2023

View reviewed changes

electrocucaracha force-pushed the update_checksum branch from 7596fec to 68cc3eb Compare June 22, 2023 18:25

k8s-ci-robot assigned MrFreezeex Jun 22, 2023

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 22, 2023

electrocucaracha force-pushed the update_checksum branch from 68cc3eb to 5e1c6ba Compare June 22, 2023 19:30

k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 22, 2023

MrFreezeex reviewed Jun 22, 2023

View reviewed changes

electrocucaracha force-pushed the update_checksum branch from 5e1c6ba to ceac870 Compare June 22, 2023 21:16

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 23, 2023

k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 4, 2023

Upgrade versions

917bcd1

The following applications have been upgraded: * Cilium * Helm * crun * Katacontainers * youki * gvisor * skopeo * yq Signed-off-by: Victor Morales <chipahuac@hotmail.com>

electrocucaracha force-pushed the update_checksum branch from ceac870 to 917bcd1 Compare July 4, 2023 19:01

k8s-ci-robot removed lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jul 4, 2023

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 5, 2023

mzaian approved these changes Jul 5, 2023

View reviewed changes

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 5, 2023

k8s-ci-robot merged commit 78c1775 into kubernetes-sigs:master Jul 5, 2023

electrocucaracha deleted the update_checksum branch July 5, 2023 13:57

yankay mentioned this pull request Aug 24, 2023

Release Proposal v2.23 #10389

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Upgrade versions #9798

Upgrade versions #9798

electrocucaracha commented Feb 15, 2023 •

edited by floryut

Loading

JaneLiuL left a comment

MrFreezeex left a comment

MrFreezeex Jun 22, 2023

electrocucaracha Jun 22, 2023

MrFreezeex Jun 22, 2023

MrFreezeex commented Jun 22, 2023

MrFreezeex Jun 22, 2023

electrocucaracha Jun 22, 2023

MrFreezeex Jun 22, 2023

electrocucaracha Jun 22, 2023

MrFreezeex Jun 22, 2023 •

edited

Loading

MrFreezeex commented Jun 23, 2023

MrFreezeex commented Jul 5, 2023

mzaian left a comment

k8s-ci-robot commented Jul 5, 2023

Upgrade versions #9798

Upgrade versions #9798

Conversation

electrocucaracha commented Feb 15, 2023 • edited by floryut Loading

JaneLiuL left a comment

Choose a reason for hiding this comment

MrFreezeex left a comment

Choose a reason for hiding this comment

MrFreezeex Jun 22, 2023

Choose a reason for hiding this comment

electrocucaracha Jun 22, 2023

Choose a reason for hiding this comment

MrFreezeex Jun 22, 2023

Choose a reason for hiding this comment

MrFreezeex commented Jun 22, 2023

MrFreezeex Jun 22, 2023

Choose a reason for hiding this comment

electrocucaracha Jun 22, 2023

Choose a reason for hiding this comment

MrFreezeex Jun 22, 2023

Choose a reason for hiding this comment

electrocucaracha Jun 22, 2023

Choose a reason for hiding this comment

MrFreezeex Jun 22, 2023 • edited Loading

Choose a reason for hiding this comment

MrFreezeex commented Jun 23, 2023

MrFreezeex commented Jul 5, 2023

mzaian left a comment

Choose a reason for hiding this comment

k8s-ci-robot commented Jul 5, 2023

electrocucaracha commented Feb 15, 2023 •

edited by floryut

Loading

MrFreezeex Jun 22, 2023 •

edited

Loading