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

Test mac on self-hosted runner #334

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

ElliottKasoar
Copy link
Member

No description provided.

@ElliottKasoar ElliottKasoar added the testing Unit testing or CI label Oct 15, 2024
@ElliottKasoar ElliottKasoar self-assigned this Oct 15, 2024
@ElliottKasoar
Copy link
Member Author

We should be able to replace the python installation once #358 is merged, which should solve the current errors.

I'll probably stick this in the dev guide at some point, but for future reference, we haven't managed to find a nice way to set up the self-hosted runner as a service (see issues e.g. actions/runner#349, actions/runner#1959, actions/runner#1476), which may partly be because we are unable to set auto login, and as we wanted to avoided running things with sudo access.

The solution, which seems to be working for now, is to use the standard run.sh script in the background, and disowning the task, so we can log out:

nohup ./run.sh &!

Sleep options etc. may need revisiting, but the runner has remained online for over an hour.

@ElliottKasoar ElliottKasoar force-pushed the test-mac-self branch 6 times, most recently from 17e42dc to 6d5df4e Compare December 13, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Unit testing or CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant