Updating the json path for db instance name retrieval

It seems that the structure of the terraform state has changed since terraform v0.12.x. We've updated the json path to ensure that the system test can retrieve the RDS database instance identifier. Co-authored-by: Nick Ruffles <[email protected]> Co-authored-by: Lilly Daniell <[email protected]> Co-authored-by: Dominic Meddick <[email protected]>
EngineerBetter · Nov 15, 2022 · 767b832 · 767b832
1 parent ef9784c
commit 767b832
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/ci/tasks/system-test-rds-ca.sh b/ci/tasks/system-test-rds-ca.sh
@@ -28,7 +28,7 @@ domain=$(echo "$config" | jq -r '.config.domain')
 
 echo "Changing to new cert"
 aws --region "$region" s3 cp "s3://control-tower-$deployment-$region-config/terraform.tfstate" terraform.tfstate
-db_identifier="$(jq -r '.modules[0].resources."aws_db_instance.default".primary.id' < terraform.tfstate)"
+db_identifier="$(jq -r '.resources[] | select( .type == "aws_db_instance") | .instances[0].attributes.id' < terraform.tfstate)"
 aws --region "$region" rds modify-db-instance --db-instance-identifier "$db_identifier" --ca-certificate-identifier rds-ca-2019 --apply-immediately
 
 echo "Waiting for CA cert to change"