Change validation for max_concurrent_runs in databricjs_job to allow 0 value (#2682)

alexott · web-flow · commit a26000f38454 · 2023-09-12T16:49:52.000Z
Jobs API allows 0 value for `max_concurrent_runs` attribute [API docs](https://docs.databricks.com/api/workspace/jobs/create), but TF required to specify at least `1`. This fixes #2681
diff --git a/jobs/resource_job.go b/jobs/resource_job.go
@@ -662,7 +662,7 @@ var jobSchema = common.StructToSchema(JobSettings{},
 		if p, err := common.SchemaPath(s, "schedule", "pause_status"); err == nil {
 			p.ValidateFunc = validation.StringInSlice([]string{"PAUSED", "UNPAUSED"}, false)
 		}
-		s["max_concurrent_runs"].ValidateDiagFunc = validation.ToDiagFunc(validation.IntAtLeast(1))
+		s["max_concurrent_runs"].ValidateDiagFunc = validation.ToDiagFunc(validation.IntAtLeast(0))
 		s["max_concurrent_runs"].Default = 1
 		s["url"] = &schema.Schema{
 			Type:     schema.TypeString,

Original file line number	Diff line number	Diff line change
`@@ -662,7 +662,7 @@ var jobSchema = common.StructToSchema(JobSettings{},`
`662`	`662`	`if p, err := common.SchemaPath(s, "schedule", "pause_status"); err == nil {`
`663`	`663`	`p.ValidateFunc = validation.StringInSlice([]string{"PAUSED", "UNPAUSED"}, false)`
`664`	`664`	`}`
`665`		`- s["max_concurrent_runs"].ValidateDiagFunc = validation.ToDiagFunc(validation.IntAtLeast(1))`
	`665`	`+ s["max_concurrent_runs"].ValidateDiagFunc = validation.ToDiagFunc(validation.IntAtLeast(0))`
`666`	`666`	`s["max_concurrent_runs"].Default = 1`
`667`	`667`	`s["url"] = &schema.Schema{`
`668`	`668`	`Type: schema.TypeString,`