roachtest/mixedversion: use minimumBootstrapVersion instead of maxUpgrades

Previously, mixed versions tests that required the cluster to be started
within a certain version would set the MaxUpgrades option. This works
but will rot every time we advance versions. Instead this change switches
those tests to use MinimumBootstrapVersion instead.

Additionally the http-register-routes/mixed-versions test
now sets the MinBootstrapVersion to 24.2, the oldest release that
has the fix contained in #138342

Author: DarrylWong

pkg/cmd/roachtest/tests/admission_control_elastic_mixed_version.go

@@ -56,8 +56,7 @@ func registerElasticWorkloadMixedVersion(r registry.Registry) {
 				mixedversion.AlwaysUseLatestPredecessors,
 				// Don't go back too far. We are mostly interested in upgrading to v24.3
 				// where RACv2 was introduced.
-				mixedversion.MaxUpgrades(2),
-				mixedversion.MinimumSupportedVersion("v24.1.0"),
+				mixedversion.MinimumBootstrapVersion("v24.1.0"),
 			)
 
 			// Limit the disk throughput to 128 MiB/s, to more easily stress the

pkg/cmd/roachtest/tests/validate_system_schema_after_version_upgrade.go

@@ -190,9 +190,8 @@ func runValidateSystemSchemaAfterVersionUpgrade(
 	ctx context.Context, t test.Test, c cluster.Cluster,
 ) {
 	validateSystemSchemaAfterUpgradeTest(ctx, t, c,
-		// We limit the number of upgrades since the test is not expected to work
-		// on versions older than 22.2.
-		mixedversion.MaxUpgrades(3),
+		// The test is not expected to work on versions older than 22.2.
+		mixedversion.MinimumBootstrapVersion("v22.2.0"),
 		// Fixtures are generated on a version that's too old for this test.
 		mixedversion.NeverUseFixtures,
 		// Separate-process deployments can't run in 1-node clusters since
@@ -212,9 +211,8 @@ func runValidateSystemSchemaAfterVersionUpgradeSeparateProcess(
 	ctx context.Context, t test.Test, c cluster.Cluster,
 ) {
 	validateSystemSchemaAfterUpgradeTest(ctx, t, c,
-		// We limit the number of upgrades since the test is not expected to work
-		// on versions older than 22.2.
-		mixedversion.MaxUpgrades(3),
+		// The test is not expected to work on versions older than 22.2.
+		mixedversion.MinimumBootstrapVersion("v22.2.0"),
 		// Fixtures are generated on a version that's too old for this test.
 		mixedversion.NeverUseFixtures,
 		mixedversion.EnabledDeploymentModes(mixedversion.SeparateProcessDeployment),

pkg/cmd/roachtest/tests/versionupgrade.go

@@ -313,6 +313,12 @@ func runHTTPRestart(ctx context.Context, t test.Test, c cluster.Cluster) {
 	mvt := mixedversion.NewTest(ctx, t, t.L(), c,
 		c.CRDBNodes(),
 		mixedversion.AlwaysUseLatestPredecessors,
+		// We set the min bootstrap version to v24.2, as the fix for the
+		// race condition was only backported to v24.2+ but exists as early
+		// as v23.2. We use this over setting min supported version as this
+		// test is concerned about testing cluster startup. We don't want the
+		// framework to bootstrap the cluster before we can start running hooks.
+		mixedversion.MinimumBootstrapVersion("v24.2.0"),
 	)
 
 	// Any http request requiring auth will do.