e2e: fix and modernize rescheduling test

The E2E test suite for rescheduling had a few bugs:
* Using the command line to stop a job with a failing deployment returns a non-zero exit
  code, which would cause an otherwise passing test to fail.
* Two of the input jobs were actually invalid but were only correctly detected
  as such because of #17342

This changeset also updates the whole test suite to move it off the v1
"framework". The test assertions are otherwise left untouched.

e2e/e2e_test.go

@@ -31,7 +31,6 @@ import (
 	_ "github.com/hashicorp/nomad/e2e/podman"
 	_ "github.com/hashicorp/nomad/e2e/quotas"
 	_ "github.com/hashicorp/nomad/e2e/remotetasks"
-	_ "github.com/hashicorp/nomad/e2e/rescheduling"
 	_ "github.com/hashicorp/nomad/e2e/scaling"
 	_ "github.com/hashicorp/nomad/e2e/scalingpolicies"
 	_ "github.com/hashicorp/nomad/e2e/scheduler_sysbatch"
@@ -45,6 +44,7 @@ import (
 	_ "github.com/hashicorp/nomad/e2e/disconnectedclients"
 	_ "github.com/hashicorp/nomad/e2e/namespaces"
 	_ "github.com/hashicorp/nomad/e2e/nodedrain"
+	_ "github.com/hashicorp/nomad/e2e/rescheduling"
 	_ "github.com/hashicorp/nomad/e2e/volumes"
 )
 

e2e/rescheduling/doc.go

@@ -0,0 +1,7 @@
+// Copyright (c) HashiCorp, Inc.
+// SPDX-License-Identifier: BUSL-1.1
+
+package rescheduling
+
+// This package contains only tests, so this is a placeholder file to
+// make sure builds don't fail with "no non-test Go files in" errors

e2e/rescheduling/input/rescheduling_canary_autorevert.nomad

@@ -30,7 +30,7 @@ job "test" {
       min_healthy_time  = "1s"
       auto_revert       = true
       healthy_deadline  = "2s"
-      progress_deadline = "3s"
+      progress_deadline = "5s"
     }
 
     restart {

e2e/rescheduling/input/rescheduling_maxp_autorevert.nomad

@@ -29,7 +29,7 @@ job "demo3" {
       min_healthy_time  = "1s"
       auto_revert       = true
       healthy_deadline  = "2s"
-      progress_deadline = "3s"
+      progress_deadline = "5s"
     }
 
     restart {