Skip to content

Commit be29b6b

Browse files
committed
roachtest: fix client retry token bucket setting
Previously, backup fixtures were failing to generate due to a misnamed setting. The setting was originally correct, but became incorrect when I renamed the setting to fix the setting name lint. We don't have to backport this because the roachtest change hit a merge conflict during the backport and I removed the roachtest changes from the backport since they were non-essential. Release note: none Informs: #148334
1 parent d5e3063 commit be29b6b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/cmd/roachtest/tests/backup_fixtures.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -188,9 +188,9 @@ func (bd *backupDriver) prepareCluster(ctx context.Context) {
188188
// there is a snapshot backlog, which makes the snapshot backlog worse
189189
// because add sst causes ranges to fall behind and need recovery snapshots
190190
// to catch up.
191-
"kv.snapshot_rebalance.max_rate": "256 MiB",
192-
"server.debug.default_vmodule": "s3_storage=2",
193-
"cloudstorage.s3.enable_client_retry_token_bucket": "false",
191+
"kv.snapshot_rebalance.max_rate": "256 MiB",
192+
"server.debug.default_vmodule": "s3_storage=2",
193+
"cloudstorage.s3.client_retry_token_bucket.enabled": "false",
194194
},
195195
install.EnvOption{
196196
fmt.Sprintf("COCKROACH_AZURE_APPLICATION_CREDENTIALS_FILE=%s", azureCredentialsFilePath),

0 commit comments

Comments
 (0)