From 9605feac15ea06596c269815bfec367840226e70 Mon Sep 17 00:00:00 2001 From: DarrylWong Date: Mon, 8 Jan 2024 19:37:10 +0000 Subject: [PATCH] roachtest: assign adminui ports dynamically for virtual clusters This was originally removed in #115599 due to #114097 merging, but adminui was reverted in #117141 and mistakenly did not revert the special case for virtual clusters. Release note: None --- pkg/cmd/roachtest/option/options.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pkg/cmd/roachtest/option/options.go b/pkg/cmd/roachtest/option/options.go index 3d24df4e9a79..5fa9453986c5 100644 --- a/pkg/cmd/roachtest/option/options.go +++ b/pkg/cmd/roachtest/option/options.go @@ -63,6 +63,8 @@ func DefaultStartVirtualClusterOpts(tenantName string, sqlInstance int) StartOpt startOpts.RoachprodOpts.Target = install.StartServiceForVirtualCluster startOpts.RoachprodOpts.VirtualClusterName = tenantName startOpts.RoachprodOpts.SQLInstance = sqlInstance + // TODO(DarrylWong): remove once #117125 is addressed. + startOpts.RoachprodOpts.AdminUIPort = 0 return startOpts }