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

Set runner's information to the output much earlier #10

Merged
merged 3 commits into from
Dec 30, 2020
Merged

Conversation

machulav
Copy link
Owner

When the GitHub Action workflow is canceled during the execution of the "Start self-hosted EC2 runner" job, the final job, "Stop self-hosted EC2 runner" failed as the input was provided only at the end of the first job. Though, the EC2 runner data available already in the first seconds of the first job execution. The rest of the time, the script waits for the runner to be registered as a self-hosted runner in GitHub actions.

These changes make the output available earlier, right after the EC2 runner data is available. In this way, the EC2 runner can be properly terminated even if the GitHub Actions workflow is canceled during the execution of the "Start self-hosted EC2 runner" job.

Related to issue #7

@machulav machulav merged commit c8f4d53 into main Dec 30, 2020
@machulav machulav deleted the issue7 branch December 30, 2020 12:51
@machulav machulav self-assigned this Jan 7, 2021
@machulav machulav added the enhancement New feature or request label Jan 8, 2021
@machulav machulav linked an issue Jan 8, 2021 that may be closed by this pull request
@machulav machulav added bug Something isn't working and removed enhancement New feature or request labels Jan 15, 2021
@machulav machulav removed their assignment Jan 20, 2021
mmoriani pushed a commit to ottimis/ec2-github-runner that referenced this pull request Feb 15, 2024
* Set runner's information to the output earlier
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.

Are instances guaranteed to be terminated?
1 participant