Merge #122187

122187: roachtest: run rebalance/by-load/*/mixed-version on insecure mode r=DarrylWong a=kvoli Authentication fails if migrating from 22.2 to 23.1. Disable secure mode to avoid failing the test on this unrelated issue. Fixes: #122179 Fixes: #122084 Release note: None Co-authored-by: Austen McClernon <[email protected]>
cockroachdb · Apr 11, 2024 · c6c9461 · c6c9461
2 parents 8c40c18 + 7256197
commit c6c9461
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/pkg/cmd/roachtest/tests/rebalance_load.go b/pkg/cmd/roachtest/tests/rebalance_load.go
@@ -95,7 +95,15 @@ func registerRebalanceLoad(r registry.Registry) {
 
 		if mixedVersion {
 			mvt := mixedversion.NewTest(ctx, t, t.L(), c, roachNodes, mixedversion.NeverUseFixtures,
-				mixedversion.ClusterSettingOption(install.ClusterSettingsOption(settings.ClusterSettings)),
+				mixedversion.ClusterSettingOption(
+					install.ClusterSettingsOption(settings.ClusterSettings),
+					// In 23.1, the `user_id` field was added to `system.web_sessions`.
+					// If the cluster is migrating to 23.1, auth-session login will not
+					// be aware of this new field and authentication will fail.
+					// TODO(DarrylWong): When 22.2 is no longer supported, we won't run
+					// into the above issue anymore and can enable secure clusters.
+					install.SecureOption(false),
+				),
 			)
 			mvt.InMixedVersion("rebalance load run",
 				func(ctx context.Context, l *logger.Logger, r *rand.Rand, h *mixedversion.Helper) error {