allow only positive shutdown delay

client/allocrunner/alloc_runner_test.go

@@ -143,7 +143,6 @@ func TestAllocRunner_TaskLeader_KillTG(t *testing.T) {
 
 func TestAllocRunner_TaskGroup_ShutdownDelay(t *testing.T) {
 	t.Parallel()
-	shutdownDelay := 1 * time.Second
 
 	alloc := mock.Alloc()
 	tr := alloc.AllocatedResources.Tasks[alloc.Job.TaskGroups[0].Tasks[0].Name]
@@ -170,6 +169,7 @@ func TestAllocRunner_TaskGroup_ShutdownDelay(t *testing.T) {
 	alloc.AllocatedResources.Tasks[task2.Name] = tr
 
 	// Set a shutdown delay
+	shutdownDelay := 1 * time.Second
 	alloc.Job.TaskGroups[0].ShutdownDelay = &shutdownDelay
 
 	conf, cleanup := testAllocRunnerConfig(t, alloc)

client/allocrunner/interfaces/runner_lifecycle.go

@@ -16,6 +16,15 @@ type RunnerPrerunHook interface {
 	Prerun() error
 }
 
+// RunnerPreKillHooks are executed inside of KillTasks before
+// iterating and killing each task. It will run before the Leader
+// task is killed.
+type RunnerPreKillHook interface {
+	RunnerHook
+
+	PreKill()
+}
+
 // RunnerPostrunHooks are executed after calling TaskRunner.Run, even for
 // terminal allocations. Therefore Postrun hooks must be safe to call without
 // first calling Prerun hooks.
@@ -53,10 +62,3 @@ type ShutdownHook interface {
 
 	Shutdown()
 }
-
-//
-type RunnerPreKillHook interface {
-	RunnerHook
-
-	PreKill()
-}

nomad/structs/structs.go

@@ -3547,9 +3547,9 @@ func (j *Job) Validate() error {
 			taskGroups[tg.Name] = idx
 		}
 
-		// if tg.ShutdownDelay < 0 {
-		// 	mErr.Errors = append(mErr.Errors, errors.New("ShutdownDelay must be a positive value"))
-		// }
+		if tg.ShutdownDelay != nil && *tg.ShutdownDelay < 0 {
+			mErr.Errors = append(mErr.Errors, errors.New("ShutdownDelay must be a positive value"))
+		}
 
 		if j.Type == "system" && tg.Count > 1 {
 			mErr.Errors = append(mErr.Errors,