Skip to content

Commit

Permalink
op-challenger: Fix run-trace to only run mt-cannon in parallel with c…
Browse files Browse the repository at this point in the history
…annon traces. (ethereum-optimism#12052)

Previously it was also running it in parallel with kona-asterisc traces, resulting in errors because there were two mt-cannon traces sharing a preimages directory.
  • Loading branch information
ajsutton authored and samlaf committed Nov 10, 2024
1 parent 26ca65a commit 79ae7e1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion op-challenger/runner/runner.go
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ func (r *Runner) runAndRecordOnce(ctx context.Context, traceType types.TraceType
recordError(err, traceType.String(), r.m, r.log)
}()

if r.addMTCannonPrestate != (common.Hash{}) && r.addMTCannonPrestateURL != nil {
if traceType == types.TraceTypeCannon && r.addMTCannonPrestate != (common.Hash{}) && r.addMTCannonPrestateURL != nil {
wg.Add(1)
go func() {
defer wg.Done()
Expand Down

0 comments on commit 79ae7e1

Please sign in to comment.