Skip to content
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

BACKPORT hashicorp#27390 fix resource/aws_ecs_service: make task_definition truly optional #709

Merged
merged 3 commits into from
Oct 24, 2022

Conversation

jusunglee
Copy link

AWS introduced a breaking change on their task definition API. Fixed in a PR on the upstream repo, but obviously we need to backport here. hashicorp#27390

Context https://figma.slack.com/archives/C028C7B8XUP/p1666619400244529

@jusunglee jusunglee changed the title BACKPORT fix resource/aws_ecs_service: make task_definition truly optional BACKPORT hashicorp#27390 fix resource/aws_ecs_service: make task_definition truly optional Oct 24, 2022
Copy link

@caesarxuchao caesarxuchao left a comment

Choose a reason for hiding this comment

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

After this gets merged, do we need to make any change in figma/figma to pickup the changes?

@jusunglee
Copy link
Author

@caesarxuchao Yes we'll have to update the provider version in the terraform code

We can copy this: https://github.com/figma/figma/pull/79294

@caesarxuchao
Copy link

I assume I will also need to manually create a new tag v3.69.0-figma.5?

@jusunglee jusunglee merged commit 8e133b6 into figma Oct 24, 2022
caesarxuchao added a commit that referenced this pull request Oct 26, 2022
Fix #709, which was backport hashicorp#27390, but missed a line of deletion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants