-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Add launch type to ecs task #49081
Add launch type to ecs task #49081
Conversation
Hi @ezmac, thank you for submitting this pull-request! |
@ezmac, just so you are aware we have a dedicated Working Group for aws. |
The test
|
This looks good to me - I'll run the test suite over it later and get this merged if all goes well. |
I've added a trivial patch to this to allow ecs:StopTask and ecs:StartTask rebuild_merge |
bot_status |
Componentschangelogs/fragments/49081-add-launch-type-to-ecs-task.yml changelogs/fragments/ecs_task_fargate.yml hacking/aws_config/testing_policies/container-policy.json lib/ansible/modules/cloud/amazon/ecs_task.py test/integration/targets/ecs_cluster/playbooks/roles/ecs_cluster/files/ecs-trust-policy.json test/integration/targets/ecs_cluster/playbooks/roles/ecs_cluster/tasks/main.yml Metadatawaiting_on: ansible |
e637bd4
to
8ffde40
Compare
force push was just to remove an unnecessary commit I made adding a secondary changelog fragment that was already there. |
SUMMARY
Adds launch_type support to ecs_task
ISSUE TYPE
COMPONENT NAME
ecs_task
ADDITIONAL INFORMATION
Requires a little more permission in the ecs trust policy.