-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
CI: add Ubuntu 24.04 support #11132
CI: add Ubuntu 24.04 support #11132
Conversation
Hi @tico88612. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
test-infra/image-builder/roles/kubevirt-images/defaults/main.yml
Outdated
Show resolved
Hide resolved
ede7689
to
0576478
Compare
If there is one, please also add a vagrant box in the supported_os map in Vagrantfile. That makes it vastly easier to locally test stuff, rather than relying on CI (on my machine that's 5 minutes vs more than 1 hour) |
0576478
to
c9a3c82
Compare
do you have access the the CI logs ? here the error:
and others:
|
@ant31 Docker has a history of past commits, so I think I can handle that part, but I don't have any idea about CoreDNS for now. |
yes, hard to tell without the pod logs. |
Hugh. I changed the default for `unsafe_show_logs` in vagrant but not in CI...
EDIT: in fact it's specified in tests/common/_kubespray_test_settings.yml so that's weird 🤔
|
@VannTen How to run local tests via Vagrantfile? |
I think you can run the tests playbook the same way that the other on the vagrant inventory. I don't have the exact commands on top of my head but docs/vagrant.md should have that
|
c9a3c82
to
97770a5
Compare
97770a5
to
e47c544
Compare
I tested in the local Vagrant and searched for some relevant documentation, but I still had no idea why it happened.
(This error wasn't seen in other OS.) |
@tico88612 you can copy and create an ubuntu-24.yml |
c21e4e5
to
19a6251
Compare
all green ! /lgtm |
Thanks @tico88612 |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ant31, tico88612, yankay 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:
Approvers can indicate their approval by writing |
962b3b5
to
725f160
Compare
/lgtm |
Signed-off-by: tico88612 <[email protected]>
725f160
to
1c71dee
Compare
/lgtm
Thanks, hope this finally merge!
|
Signed-off-by: tico88612 <[email protected]>
Signed-off-by: tico88612 <[email protected]>
Signed-off-by: tico88612 <[email protected]>
Signed-off-by: tico88612 <[email protected]>
Signed-off-by: tico88612 <[email protected]>
What type of PR is this?
/kind feature
What this PR does / why we need it:
Support Ubuntu 24.04
Which issue(s) this PR fixes:
Related #11124
Special notes for your reviewer:
Ubuntu 22.04 Ref. #8958
Does this PR introduce a user-facing change?: