Skip to content

Commit 28c00b4

Browse files
committed
Misc clean up
1 parent 17ce833 commit 28c00b4

File tree

6 files changed

+6
-39
lines changed

6 files changed

+6
-39
lines changed

.gitlab-ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ tests:
1212

1313
# Setup Terraform
1414
- curl -fsSL https://releases.hashicorp.com/terraform/1.6.6/terraform_1.6.6_linux_amd64.zip -o terraform.zip
15-
- unzip terraform.zip
15+
- unzip terraform.zip > /dev/null
1616
- mv terraform /usr/local/bin/terraform
1717
- chmod +x /usr/local/bin/terraform
1818
- terraform version

modules/ecs_fargate/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ resource "aws_ecs_task_definition" "this" {
3939

4040
family = var.family
4141

42-
# Fargate incompatible parameter
42+
# Fargate incompatible parameter on v6.0.0
4343
# dynamic "inference_accelerator" {
4444
# for_each = var.inference_accelerator != null ? var.inference_accelerator : []
4545

modules/ecs_fargate/outputs.tf

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,7 @@ output "family" {
3535

3636
output "inference_accelerator" {
3737
description = "Inference accelerator settings."
38-
# value = aws_ecs_task_definition.this.inference_accelerator
39-
value = []
38+
value = null
4039
}
4140

4241
output "ipc_mode" {

smoke_tests/ecs_fargate/null.tf

Lines changed: 0 additions & 25 deletions
This file was deleted.
Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
provider "aws" {
2-
region = "us-east-1"
3-
retry_mode = "standard"
4-
max_retries = 10
2+
region = "us-east-1"
53
}

tests/main_test.go

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,17 +49,12 @@ func (s *ECSFargateSuite) SetupSuite() {
4949
"test_prefix": s.testPrefix,
5050
},
5151
RetryableTerraformErrors: map[string]string{
52-
"couldn't find resource": "terratest could not find the resource",
52+
"couldn't find resource": "terratest could not find the resource. check for access denied errors in cloudtrial",
5353
},
54-
// NoColor: true,
55-
// MaxRetries: 0,
56-
// TimeBetweenRetries: 10 * time.Second,
57-
// Lock: true,
5854
}
5955

6056
// Run terraform init and apply
61-
terraform.Init(s.T(), s.terraformOptions)
62-
terraform.Apply(s.T(), s.terraformOptions)
57+
terraform.InitAndApply(s.T(), s.terraformOptions)
6358
}
6459

6560
// TearDownSuite is run once at the end of the test suite

0 commit comments

Comments
 (0)