-
Notifications
You must be signed in to change notification settings - Fork 23
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.