sourcefuse / terraform-aws-arc-ecs

Repo for managing the ECS Fargate Terraform Module.
Apache License 2.0
1 stars 3 forks source link

added depends_on and updated the version #46

Closed mayank0202 closed 7 months ago

mayank0202 commented 7 months ago

Issue:

│ Error: modifying ELBv2 Listener (arn:aws:elasticloadbalancing:us-east-1:136566554811:listener/app/phoenix-phxdev-cluster/5166b7fec394512c/fbf24b2fd4423e18): UnsupportedCertificate: The certificate 'arn:aws:acm:us-east-1:136566554811:certificate/0fdb07c2-e072-4a56-8a67-75073497ee89' must have a fully-qualified domain name, a supported signature, and a supported key size.
│       status code: 400, request id: 77007e2d-c3d9-4921-b50f-c29eb47914d3
│
│   with module.ecs.aws_lb_listener.https,
│   on .terraform\modules\ecs\main.tf line 200, in resource "aws_lb_listener" "https":
│  200: resource "aws_lb_listener" "https" {
│

fix of issue:

  1. updated the 1-2 year old tag to the newest one of CP
  2. added a depends_on
  3. updated the hosted zone attribute to use vars instead of locals as it was expecting a new hosted zone whenever someone changes the prefix of acm
sonarcloud[bot] commented 7 months ago

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud