Skip to content

Commit c5fc585

Browse files
committed
chore(e2e): try deflake high tps test
The timetable settings were too tight and were causing failures due to validators being too slow to attest. This PR changes the settings so we have longer times. Ultimately, the goal of this test is to check that a proposer will not overrun its time for building blocks causing validators to time out, so it is very time-dependant.
1 parent 20d8385 commit c5fc585

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

yarn-project/end-to-end/src/e2e_epochs/epochs_high_tps_block_building.test.ts

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,13 @@ jest.setTimeout(1000 * 60 * 10);
2323

2424
const NODE_COUNT = 3;
2525

26-
// We send 8 txs total, each taking 1s to process (see sequencerFakeDelayPerTxMs), with a total
27-
// L2 slot time of 16s, with l1PublishingTime set to the full L1 slot duration and attestationPropagationTime of 0.5.
28-
// This leaves us with roughly 2s for executing txs. This test will check that proposers honor the timetable
29-
// and do not try to include more than 2 txs per block. Should we ever implement preemptive block building,
26+
// We send 8 txs total, each taking several seconds to process (see sequencerFakeDelayPerTxMs), with a total
27+
// L2 slot time of 24s, with l1PublishingTime set to the full 8s L1 slot duration and attestationPropagationTime of 1s.
28+
// This leaves us with a few seconds for executing txs. This test will check that proposers honor the timetable
29+
// and do not try to include more than N txs per block. Should we ever implement preemptive block building,
3030
// sequencers will end up with more time, so we'll need to bump the EXPECTED_MAX_TXS_PER_BLOCK value.
3131
const TX_COUNT = 8;
32+
const TX_DURATION_MS = 2500;
3233
const EXPECTED_MAX_TXS_PER_BLOCK = 3;
3334

3435
// Test that sequencers and validators can handle a large backlog of transactions.
@@ -64,9 +65,9 @@ describe('e2e_epochs/epochs_high_tps_block_building', () => {
6465
enforceTimeTable: true,
6566
ethereumSlotDuration: 8,
6667
l1PublishingTime: 8,
67-
aztecSlotDuration: 16,
68-
fakeProcessingDelayPerTxMs: 850,
69-
attestationPropagationTime: 0.5,
68+
aztecSlotDuration: 24,
69+
fakeProcessingDelayPerTxMs: TX_DURATION_MS,
70+
attestationPropagationTime: 1,
7071
minTxsPerBlock: 1,
7172
maxTxsPerBlock: 100,
7273
});

0 commit comments

Comments
 (0)