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
Instructions for managing M1 self-hosted runner #28086
Conversation
190c8bf
to
29827d4
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.
Spotted a few micro things, otherwise it looks good. Thanks!
ACTIONS.md
Outdated
### Non-root user | ||
|
||
GitHub actions should not run with administrator privileges. | ||
Make an account for the actions runner. *Grant administrator privileges to the account* (we will remove them later). |
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.
Given you have a name for the account later, maybe add it there to be consistent?
ACTIONS.md
Outdated
systemctl reboot | ||
``` | ||
|
||
Now remove administrator privileges from the `githubactions` user. |
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.
Maybe an hint of how it can be done? Or it's obvious for someone familiar with macOS?
Also added capitalisation corrections and improved instructions for user management. Co-Authored-By: Guillaume Smet <guillaume.smet@gmail.com>
Suggestions implemented - thanks! |
Could you squash the commits? Thanks. |
b748402
to
edb7449
Compare
The good news is that I actually had squashed the commits. The bad news is I hadn't pushed. So I'm glad you didn't merge! There's a bit of extra commentary about managing users (per your suggestions) on top of what you saw when you last looked. |
As discussed with @gsmet, I have documented the set up instructions for the Mac M1 self-hosted runner added in #27156.
I also removed some hardcoding from our podman instructions.