Skip to content

[release/1.7] *: enable ARM64 runner#9502

Merged
AkihiroSuda merged 1 commit intocontainerd:release/1.7from
akhilerm:cherry-pick-9456-1.7
Dec 15, 2023
Merged

[release/1.7] *: enable ARM64 runner#9502
AkihiroSuda merged 1 commit intocontainerd:release/1.7from
akhilerm:cherry-pick-9456-1.7

Conversation

@akhilerm
Copy link
Member

There are many Kubernetes clusters running on ARM64. Enable ARM64 runner is to commit to support ARM64 platform officially.

(cherry picked from commit cb5a48e)

not a clean cherry-pick of #9456

strategy:
matrix:
os: [ubuntu-20.04, macos-12, windows-2019]
os: [ubuntu-22.04, actuated-arm64-4cpu-16gb, macos-12, windows-2019]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure we want to upgrade the base Ubuntu version for the existing release?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Will keep it as ubuntu 20 itself.

@akhilerm akhilerm force-pushed the cherry-pick-9456-1.7 branch from 8901ff9 to 3d62a59 Compare December 13, 2023 06:03
@estesp
Copy link
Member

estesp commented Dec 13, 2023

Looks like there is a cgroups issue during tests only for the deprecated Linux runtimes, only on the arm64 hardware; more details in the CI annotated output, but all of them end up with this core error:

cgroups: cgroup mountpoint does not exist: unknown

@akhilerm
Copy link
Member Author

Looks like there is a cgroups issue during tests only for the deprecated Linux runtimes, only on the arm64 hardware; more details in the CI annotated output, but all of them end up with this core error:

cgroups: cgroup mountpoint does not exist: unknown

Yes . It happens because the arm64 runners are on ubuntu 22, which has cgroupv2 by default (cgroupv1 is disabled) . The v1 version of runc and shim only supports cgroupv1. Was thinking about skipping those tests on arm64.

Another question that I had was, when we want to migrate release/1.7 and release/1.6 branch to ubuntu22, at that time these tests will start to fail on other runners also, not just arm64.

There are many Kubernetes clusters running on ARM64. Enable ARM64 runner
is to commit to support ARM64 platform officially.

runc.v1 and runtime.v1 runtime tests are skipped on arm64 runners
because they doesnot support cgroupv2; arm64 nodes has only ubuntu22.04
at the moment, and ubuntu22.04 has cgroupv2 enabled and cgroupv1
disabled by default.

Signed-off-by: Wei Fu <fuweid89@gmail.com>
(cherry picked from commit cb5a48e)
Signed-off-by: Akhil Mohan <akhilerm@gmail.com>
@akhilerm akhilerm force-pushed the cherry-pick-9456-1.7 branch from 3d62a59 to c631651 Compare December 14, 2023 15:30
@AkihiroSuda AkihiroSuda merged commit 8d0b1c8 into containerd:release/1.7 Dec 15, 2023
@akhilerm akhilerm deleted the cherry-pick-9456-1.7 branch December 15, 2023 03:55
@akhilerm
Copy link
Member Author

/cherrypick release/1.6

@k8s-infra-cherrypick-robot

@akhilerm: #9502 failed to apply on top of branch "release/1.6":

Applying: *: enable ARM64 runner
Using index info to reconstruct a base tree...
M	.github/workflows/ci.yml
Falling back to patching base and 3-way merge...
Auto-merging .github/workflows/ci.yml
CONFLICT (content): Merge conflict in .github/workflows/ci.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 *: enable ARM64 runner
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Details

In response to this:

/cherrypick release/1.6

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.

mxpv added a commit to mxpv/containerd that referenced this pull request Apr 18, 2025
This commit removes v1 runtime from integration tests.

These rely on cgroups v1, which are disabled on Ubuntu 24 in favor of
cgroups v2.

See more context here: containerd#9502 (comment)

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
akhilerm pushed a commit to akhilerm/containerd that referenced this pull request May 6, 2025
This commit removes v1 runtime from integration tests.

These rely on cgroups v1, which are disabled on Ubuntu 24 in favor of
cgroups v2.

See more context here: containerd#9502 (comment)

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
(cherry picked from commit 6f4ffad)
akhilerm pushed a commit to akhilerm/containerd that referenced this pull request May 6, 2025
This commit removes v1 runtime from integration tests.

These rely on cgroups v1, which are disabled on Ubuntu 24 in favor of
cgroups v2.

See more context here: containerd#9502 (comment)

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
(cherry picked from commit 6f4ffad)
akhilerm pushed a commit to akhilerm/containerd that referenced this pull request May 6, 2025
This commit removes v1 runtime from integration tests.

These rely on cgroups v1, which are disabled on Ubuntu 24 in favor of
cgroups v2.

See more context here: containerd#9502 (comment)

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
(cherry picked from commit 6f4ffad)
akhilerm pushed a commit to akhilerm/containerd that referenced this pull request May 6, 2025
This commit removes v1 runtime from integration tests.

These rely on cgroups v1, which are disabled on Ubuntu 24 in favor of
cgroups v2.

See more context here: containerd#9502 (comment)

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
(cherry picked from commit 6f4ffad)
akhilerm pushed a commit to akhilerm/containerd that referenced this pull request May 6, 2025
This commit removes v1 runtime from integration tests.

These rely on cgroups v1, which are disabled on Ubuntu 24 in favor of
cgroups v2.

See more context here: containerd#9502 (comment)

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
(cherry picked from commit 6f4ffad)
@akhilerm akhilerm mentioned this pull request May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants