Skip to content

feat: support custom platform version #170

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

Merged

Conversation

jychen7
Copy link
Contributor

@jychen7 jychen7 commented Dec 10, 2020

Description

Allow to set custom ECS Fargate platform version

Motivation and Context

currently, by default, ECS Fargate is using platform version 1.3.0
However, 1.4.0 introduce more features I would like Atlantis to use, including

  • For tasks using platform version 1.4.0 or later that are launched on May 28, 2020 or later, the ephemeral storage is encrypted with an AES-256 encryption algorithm using an AWS Fargate-managed encryption key

https://docs.aws.amazon.com/AmazonECS/latest/developerguide/platform_versions.html
https://docs.aws.amazon.com/AmazonECS/latest/developerguide/fargate-task-storage.html

Breaking Changes

Nope, new variables are backward compatible and match existing defaults

How Has This Been Tested?

planed and applied

@jychen7 jychen7 changed the title support custom platform version and optionally force new deployment w… feat: support custom platform version and optionally force new deployment w… Dec 10, 2020
@jychen7 jychen7 force-pushed the ecs-support-platform-version branch 2 times, most recently from 91df9ba to ae820e2 Compare December 10, 2020 23:53
@jychen7 jychen7 changed the title feat: support custom platform version and optionally force new deployment w… feat: support custom platform version Dec 11, 2020
@jychen7 jychen7 force-pushed the ecs-support-platform-version branch from ae820e2 to 6aeebf6 Compare December 11, 2020 03:40
@jychen7 jychen7 force-pushed the ecs-support-platform-version branch from 6aeebf6 to 1756d3f Compare December 11, 2020 03:41
Copy link
Contributor

@raizyr raizyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed. 👍🏻

Copy link
Member

@bryantbiggs bryantbiggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@antonbabenko verified this change - good to merge 👍🏼

@bryantbiggs
Copy link
Member

@jychen7 it would be good if you could update the github-complete example to add the new parameter ecs_service_platform_version thanks!

@jychen7
Copy link
Contributor Author

jychen7 commented Dec 24, 2020

it would be good if you could update the github-complete example to add the new parameter ecs_service_platform_version thanks!

@bryantbiggs sure, done in 5826a01

@bryantbiggs
Copy link
Member

thank you @jychen7 - @antonbabenko good to go 👍🏼

@antonbabenko antonbabenko merged commit ccc18d5 into terraform-aws-modules:master Dec 24, 2020
@antonbabenko
Copy link
Member

Great! v2.29.0 has been just released.

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants