Skip to content

Commit 28719b1

Browse files
authored
fix: Do not try flushing validators when adding (#18102)
Fixes failing test in `deploy_l1_contracts` on v2
1 parent 7417051 commit 28719b1

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

yarn-project/ethereum/src/deploy_l1_contracts.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1262,7 +1262,7 @@ export const addMultipleValidators = async (
12621262
data: encodeFunctionData({
12631263
abi: MultiAdderArtifact.contractAbi,
12641264
functionName: 'addValidators',
1265-
args: [c, BigInt(chunkSize)],
1265+
args: [c, BigInt(0)],
12661266
}),
12671267
},
12681268
{
@@ -1281,15 +1281,18 @@ export const addMultipleValidators = async (
12811281
while (true) {
12821282
// If the queue is empty, we can break
12831283
if ((await rollup.getEntryQueueLength()) == 0n) {
1284+
logger.debug('Entry queue is empty, stopping flush attempts');
12841285
break;
12851286
}
12861287

12871288
// If there are no available validator flushes, no need to even try
12881289
if ((await rollup.getAvailableValidatorFlushes()) == 0n) {
1290+
logger.debug('No available validator flushes, stopping flush attempts');
12891291
break;
12901292
}
12911293

12921294
// Note that we are flushing at most `chunkSize` at each call
1295+
logger.debug(`Flushing entry queue for ${chunkSize} validators`);
12931296
await deployer.l1TxUtils.sendAndMonitorTransaction(
12941297
{
12951298
to: rollup.address,

0 commit comments

Comments
 (0)