-
Notifications
You must be signed in to change notification settings - Fork 574
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 the ARM64 e2e tests on Equinix hardware #2191
Conversation
6c6a66c
to
cb9a9d5
Compare
```shell | ||
sudo ./arm64.sh <TOKEN> | ||
curl -sL https://raw.githubusercontent.com/fluxcd/flux2/main/.github/runners/runner-setup.sh > runner-setup.sh \ |
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.
Surely you only need one of these, rather than one in each directory?
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.
The script assumes the runner scripts are in the same dir, please suggest changes to the script so this can be downloaded only once.
- Retrieve the GitHub runner token from the repository [settings page](https://github.com/fluxcd/flux2/settings/actions/runners/new?arch=arm64&os=linux) | ||
- Run the provisioning script passing the token as the first argument | ||
|
||
- Create 3 directories `runner1`, `runner2`, `runner3` |
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.
for d in runner1 runner2 runner3; do ...; done
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
cb9a9d5
to
309fd86
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.
Cool, thanks for rejigging this Stefan 🍇
I'm not totally sure about how the GitHub Actions workflow works -- in particular runs-on
-- where is it defined what equinix
means?. But that is secondary to getting the scripts and instructions in order :-)
@squaremo We could delete the label and remove it from the workflow once the Oracle runners are removed. |
Move the runners from Oracle Cloud free tier to Equinix ARM64 hardware (account sponsored by CNCF).
PS. @squaremo @hiddeco you can delete your Oracle runners after this PR is merged.