Skip to content

run CI/CD tests also on arm #4848

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

Merged
merged 3 commits into from
Jun 13, 2025
Merged

run CI/CD tests also on arm #4848

merged 3 commits into from
Jun 13, 2025

Conversation

nadiamoe
Copy link
Member

@nadiamoe nadiamoe commented Jun 12, 2025

This PR enables CI/CD pipelines to additionally run on arm machines, more precisely on ubuntu-24.04-arm. This runner is provided by Github to the general public in beta quality, and is documented here: https://github.com/actions/partner-runner-images/blob/main/images/arm-ubuntu-24-image.md

Running CI on arm is a first step towards increasing our confidence on the reliability of k6 and browser in this platform.

Some considerations for reviewers:

  • Unlike ubuntu-24.04, ubuntu-24.04-arm does not come with chrome or chromium preinstalled, thus the reason of the conditional "install chromium" step.
  • Browser e2e tests force the use of google-chrome binary, as opposed to chromium:
    if [ "$RUNNER_OS" == "Linux" ] && [ "$RUNNER_ARCH" == "X64" ]; then
    . For arm, a modern chromium is used instead, as that's what is available.

@nadiamoe nadiamoe force-pushed the ci-arm branch 2 times, most recently from 3559fc0 to fadd08e Compare June 12, 2025 10:40

This comment has been minimized.

@nadiamoe nadiamoe force-pushed the ci-arm branch 2 times, most recently from 01b28f3 to 99eca80 Compare June 12, 2025 13:04
@nadiamoe
Copy link
Member Author

Tired to use crocochrome for this in 99eca80, but unfortunately that did not work: The test suite includes tests for the lifecycle of the built-in browser process manager, so that's not an option.

@nadiamoe nadiamoe force-pushed the ci-arm branch 7 times, most recently from 61a70ac to 00fb72e Compare June 12, 2025 15:02
@nadiamoe nadiamoe changed the title wip: run (part of) CI/CD tests also on arm run CI/CD tests also on arm Jun 12, 2025
@nadiamoe nadiamoe marked this pull request as ready for review June 12, 2025 15:47
@nadiamoe nadiamoe requested a review from a team as a code owner June 12, 2025 15:47
@nadiamoe nadiamoe requested review from inancgumus, ankur22, mstoykov and joanlopez and removed request for a team June 12, 2025 15:47
mstoykov
mstoykov previously approved these changes Jun 13, 2025
Copy link
Contributor

@mstoykov mstoykov left a comment

Choose a reason for hiding this comment

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

Thank you for the PR 🙇

It is strange that 24.04 works with arm but not x86 ... I do wonder if we want to run on 22.04 in both cases 🤔 or not.

But we definitely want to also run this on gotip

@nadiamoe nadiamoe requested review from joanlopez and mstoykov June 13, 2025 09:57
@joanlopez joanlopez merged commit 0641c73 into master Jun 13, 2025
35 of 39 checks passed
@joanlopez joanlopez deleted the ci-arm branch June 13, 2025 10:57
@joanlopez joanlopez added this to the v1.1.0 milestone Jun 13, 2025
@mstoykov mstoykov mentioned this pull request Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants