clientIO / appmixer-module-aws

Appmixer Infrastructure-as-Code Terraform module for AWS.
Apache License 2.0
0 stars 0 forks source link

bug: `terraform plan` fails for production example #1

Closed jirihofman closed 10 months ago

jirihofman commented 10 months ago

Summary

I wanted to check the plan for production with terraform plan in the examples.

Issue Type

Bug Report

Terraform Version

terraform --version
Terraform v1.7.0
on darwin_amd64
+ provider registry.terraform.io/cloudposse/template v2.2.0
+ provider registry.terraform.io/cloudposse/utils v1.15.0
+ provider registry.terraform.io/hashicorp/aws v5.33.0
+ provider registry.terraform.io/hashicorp/http v3.4.1
+ provider registry.terraform.io/hashicorp/local v2.4.1
+ provider registry.terraform.io/hashicorp/null v3.2.2
+ provider registry.terraform.io/hashicorp/random v3.6.0
+ provider registry.terraform.io/hashicorp/time v0.10.0

Your version of Terraform is out of date! The latest version
is 1.7.1. You can update by downloading from https://www.terraform.io/downloads.html

Steps to Reproduce

cd examples/production
appmixer plan

Expected Results

No error

Actual Results

...
...
...

Plan: 187 to add, 0 to change, 0 to destroy.
╷
│ Warning: Argument is deprecated
│ 
│   with module.appmixer_module.module.document_db_ssm_password.aws_ssm_parameter.default["/cio-prod-appmixer/documentdb/master_password"],
│   on .terraform/modules/appmixer_module.document_db_ssm_password/main.tf line 22, in resource "aws_ssm_parameter" "default":
│   22:   overwrite       = each.value.overwrite
│ 
│ this attribute has been deprecated
│ 
│ (and 6 more similar warnings elsewhere)
╵
╷
│ Error: Error in function call
│ 
│   on ../../ecs-ec2.tf line 52, in locals:
│   52:         ECS_ENGINE_AUTH_DATA=${sensitive(base64decode(var.ecs_registry_auth_data))} # pragma: allowlist secret
│     ├────────────────
│     │ while calling base64decode(str)
│     │ var.ecs_registry_auth_data is "XXX"
│ 
│ Call to function "base64decode" failed: failed to decode base64 data "XXX".
╵
Riqardos commented 10 months ago

Fixed in PR