Commit 4ca169c0 authored by OptimismBot's avatar OptimismBot Committed by GitHub

Merge pull request #6433 from ethereum-optimism/feat/e2e-genesis-error-handling

op-e2e: faster failures on bad genesis
parents 0ecda15d 2e2dd25f
...@@ -17,6 +17,7 @@ import ( ...@@ -17,6 +17,7 @@ import (
"github.com/ethereum-optimism/optimism/op-node/rollup" "github.com/ethereum-optimism/optimism/op-node/rollup"
"github.com/ethereum-optimism/optimism/op-node/rollup/derive" "github.com/ethereum-optimism/optimism/op-node/rollup/derive"
"github.com/ethereum-optimism/optimism/op-node/rollup/driver" "github.com/ethereum-optimism/optimism/op-node/rollup/driver"
"github.com/ethereum-optimism/optimism/op-node/rollup/sync"
"github.com/ethereum-optimism/optimism/op-node/sources" "github.com/ethereum-optimism/optimism/op-node/sources"
"github.com/ethereum-optimism/optimism/op-node/testutils" "github.com/ethereum-optimism/optimism/op-node/testutils"
) )
...@@ -215,6 +216,9 @@ func (s *L2Verifier) ActL2PipelineStep(t Testing) { ...@@ -215,6 +216,9 @@ func (s *L2Verifier) ActL2PipelineStep(t Testing) {
return return
} else if err != nil && errors.Is(err, derive.ErrTemporary) { } else if err != nil && errors.Is(err, derive.ErrTemporary) {
s.log.Warn("Derivation process temporary error", "err", err) s.log.Warn("Derivation process temporary error", "err", err)
if errors.Is(err, sync.WrongChainErr) { // action-tests don't back off on temporary errors. Avoid a bad genesis setup from looping.
t.Fatalf("genesis setup issue: %v", err)
}
return return
} else if err != nil && errors.Is(err, derive.ErrCritical) { } else if err != nil && errors.Is(err, derive.ErrCritical) {
t.Fatalf("derivation failed critically: %v", err) t.Fatalf("derivation failed critically: %v", err)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment