-
Notifications
You must be signed in to change notification settings - Fork 99
tools: Update clear_offline_runners to clear ec2 #6721
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
Conversation
We recently had an outage that required that we spin down all instances that were not actively connected to GitHub. Signed-off-by: Eli Uriegas <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
Signed-off-by: Eli Uriegas <[email protected]>
Signed-off-by: Eli Uriegas <[email protected]>
Github, | ||
PaginatedList, | ||
SelfHostedActionsRunner, | ||
) | ||
from tqdm import tqdm # type: ignore[import-untyped] | ||
|
||
|
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.
Github won't let me propose changes to lines that didn't change :(
Adding a new argument for runner_name below would be useful as LF account has a different action runner name.
parser.add_argument(
"--runner-name",
help="AWS Runner Name to filter for EC2 instances.",
type=str,
default="gh-ci-action-runner",
)
Co-authored-by: Thanh Ha <[email protected]>
Signed-off-by: Eli Uriegas <[email protected]>
The linter issues are not my issues, will follow up with a PR to fix most of them |
We recently had an outage that required that we spin down all instances that were not actively connected to GitHub.
Requirements to run:
To run you can use:
This was used to remediate issues from pytorch/pytorch#155265