-
Notifications
You must be signed in to change notification settings - Fork 23
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
docs/ec2-debug-and-manual-cleanup #240
docs/ec2-debug-and-manual-cleanup #240
Conversation
I plan to merge this with mine that's still a draft (because I plan to stop procrastinating on it soon...) @iterative/cml thoughts on the messaging? (I can incorporate the feedback and collapse this PR in with mine linked above.) @JacksonMaxfield I have copied the below from an old message on our discord server.
|
Thanks for the info! That is very useful! Not quite following your sentence, can I add that table / list to this PR or are you going to take this PR and shove it into the existing draft PR of yours? Happy with either. |
@JacksonMaxfield feel free to make edits and improvements here, what I mean is: If there are conflicts with the other branch I mentioned, I will fix them 😅 |
Resolved comments and added the IAM permissions section 👍 |
Associated with: iterative/cml#1006
cc @dacbd
Further sidenote: If someone can tell me the exact IAM permissions required for the bare minimum EC2 runner to spin up, I would happily add that in this PR too.