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

Revert terraform resources aws version 3.37.0 implementation #972

Conversation

avrohomgottlieb
Copy link
Contributor

@avrohomgottlieb avrohomgottlieb commented Nov 22, 2024

Issue Number

#964

Purpose/Implementation Notes

We are attempting to revert the aws_provider version to what it was before we began making changes, which was 3.37.0. This PR updates the resources to be compatible again with 3.37.0 (and comments out the newer implementation).

The next step would be to incrementally update aws_provider version to where we need it to be, which is 5.12.0, but gradually updating the resources as we update the versions.

Types of changes

What types of changes does your code introduce?

  • Bugfix (non-breaking change which fixes an issue)

Functional tests

N/A

Checklist

  • Lint and unit tests pass locally with my changes

Screenshots

N/A

…g changes from feature/batch to be in accordance with old 3.37.0 implementation.
@avrohomgottlieb avrohomgottlieb changed the title Revert tf resources aws version 3.37.0 implementation Revert terraform resources aws version 3.37.0 implementation Nov 22, 2024
infrastructure/api.tf Show resolved Hide resolved
infrastructure/database.tf Show resolved Hide resolved
infrastructure/networking.tf Show resolved Hide resolved
infrastructure/s3.tf Show resolved Hide resolved
infrastructure/s3.tf Show resolved Hide resolved
infrastructure/s3.tf Show resolved Hide resolved
infrastructure/variables.tf Show resolved Hide resolved
@avrohomgottlieb avrohomgottlieb merged commit bf43e39 into dev Nov 22, 2024
4 of 5 checks passed
@avrohomgottlieb avrohomgottlieb deleted the avrohom/revert-tf-resources-aws-version-3-37-0-implementation branch November 24, 2024 22:00
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