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

OPS-3896 #34

Merged
merged 4 commits into from
Sep 10, 2020
Merged

OPS-3896 #34

merged 4 commits into from
Sep 10, 2020

Conversation

runderwoodcr14
Copy link
Contributor

Bump modules versions to be compatible with aws provider 3.X.X also updated examples, changed makefile to use terraform verision 0.12.29 as ver was using light and light uses latest release available in this case 0.13.x
terraform-aws-rds
cloudposse dynamodb

Copy link
Contributor

@danvaida danvaida left a comment

Choose a reason for hiding this comment

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

I trust @runderwoodcr14 to have checked the changelog of each and every of those external resources and ensure that there is no difference when it comes to inputs and outputs 😄

@@ -13,7 +13,7 @@ resource "null_resource" "dynamodb3_checker" {

module "dynamodb3" {
Copy link
Contributor

Choose a reason for hiding this comment

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

Funny how I have absolutely zero clue what the heck dynamodb_2 or dynamodb_3 means 😄

@runderwoodcr14
Copy link
Contributor Author

runderwoodcr14 commented Sep 10, 2020

I trust @runderwoodcr14 to have checked the changelog of each and every of those external resources and ensure that there is no difference when it comes to inputs and outputs 😄

Man if you are going to ask those kind of things here is no fun for deploying in production then 😂

@runderwoodcr14 runderwoodcr14 merged commit 99ad688 into master Sep 10, 2020
@runderwoodcr14 runderwoodcr14 deleted the OPS-3896 branch September 10, 2020 11:44
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