Skip to content

Commit 386b3da

Browse files
craig[bot]fqazi
craig[bot]
andcommitted
Merge #131663
131663: roachtest/large-schema-benchmark: use SSL for web console API r=fqazi a=fqazi Previously, we have been noticing flakes lately on certain scales of this test due to the end point giving connection refused, and TLS errors in the log files. To address this we are going to switch to the HTTPS end point, which should hopefully help with the flake we have been seeing. Fixes: #131566 Fixes: #131556 Fixes: #131554 Release note: None Co-authored-by: Faizan Qazi <[email protected]>
2 parents 9c56d94 + 6ad68d4 commit 386b3da

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: pkg/cmd/roachtest/tests/large_schema_benchmark.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ func registerLargeSchemaBenchmark(r registry.Registry, numTables int, isMultiReg
192192
webConsoleURLs, err := c.ExternalAdminUIAddr(ctx, t.L(), c.Range(1, c.Spec().NodeCount-1))
193193
require.NoError(t, err)
194194
for urlIdx := range webConsoleURLs {
195-
webConsoleURLs[urlIdx] = "http://" + webConsoleURLs[urlIdx]
195+
webConsoleURLs[urlIdx] = "https://" + webConsoleURLs[urlIdx]
196196
}
197197
// Next startup the workload for our list of databases from earlier.
198198
for dbListType, dbList := range [][]string{activeDBList, inactiveDBList} {

0 commit comments

Comments
 (0)