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

Check if service exists before stopping it (Linux) #1135

Merged
merged 4 commits into from
Jun 8, 2021

Conversation

fhammerl
Copy link
Contributor

@fhammerl fhammerl commented Jun 7, 2021

Closes #1022

If the runner was installed as a service but disabled using anything else than svc.sh uninstall, running svc.sh uninstall now will fail and it never deletes .service in the runner's home folder. This in turn prevents users from running config.sh remove, because the runner will think it's still configured as a service.

This PR changes svc.sh uninstall on Linux so it deletes .service even is the service no longer exists.

@fhammerl fhammerl added the bug Something isn't working label Jun 7, 2021
@fhammerl fhammerl requested a review from a team as a code owner June 7, 2021 13:05
@fhammerl
Copy link
Contributor Author

fhammerl commented Jun 7, 2021

I need to check the service behaviour on macOS

@TingluoHuang
Copy link
Member

Make sure you tested before merging the PR.

@fhammerl
Copy link
Contributor Author

fhammerl commented Jun 8, 2021

Make sure you tested before merging the PR.

Tested successfully on Linux 👍

@fhammerl fhammerl merged commit a1bcd59 into actions:main Jun 8, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

./config.sh remove - Failed: Removing service - even after service uninstall
2 participants