add some fixes

hashicorp · Sep 18, 2023 · 83c2b19 · 83c2b19
1 parent 6e8e341
commit 83c2b19
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/modules/lambda-registrator/main.tf b/modules/lambda-registrator/main.tf
@@ -156,7 +156,7 @@ resource "aws_iam_role_policy_attachment" "lambda_logs" {
   policy_arn = aws_iam_policy.policy.arn
 }
 resource "random_id" "repo_id" {
-  byte_length = 8
+  byte_length = 4
 }
 
 resource "aws_ecr_repository" "lambda-registrator" {

diff --git a/test/acceptance/setup-terraform/ecr.tf b/test/acceptance/setup-terraform/ecr.tf
@@ -8,7 +8,6 @@ locals {
 }
 
 resource "aws_ecr_repository" "lambda-registrator" {
-  count        = var.enable_auto_publish_ecr_image ? 1 : 0
   name         = local.ecr_repository_name
   force_delete = true
 }

diff --git a/test/acceptance/tests/basic_test.go b/test/acceptance/tests/basic_test.go
@@ -67,7 +67,7 @@ func TestBasic(t *testing.T) {
 			namespace := ""
 			partition := ""
 			queryString := ""
-			tfVars["consul_image"] = "public.ecr.aws/hashicorp/consul:1.15.1"
+			tfVars["consul_image"] = "public.ecr.aws/hashicorp/consul:1.16.1"
 			if c.enterprise {
 				tfVars["consul_license"] = os.Getenv("CONSUL_LICENSE")
 				require.NotEmpty(t, tfVars["consul_license"], "CONSUL_LICENSE environment variable is required for enterprise tests")