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

Terraform 0.14 upgrade #28

Merged
merged 4 commits into from
Dec 11, 2020
Merged

Terraform 0.14 upgrade #28

merged 4 commits into from
Dec 11, 2020

Conversation

Nuru
Copy link
Contributor

@Nuru Nuru commented Dec 11, 2020

what

  • Upgrade to support Terraform 0.14 and bring up to current Cloud Posse standard
  • Remove IAM role and policies

why

  • Support Terraform 0.14
  • Not needed now that we have service-linked roles
  • Closes Support context #25

@Nuru Nuru requested a review from a team as a code owner December 11, 2020 02:40
@Nuru Nuru requested review from htplbc and brcnblc December 11, 2020 02:40
@Nuru
Copy link
Contributor Author

Nuru commented Dec 11, 2020

/test all

aknysh
aknysh previously requested changes Dec 11, 2020
Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

need to fix

var.enabled is null

@Nuru Nuru requested a review from a team as a code owner December 11, 2020 02:48
@Nuru
Copy link
Contributor Author

Nuru commented Dec 11, 2020

/test all

1 similar comment
@Nuru
Copy link
Contributor Author

Nuru commented Dec 11, 2020

/test all

@Nuru
Copy link
Contributor Author

Nuru commented Dec 11, 2020

/test all

@Nuru
Copy link
Contributor Author

Nuru commented Dec 11, 2020

/test all

@Nuru Nuru requested review from aknysh and osterman December 11, 2020 03:23
@@ -51,7 +51,9 @@ description: |-
usage: |-
```hcl
module "dynamodb_autoscaler" {
source = "git::https://github.com/cloudposse/terraform-aws-dynamodb-autoscaler.git?ref=master"
source = "cloudposse/dynamodb-autoscaler/aws"
Copy link
Member

Choose a reason for hiding this comment

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

Our regex should capture the whitespace to preserve the formatting

Copy link
Contributor Author

Choose a reason for hiding this comment

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

With the comment below it, the example is formatted as terraform fmt would format it.

Copy link
Member

Choose a reason for hiding this comment

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

Technically correct ;)

@Nuru Nuru merged commit 8d99de1 into master Dec 11, 2020
@Nuru Nuru deleted the terraform-0.14-upgrade branch December 11, 2020 04:40
sidprak added a commit to verkada/terraform-aws-dynamodb-autoscaler that referenced this pull request Apr 28, 2021
* make this module v14 compatible (cloudposse#27)

Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>

* Terraform 0.14 upgrade (cloudposse#28)

* Fix auto-context and validate-codeowners (cloudposse#29)

* Update README.md and docs (cloudposse#30)

Co-authored-by: Nuru <Nuru@users.noreply.github.com>

* Terraform 0.14 upgrade (cloudposse#31)

* Terraform 0.14 upgrade

* updated workflows

* Updated README.md

Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>

* context.tf updated to v0.24.1, minimum required Terraform version bumped to 0.13.0 when needed, readme updated (cloudposse#33)

Co-authored-by: Jürgen W <jurgenweber@users.noreply.github.com>
Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
Co-authored-by: Nuru <Nuru@users.noreply.github.com>
Co-authored-by: Cloud Posse Bot (CI/CD) <bot@cloudposse.com>
Co-authored-by: Maxim Mironenko <maxim@cloudposse.com>
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.

3 participants