-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Update k8s version to 1.33 #12221
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
Update k8s version to 1.33 #12221
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #12221 +/- ##
===========================================
- Coverage 40.36% 19.81% -20.56%
===========================================
Files 187 184 -3
Lines 19211 19146 -65
===========================================
- Hits 7755 3793 -3962
- Misses 10273 14923 +4650
+ Partials 1183 430 -753
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
scripts/download
Outdated
esac | ||
|
||
git clone --single-branch --branch=${VERSION_CONTAINERD} --depth=1 https://${PKG_CONTAINERD_K3S/\/v*/} ${CONTAINERD_DIR} | ||
git clone --single-branch --branch=${VERSION_CONTAINERD} --depth=1 https://github.com/vitorsavian/containerd ${CONTAINERD_DIR} |
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.
nit: you shouldn't have to change this when using a personal for, as PKG_CONTAINERD_K3S is automatically set to whatever the replace entry in go.mod points at
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.
This is a funny change, the https://${PKG_CONTAINERD_K3S/\/v*/}
is intended to delete /v2
in k3s-io/containerd/v2
, but for me it was deleting vitorsavian/containerd/v2
in github.com/vitorsavian/containerd/v2
, so I was only getting https://github.com
in the git clone
ps: it's the first time I had something like that related to my name lol
785e47c
to
ebd7989
Compare
/trivy |
|
Signed-off-by: Vitor Savian <[email protected]> Fix prints that broke unit tests Signed-off-by: Vitor Savian <[email protected]>
ebd7989
to
7e5d7df
Compare
Signed-off-by: Vitor Savian <[email protected]>
7e5d7df
to
a77fd3c
Compare
/trivy |
|
Signed-off-by: Vitor Savian <[email protected]>
scripts/binary_size_check.sh
Outdated
@@ -11,9 +11,9 @@ if [ "${DEBUG}" = 1 ]; then | |||
set -x | |||
fi | |||
|
|||
# Try to keep the K3s binary under 70 megabytes. | |||
# Try to keep the K3s binary under 80 megabytes. |
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.
75 or 80?
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, one nit on the size comment ;)
Signed-off-by: Vitor Savian <[email protected]>
e472085
to
e689d32
Compare
Various
|
Signed-off-by: Vitor Savian <[email protected]>
3ec965b
to
6a8d113
Compare
Signed-off-by: Vitor Savian <[email protected]>
Proposed Changes
Update k8s version and components
Types of Changes
Verification
Testing
Linked Issues
User-Facing Change
Further Comments
This Draft/PR is to see how the changes behave with my forks, but afterwards I will change for the tags in k3s forks
The
scripts
folder changes will not be added in the final PR, is only for testing!There is a version in the
go.mod
-> github.com/vitorsavian/cadvisor v0.52.0-k3s1 that is not the final version, I just messed up with the tag, but the tag will bev0.52.1-k3s1