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

chore(logic): Runner should be removed before termination #161

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dev-slatto
Copy link

@dev-slatto dev-slatto commented Aug 9, 2023

The EC2 instance should be marked as unschedulable by Github Actions before termination to make sure that Github Actions dosen't schedule any jobs to it while it's being terminated.

garbas added a commit to flox/ec2-github-runner that referenced this pull request Sep 28, 2023
The EC2 instance should be marked as unschedulable by Github Actions
before termination to make sure that Github Actions dosen't schedule any
jobs to it while it's being terminated.

machulav#161
garbas added a commit to flox/ec2-github-runner that referenced this pull request Oct 10, 2023
The EC2 instance should be marked as unschedulable by Github Actions
before termination to make sure that Github Actions dosen't schedule any
jobs to it while it's being terminated.

machulav#161
@Dmitry1987
Copy link

definitely the correct way to do it, why it wasn't merged 😢

@Dmitry1987
Copy link

oh i see the comments, gh.remove() will mark it unschedulable, aha. PRs remain open, should be just closed if the fix is not relevant anymore 🙃

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