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.6] CI: arm64-8core-32gb -> ubuntu-24.04-arm #11438

Merged
merged 2 commits into from
Feb 27, 2025
Merged

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

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:56
@akhilerm @AkihiroSuda
increase xfs base image size to 300Mb
2cc6b5b 
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
f5ab73c 
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-6

@samuelkarp
Copy link
Member

/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-6 f5ab73c link true /test pull-containerd-node-e2e-1-6

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

The pull-containerd-node-e2e-1-* tests have been extremely flaky lately. We really didn't need to run it with this PR because prow is completely separate infrastructure that is not impacted by our choice of GH Actions CI runners. Going to merge this without the e2e test passing as it is irrelevant.

@estesp estesp merged commit 6f27a18 into containerd:release/1.6 Feb 27, 2025
48 of 49 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/M
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