Skip to content

Commit

Permalink
roachtest: assign adminui ports dynamically for virtual clusters
Browse files Browse the repository at this point in the history
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
  • Loading branch information
DarrylWong committed Jan 8, 2024
1 parent 26c8522 commit 9605fea
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/cmd/roachtest/option/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
}

Expand Down

0 comments on commit 9605fea

Please sign in to comment.