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.

Sign up for GitHub

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

Jump to bottom

e2e: bump to k8s v1.30.1 #429

Merged
merged 1 commit into from
Sep 10, 2024
Merged

e2e: bump to k8s v1.30.1 #429

merged 1 commit into from
Sep 10, 2024

Conversation

mythi
Copy link
Contributor

@mythi mythi commented Aug 30, 2024

Closes: #161
Fixes: #427

@mythi mythi mentioned this pull request Sep 4, 2024
Closed
@mythi
Copy link
Contributor Author

mythi commented Sep 5, 2024

All of the failures are with the newly added Install kube binaries but interestingly it works on some targets 🤔

@ldoktor
Copy link
Contributor

ldoktor commented Sep 5, 2024

All of the failures are with the newly added Install kube binaries but interestingly it works on some targets 🤔

Perhaps it's related to rate limits @GabyCT addressed for kustomize in #404

@mythi
Copy link
Contributor Author

mythi commented Sep 5, 2024

It looks extra_opts setting was failing on some targets. ebb1a25 moves away from using it and tests seem going further at least

@mythi mythi marked this pull request as ready for review September 5, 2024 11:30
@mythi mythi changed the title WIP: more k8s updates e2e: bump to k8s v1.30.1 Sep 5, 2024
@mythi
Copy link
Contributor Author

mythi commented Sep 5, 2024

./cluster/up.sh: line 21: /usr/local/bin/kubeadm: Permission denied

This is the remaining error AFAICS.

@mythi mythi mentioned this pull request Sep 9, 2024
Merged
@mythi
e2e: bump to k8s v1.30.1
170c383 
Signed-off-by: Mikko Ylinen <[email protected]>
@mythi
Copy link
Contributor Author

mythi commented Sep 10, 2024

@fidencio @stevenhorsman @ldoktor this looks to be in a good shape now

stevenhorsman
stevenhorsman approved these changes Sep 10, 2024
View reviewed changes
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 @mythi!

fidencio
fidencio approved these changes Sep 10, 2024
View reviewed changes
Copy link
Member

@fidencio fidencio 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 @mythi!

@fidencio fidencio merged commit ee57bc4 into confidential-containers:main Sep 10, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stevenhorsman stevenhorsman stevenhorsman approved these changes

@fidencio fidencio fidencio approved these changes

Assignees
No one assigned
Labels
ok-to-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade the default kubernetes version
4 participants
@mythi @ldoktor @fidencio @stevenhorsman