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.

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

Change to self-hosted runner in PSI for ARM64 #408

Merged
merged 10 commits into from
Sep 22, 2023
Merged

Conversation

unkcpz
Copy link
Member

@unkcpz unkcpz commented Sep 7, 2023

The self-hosted runner is deployed in PSI, it is a macOS M2 server that aims to test the docker build on ARM64 machine. The instructions on how this self-hosted runner is set up is create for reference.

@unkcpz unkcpz changed the title Change the self-runner Change to use the self-rhosted runner in PSI Sep 7, 2023
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch 4 times, most recently from e66791a to 81b5855 Compare September 7, 2023 16:23
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch from 81b5855 to 783911b Compare September 8, 2023 14:09
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch 2 times, most recently from 62dd251 to eb6ffdf Compare September 11, 2023 23:30
[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch 15 times, most recently from 1564bf9 to 2af5c55 Compare September 12, 2023 12:47
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch from 2af5c55 to 4f5f672 Compare September 12, 2023 12:50
@unkcpz unkcpz changed the title Change to use the self-rhosted runner in PSI Change to use the self-hosted runner in PSI Sep 12, 2023
@unkcpz
Copy link
Member Author

unkcpz commented Sep 12, 2023

Hi @mbercx, I run all tests on my Macbook air, and seems all good. Can you give it a look and set up the self-host runner on your Mac when you get time?

I use colima as the docker runtime instead of the standard Docker Desktop which I assume you will use for your daily work. This was adapted to avoid the images created and deleted by the CI actions having side effects on your daily work.

P.S. Your machine has IP and able to access to internet with wire connection. Let me know if you need ssh service to login remotely.

@unkcpz unkcpz added the blocked This issue/PR is blocked by another issue/PR. label Sep 13, 2023
@unkcpz
Copy link
Member Author

unkcpz commented Sep 13, 2023

This PR is blocked by aiidalab/aiidalab-qe#449. Since I remove the permission of this repo to push to package for aiidalab/qe image. After aiidalab/aiidalab-qe#449, I'll create PR in this repo to remove QE image build.

@unkcpz unkcpz self-assigned this Sep 14, 2023
@unkcpz
Copy link
Member Author

unkcpz commented Sep 16, 2023

After I tried this on my laptop, I ran into some issues with homebrew (since I reinstalled it), and it led me to this post https://www.codejam.info/2021/11/homebrew-multi-user.html.

I think to avoid the self-runner potentially breaking the regular system user (that is @mbercx in our case), I think the homebrew should installed only for Marnik, and the newly created runner-user should use sudo to run the script. But I may worried too much since the setup script only runs once to setup, and no need to be fully automatic.

@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch 2 times, most recently from 44fe1bc to f1e032a Compare September 20, 2023 17:17
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch from f1e032a to 5c2755f Compare September 21, 2023 07:43
@unkcpz unkcpz removed the blocked This issue/PR is blocked by another issue/PR. label Sep 21, 2023
@unkcpz unkcpz changed the title Change to use the self-hosted runner in PSI Change to macos-latest runner for ARM64 Sep 21, 2023
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch 2 times, most recently from c745c23 to 0c74839 Compare September 21, 2023 13:46
@unkcpz unkcpz changed the title Change to macos-latest runner for ARM64 Change to self-hosted runner in PSI for ARM64 Sep 21, 2023
@unkcpz unkcpz force-pushed the self-hosted-arm64-runner branch from 0c74839 to 9e482c3 Compare September 21, 2023 16:55
@unkcpz unkcpz requested a review from mbercx September 21, 2023 19:28
@unkcpz unkcpz merged commit 515bd9c into main Sep 22, 2023
24 checks passed
@unkcpz unkcpz deleted the self-hosted-arm64-runner branch September 22, 2023 09:27
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.

1 participant