Commit 4ee839ae authored by Matthew Slipper's avatar Matthew Slipper Committed by GitHub

op-batcher: Fix test flake due to log (#13145)

There was one other spot where the batcher would log after tests exit, causing a panic.
parent b6c28d5d
......@@ -779,12 +779,14 @@ func (l *BatchSubmitter) publishToAltDAAndL1(txdata txData, queue *txmgr.Queue[t
if err != nil {
// Don't log context cancelled events because they are expected,
// and can happen after tests complete which causes a panic.
if !errors.Is(err, context.Canceled) {
if errors.Is(err, context.Canceled) {
l.recordFailedDARequest(txdata.ID(), nil)
} else {
l.Log.Error("Failed to post input to Alt DA", "error", err)
// requeue frame if we fail to post to the DA Provider so it can be retried
// note: this assumes that the da server caches requests, otherwise it might lead to resubmissions of the blobs
l.recordFailedDARequest(txdata.ID(), err)
}
// requeue frame if we fail to post to the DA Provider so it can be retried
// note: this assumes that the da server caches requests, otherwise it might lead to resubmissions of the blobs
l.recordFailedDARequest(txdata.ID(), err)
return nil
}
l.Log.Info("Set altda input", "commitment", comm, "tx", txdata.ID())
......
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