Merge pull request hashicorp#36458 from bschaatsbergen/f/fix-validate…

…-jobs-glue chore: allow a minimum of 1 worker for a Glue Job
mattburgess · Mar 19, 2024 · 6ec1be6 · 6ec1be6
2 parents 2c9b6e9 + 1184a16
commit 6ec1be6
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/.changelog/36458.txt b/.changelog/36458.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+resource/aws_glue_job: Adjust `number_of_workers` minimum value to `1`
+```
diff --git a/internal/service/glue/job.go b/internal/service/glue/job.go
@@ -154,7 +154,7 @@ func ResourceJob() *schema.Resource {
  Type: schema.TypeInt,
  Optional: true,
  ConflictsWith: []string{"max_capacity"},
- ValidateFunc: validation.IntAtLeast(2),
+ ValidateFunc: validation.IntAtLeast(1),
  },
  "role_arn": {
  Type: schema.TypeString,

diff --git a/internal/service/glue/job_test.go b/internal/service/glue/job_test.go
@@ -976,7 +976,7 @@ func testAccJobConfig_versionNumberOfWorkers(rName, glueVersion string) string {
 resource "aws_glue_job" "test" {
  glue_version = %[1]q
  name = %[2]q
- number_of_workers = 2
+ number_of_workers = 1
  role_arn = aws_iam_role.test.arn
  worker_type = "Standard"
 
@@ -1102,7 +1102,7 @@ func testAccJobConfig_tags1(rName, tagKey1, tagValue1 string) string {
  return acctest.ConfigCompose(testAccJobConfig_base(rName), fmt.Sprintf(`
 resource "aws_glue_job" "test" {
  name = %[1]q
- number_of_workers = 2
+ number_of_workers = 1
  role_arn = aws_iam_role.test.arn
  worker_type = "Standard"
 
@@ -1123,7 +1123,7 @@ func testAccJobConfig_tags2(rName, tagKey1, tagValue1, tagKey2, tagValue2 string
  return acctest.ConfigCompose(testAccJobConfig_base(rName), fmt.Sprintf(`
 resource "aws_glue_job" "test" {
  name = %[1]q
- number_of_workers = 2
+ number_of_workers = 1
  role_arn = aws_iam_role.test.arn
  worker_type = "Standard"