Skip to content

Commit

Permalink
Add note about AWS Terraform provider version
Browse files Browse the repository at this point in the history
We use a Terraform module that isn't yet compatible with v5 of the AWS provider, and this adds a note about the situation, since it might not be clear to someone coming in fresh to the codebase. Hopefully Cloudposse will update the module we use, but it doesn't look like that's coming soon, given that they recently updated their tutorials to specify AWS < 5. (See cloudposse-archives/tutorials#21 (comment))
  • Loading branch information
Mr0grog committed Jun 14, 2023
1 parent 71f9ceb commit a79b44a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
3 changes: 3 additions & 0 deletions terraform/api-domains.tf
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,9 @@ module "univaf_data_snaphsots_cdn" {
var.domain_name != ""
&& var.ssl_certificate_arn != "" ? 1 : 0
)
# NOTE: If upgrading this module, please check whether it's now compatible
# with the current version of the AWS provider and upgrade that, too!
# See https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/issues/279
source = "cloudposse/cloudfront-s3-cdn/aws"
version = "0.90.0"

Expand Down
6 changes: 4 additions & 2 deletions terraform/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,10 @@ terraform {

required_providers {
aws = {
source = "hashicorp/aws"
version = "~> 4.52"
source = "hashicorp/aws"
# We use a module that is not yet compatible with v5 of the AWS provider:
# https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/issues/279
version = "~ 4.52"
}
}
}
Expand Down

0 comments on commit a79b44a

Please sign in to comment.