-
Notifications
You must be signed in to change notification settings - Fork 86
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
Run RBE tests on linux-arm64 #6289
Conversation
3642087
to
1f53147
Compare
1f53147
to
f86d179
Compare
ab62a1e
to
d7ffd53
Compare
56a2918
to
ac8b331
Compare
755416d
to
cc443ac
Compare
2b1e191
to
b663332
Compare
38f538f
to
7721959
Compare
5d35f47
to
0ce5055
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few nits and questions, but LGTM.
Very excited to see this being added!
# GitHub's arm64 runners do not have the same pre-installed software as | ||
# the amd64 runners so we have to install a few things here. | ||
|
||
- name: Install bazelisk | ||
run: | | ||
curl -L --output /tmp/bazelisk https://github.com/bazelbuild/bazelisk/releases/download/v1.17.0/bazelisk-linux-arm64 | ||
chmod +x /tmp/bazelisk | ||
sudo mv /tmp/bazelisk /usr/bin/bazelisk | ||
sudo ln -s /usr/bin/bazelisk /usr/bin/bazel | ||
|
||
- name: Install apt packages | ||
run: | | ||
sudo apt-get update | ||
sudo apt-get install -y gcc g++ podman |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we run these in an ARM64 container? 🤔
https://docs.github.com/en/actions/using-jobs/running-jobs-in-a-container
It could simplify this setup by quite a bit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Container jobs aren't supported in the arm runners beta yet - idk if it's mentioned in the public docs but they mentioned that in their email.
[engine] | ||
cgroup_manager = "cgroupfs" | ||
' | ||
mkdir -p ~/.local |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dumb question: why is ~/.local
needed to be created manually here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't remember why I added this but it works without it - removed.
@@ -380,9 +385,13 @@ func TestForceRoot(t *testing.T) { | |||
} | |||
|
|||
func TestUser(t *testing.T) { | |||
if runtime.GOARCH == "arm64" { | |||
// TODO: build podman ourselves, and remove this | |||
t.Skipf("--passwd arg is not yet supported by podman version available on GitHub actions runner") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: specify the expected vs actual version number so we can flip this in the future when newer versions become available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
0ce5055
to
d952edd
Compare
d952edd
to
e1e1bd4
Compare
Adds a GitHub workflow that runs applicable RBE tests on linux-arm64 runners.
Related issues: N/A