Skip to content

Commit

Permalink
Merge #133087
Browse files Browse the repository at this point in the history
133087: roachtest: add gctrace to perturbation/* tests r=sumeerbhola a=andrewbaptist

We have had at least one perturbation test failure where the cause appears to be a ~2s process freeze. We don't have conclusive evidence that this is a GOGC rather than some other type of infra flake, but at least this can help rule out one more potential cause.

Informs: #131822

Epic: none

Release note: None

Co-authored-by: Andrew Baptist <[email protected]>
  • Loading branch information
craig[bot] and andrewbaptist committed Oct 21, 2024
2 parents 9cbb2d3 + 0175fe5 commit 1269c37
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/cmd/roachtest/tests/admission_control_latency.go
Original file line number Diff line number Diff line change
Expand Up @@ -924,7 +924,8 @@ func (v variations) startNoBackup(
opts.RoachprodOpts.ExtraArgs = append(opts.RoachprodOpts.ExtraArgs,
fmt.Sprintf("--locality=region=fake-%d", (node-1)/nodesPerRegion))
opts.RoachprodOpts.ExtraArgs = append(opts.RoachprodOpts.ExtraArgs, extraArgs...)
v.Start(ctx, t.L(), opts, install.MakeClusterSettings(), v.Node(node))
settings := install.MakeClusterSettings(install.EnvOption([]string{"GODEBUG=gctrace=1"}))
v.Start(ctx, t.L(), opts, settings, v.Node(node))
}
}

Expand Down

0 comments on commit 1269c37

Please sign in to comment.