Skip to content

Commit e7f1385

Browse files
authored
fix(l2): revert breaking when budget is reached for privileged txs (#5326)
**Motivation** [This](#5135 (comment)) change made the integration tests took longer, having to amp the amount of retries. <!-- Why does this pull request exist? What are its goals? --> **Description** Reverts this change and adds an [issue](#5325) for solving it. <!-- A clear and concise general description of the changes this PR introduces --> <!-- Link to issues: Resolves #111, Resolves #222 -->
1 parent 27be096 commit e7f1385

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

crates/l2/sequencer/block_producer/payload_builder.rs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -163,10 +163,8 @@ pub async fn fill_transactions(
163163
if head_tx.is_privileged() {
164164
if privileged_tx_count >= PRIVILEGED_TX_BUDGET {
165165
debug!("Ran out of space for privileged transactions");
166-
// We break here because if we have expired privileged transactions
167-
// in the contract, our batch will be rejected if non-privileged txs
168-
// are included.
169-
break;
166+
txs.pop();
167+
continue;
170168
}
171169
let id = head_tx.nonce();
172170
if last_privileged_nonce.is_some_and(|last_nonce| id != last_nonce + 1) {

0 commit comments

Comments
 (0)