From 72561976277212c8fd11b3eb48f02de283b58092 Mon Sep 17 00:00:00 2001 From: Austen McClernon Date: Thu, 11 Apr 2024 09:29:27 -0400 Subject: [PATCH] roachtest: run rebalance/by-load/*/mixed-version on insecure mode 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 --- pkg/cmd/roachtest/tests/rebalance_load.go | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/pkg/cmd/roachtest/tests/rebalance_load.go b/pkg/cmd/roachtest/tests/rebalance_load.go index 61e9e5657bcb..624de93c67ad 100644 --- 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 {