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

[release/1.7] CI: arm64-8core-32gb -> ubuntu-24.04-arm #11437

Merged
merged 2 commits into from
Feb 27, 2025
Merged

[release/1.7] CI: arm64-8core-32gb -> ubuntu-24.04-arm #11437

merged 2 commits into from
Feb 27, 2025

Conversation

AkihiroSuda
Copy link
Member

@dosubot dosubot bot added the github_actions Pull requests that update GitHub Actions code label Feb 26, 2025
@AkihiroSuda
Copy link
Member Author

Looks like we need 624aa49 too

akhilerm and others added 2 commits February 26, 2025 10:55
@akhilerm @AkihiroSuda
increase xfs base image size to 300Mb
561ed52 
from xfsprogs 5.19.0, minimum size of xfs volume has to be 300MB

Ref: https://git.kernel.org/pub/scm/fs/xfs/xfsprogs-dev.git/commit/?id=6e0ed3d19c54603f0f7d628ea04b550151d8a262
Signed-off-by: Akhil Mohan <[email protected]>
(cherry picked from commit 624aa49)
Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda
CI: arm64-8core-32gb -> ubuntu-24.04-arm
85f10bd 
GHA now provides ARM runners for free

https://github.blog/changelog/2025-01-16-linux-arm64-hosted-runners-now-available-for-free-in-public-repositories-public-preview/

Signed-off-by: Akihiro Suda <[email protected]>
(cherry picked from commit 4e7484d)
> Conflicts:
>	.github/workflows/ci.yml
Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda
Copy link
Member Author

/retest

@djdongjin
Copy link
Member

/test pull-containerd-node-e2e-1-7

@samuelkarp
Copy link
Member

/retest

@AkihiroSuda
Copy link
Member Author

/retest

@k8s-ci-robot
Copy link

@AkihiroSuda: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-containerd-node-e2e-1-7 85f10bd link true /test pull-containerd-node-e2e-1-7

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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-sigs/prow repository. I understand the commands that are listed here.

@estesp
Copy link
Member

estesp commented Feb 27, 2025

Change unrelated to e2e tests on completely separate infrastructure. This change is about selecting our runner for arm64 on GH Actions; going to merge and ignore flake

@estesp estesp merged commit 78b04b6 into containerd:release/1.7 Feb 27, 2025
56 of 57 checks passed
@dmcgowan dmcgowan mentioned this pull request Mar 14, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samuelkarp samuelkarp samuelkarp approved these changes

@estesp estesp estesp approved these changes

@djdongjin djdongjin djdongjin approved these changes

@austinvazquez austinvazquez austinvazquez approved these changes

Assignees
No one assigned
Labels
github_actions Pull requests that update GitHub Actions code size/S
Projects
Pull Request Review
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@AkihiroSuda @djdongjin @samuelkarp @k8s-ci-robot @estesp @austinvazquez @akhilerm