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

feat: add gh actions runner setup for mac on aws #264

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

anjannath
Copy link
Collaborator

this adds new flags to the command mapt aws mac request to to install github actions runner on the provisioned instance

it also adds the additional flags to get the various values needed to setup the github actions runner

@adrianriobo
Copy link
Collaborator

Did you test this one? I mean did you create a mac machine?

@anjannath
Copy link
Collaborator Author

Did you test this one? I mean did you create a mac machine?

yes its tested now :)

@@ -75,3 +86,10 @@ func GetActionRunnerSnippetLinux() string {
}
return util.IfNillable(args != nil, snippetLinux, "")
}

func GetActionRunnerSnippetMacos() string {
return util.If(args != nil,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should use same syntax as others with util.IfNillable otherwise f not ghtunner params passed it will panic (I think)

Copy link
Collaborator Author

@anjannath anjannath Jul 22, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahh, yes

updated now, thanks!

this adds new flags to the command `mapt aws mac request` to
to install github actions runner on the provisioned instance

it also adds the additional flags to get the various values
needed to setup the github actions runner

Signed-off-by: Anjan Nath <kaludios@gmail.com>
Signed-off-by: Anjan Nath <kaludios@gmail.com>
@adrianriobo
Copy link
Collaborator

Fixes #197

@adrianriobo adrianriobo merged commit 11e15d1 into redhat-developer:main Jul 22, 2024
4 checks passed
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.

2 participants