@aztec/sequencer-client 0.0.1-commit.b33fc05d0 → 0.0.1-commit.b3d3157a
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +281 -21
- package/dest/client/sequencer-client.d.ts +8 -14
- package/dest/client/sequencer-client.d.ts.map +1 -1
- package/dest/client/sequencer-client.js +33 -83
- package/dest/config.d.ts +5 -1
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +42 -15
- package/dest/global_variable_builder/fee_predictor.d.ts +37 -0
- package/dest/global_variable_builder/fee_predictor.d.ts.map +1 -0
- package/dest/global_variable_builder/fee_predictor.js +128 -0
- package/dest/global_variable_builder/fee_provider.d.ts +21 -0
- package/dest/global_variable_builder/fee_provider.d.ts.map +1 -0
- package/dest/global_variable_builder/fee_provider.js +58 -0
- package/dest/global_variable_builder/global_builder.d.ts +14 -14
- package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
- package/dest/global_variable_builder/global_builder.js +16 -50
- package/dest/global_variable_builder/index.d.ts +4 -2
- package/dest/global_variable_builder/index.d.ts.map +1 -1
- package/dest/global_variable_builder/index.js +2 -0
- package/dest/index.d.ts +2 -2
- package/dest/index.d.ts.map +1 -1
- package/dest/index.js +1 -1
- package/dest/publisher/config.d.ts +15 -3
- package/dest/publisher/config.d.ts.map +1 -1
- package/dest/publisher/config.js +19 -4
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +3 -4
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
- package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
- package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
- package/dest/publisher/sequencer-bundle-simulator.js +198 -0
- package/dest/publisher/sequencer-publisher-factory.d.ts +3 -5
- package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher-factory.js +2 -3
- package/dest/publisher/sequencer-publisher.d.ts +73 -65
- package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher.js +318 -523
- package/dest/sequencer/automine/automine_factory.d.ts +54 -0
- package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
- package/dest/sequencer/automine/automine_factory.js +84 -0
- package/dest/sequencer/automine/automine_sequencer.d.ts +151 -0
- package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
- package/dest/sequencer/automine/automine_sequencer.js +509 -0
- package/dest/sequencer/chain_state_overrides.d.ts +61 -0
- package/dest/sequencer/chain_state_overrides.d.ts.map +1 -0
- package/dest/sequencer/chain_state_overrides.js +98 -0
- package/dest/sequencer/checkpoint_proposal_job.d.ts +60 -10
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_proposal_job.js +748 -200
- package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
- package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
- package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
- package/dest/sequencer/checkpoint_voter.d.ts +1 -2
- package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_voter.js +2 -5
- package/dest/sequencer/events.d.ts +48 -2
- package/dest/sequencer/events.d.ts.map +1 -1
- package/dest/sequencer/index.d.ts +3 -1
- package/dest/sequencer/index.d.ts.map +1 -1
- package/dest/sequencer/index.js +2 -0
- package/dest/sequencer/metrics.d.ts +13 -10
- package/dest/sequencer/metrics.d.ts.map +1 -1
- package/dest/sequencer/metrics.js +45 -20
- package/dest/sequencer/sequencer.d.ts +63 -13
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +349 -107
- package/dest/sequencer/timetable.d.ts +11 -1
- package/dest/sequencer/timetable.d.ts.map +1 -1
- package/dest/sequencer/timetable.js +41 -36
- package/dest/sequencer/types.d.ts +2 -2
- package/dest/sequencer/types.d.ts.map +1 -1
- package/dest/test/mock_checkpoint_builder.d.ts +4 -4
- package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
- package/dest/test/utils.d.ts +1 -1
- package/dest/test/utils.d.ts.map +1 -1
- package/dest/test/utils.js +7 -6
- package/package.json +27 -28
- package/src/client/sequencer-client.ts +53 -114
- package/src/config.ts +48 -12
- package/src/global_variable_builder/README.md +44 -0
- package/src/global_variable_builder/fee_predictor.ts +172 -0
- package/src/global_variable_builder/fee_provider.ts +75 -0
- package/src/global_variable_builder/global_builder.ts +25 -62
- package/src/global_variable_builder/index.ts +3 -1
- package/src/index.ts +10 -1
- package/src/publisher/config.ts +40 -6
- package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +3 -1
- package/src/publisher/sequencer-bundle-simulator.ts +254 -0
- package/src/publisher/sequencer-publisher-factory.ts +3 -6
- package/src/publisher/sequencer-publisher.ts +377 -568
- package/src/sequencer/README.md +162 -450
- package/src/sequencer/automine/README.md +46 -0
- package/src/sequencer/automine/automine_factory.ts +145 -0
- package/src/sequencer/automine/automine_sequencer.ts +595 -0
- package/src/sequencer/chain_state_overrides.ts +162 -0
- package/src/sequencer/checkpoint_proposal_job.ts +894 -220
- package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
- package/src/sequencer/checkpoint_voter.ts +1 -12
- package/src/sequencer/events.ts +51 -2
- package/src/sequencer/index.ts +2 -0
- package/src/sequencer/metrics.ts +57 -24
- package/src/sequencer/sequencer.ts +429 -121
- package/src/sequencer/timetable.ts +49 -44
- package/src/sequencer/types.ts +1 -1
- package/src/test/mock_checkpoint_builder.ts +3 -3
- package/src/test/utils.ts +28 -10
|
@@ -1,4 +1,5 @@
|
|
|
1
1
|
import type { EpochCache } from '@aztec/epoch-cache';
|
|
2
|
+
import type { SimulationOverridesPlan } from '@aztec/ethereum/contracts';
|
|
2
3
|
import {
|
|
3
4
|
BlockNumber,
|
|
4
5
|
CheckpointNumber,
|
|
@@ -13,11 +14,12 @@ import {
|
|
|
13
14
|
generateUnrecoverableSignature,
|
|
14
15
|
} from '@aztec/foundation/crypto/secp256k1-signer';
|
|
15
16
|
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
17
|
+
import { InterruptError, TimeoutError } from '@aztec/foundation/error';
|
|
16
18
|
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
17
19
|
import { Signature } from '@aztec/foundation/eth-signature';
|
|
18
20
|
import { filter } from '@aztec/foundation/iterator';
|
|
19
21
|
import { type Logger, type LoggerBindings, createLogger } from '@aztec/foundation/log';
|
|
20
|
-
import {
|
|
22
|
+
import { InterruptibleSleep } from '@aztec/foundation/sleep';
|
|
21
23
|
import { type DateProvider, Timer } from '@aztec/foundation/timer';
|
|
22
24
|
import { type TypedEventEmitter, isErrorClass, unfreeze } from '@aztec/foundation/types';
|
|
23
25
|
import type { P2P } from '@aztec/p2p';
|
|
@@ -29,18 +31,32 @@ import {
|
|
|
29
31
|
type L2BlockSink,
|
|
30
32
|
type L2BlockSource,
|
|
31
33
|
MaliciousCommitteeAttestationsAndSigners,
|
|
34
|
+
type ProposedCheckpointSink,
|
|
35
|
+
type ValidateCheckpointResult,
|
|
32
36
|
} from '@aztec/stdlib/block';
|
|
33
|
-
import {
|
|
34
|
-
|
|
37
|
+
import {
|
|
38
|
+
type Checkpoint,
|
|
39
|
+
type ProposedCheckpointData,
|
|
40
|
+
getPreviousCheckpointOutHashes,
|
|
41
|
+
validateCheckpoint,
|
|
42
|
+
} from '@aztec/stdlib/checkpoint';
|
|
43
|
+
import { computeQuorum, getSlotStartBuildTimestamp, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
|
|
35
44
|
import { Gas } from '@aztec/stdlib/gas';
|
|
36
45
|
import {
|
|
37
|
-
|
|
38
|
-
|
|
46
|
+
type BlockBuilderOptions,
|
|
47
|
+
InsufficientValidTxsError,
|
|
39
48
|
type ResolvedSequencerConfig,
|
|
40
49
|
type WorldStateSynchronizer,
|
|
41
50
|
} from '@aztec/stdlib/interfaces/server';
|
|
42
51
|
import { type L1ToL2MessageSource, computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
|
|
43
|
-
import type {
|
|
52
|
+
import type {
|
|
53
|
+
BlockProposal,
|
|
54
|
+
BlockProposalOptions,
|
|
55
|
+
CheckpointAttestation,
|
|
56
|
+
CheckpointProposal,
|
|
57
|
+
CheckpointProposalOptions,
|
|
58
|
+
CoordinationSignatureContext,
|
|
59
|
+
} from '@aztec/stdlib/p2p';
|
|
44
60
|
import { orderAttestations, trimAttestations } from '@aztec/stdlib/p2p';
|
|
45
61
|
import type { L2BlockBuiltStats } from '@aztec/stdlib/stats';
|
|
46
62
|
import { type FailedTx, Tx } from '@aztec/stdlib/tx';
|
|
@@ -51,6 +67,8 @@ import { DutyAlreadySignedError, SlashingProtectionError } from '@aztec/validato
|
|
|
51
67
|
|
|
52
68
|
import type { GlobalVariableBuilder } from '../global_variable_builder/global_builder.js';
|
|
53
69
|
import type { InvalidateCheckpointRequest, SequencerPublisher } from '../publisher/sequencer-publisher.js';
|
|
70
|
+
import { buildCheckpointSimulationOverridesPlan } from './chain_state_overrides.js';
|
|
71
|
+
import type { CheckpointProposalJobMetricsRecorder } from './checkpoint_proposal_job_metrics.js';
|
|
54
72
|
import { CheckpointVoter } from './checkpoint_voter.js';
|
|
55
73
|
import { SequencerInterruptedError } from './errors.js';
|
|
56
74
|
import type { SequencerEvents } from './events.js';
|
|
@@ -61,6 +79,21 @@ import { SequencerState } from './utils.js';
|
|
|
61
79
|
|
|
62
80
|
/** How much time to sleep while waiting for min transactions to accumulate for a block */
|
|
63
81
|
const TXS_POLLING_MS = 500;
|
|
82
|
+
const ARCHIVER_SYNC_POLLING_MS = 200;
|
|
83
|
+
|
|
84
|
+
/** Result from proposeCheckpoint when a checkpoint was successfully built and broadcast. */
|
|
85
|
+
type CheckpointProposalBroadcast = {
|
|
86
|
+
checkpoint: Checkpoint;
|
|
87
|
+
proposal: CheckpointProposal;
|
|
88
|
+
blockProposedAt: number;
|
|
89
|
+
};
|
|
90
|
+
|
|
91
|
+
/** Result after attestation collection and signing, ready for L1 submission. */
|
|
92
|
+
type CheckpointProposalResult = {
|
|
93
|
+
checkpoint: Checkpoint;
|
|
94
|
+
attestations: CommitteeAttestationsAndSigners;
|
|
95
|
+
attestationsSignature: Signature;
|
|
96
|
+
};
|
|
64
97
|
|
|
65
98
|
/**
|
|
66
99
|
* Handles the execution of a checkpoint proposal after the initial preparation phase.
|
|
@@ -70,12 +103,32 @@ const TXS_POLLING_MS = 500;
|
|
|
70
103
|
*/
|
|
71
104
|
export class CheckpointProposalJob implements Traceable {
|
|
72
105
|
protected readonly log: Logger;
|
|
106
|
+
private readonly checkpointEventLog: Logger;
|
|
107
|
+
|
|
108
|
+
/** Tracks the fire-and-forget L1 submission promise so it can be awaited during shutdown. */
|
|
109
|
+
private pendingL1Submission: Promise<void> | undefined;
|
|
110
|
+
private readonly interruptibleSleep = new InterruptibleSleep();
|
|
111
|
+
private interrupted = false;
|
|
112
|
+
|
|
113
|
+
/**
|
|
114
|
+
* Chain state overrides built once per slot in proposeCheckpoint after the checkpoint is
|
|
115
|
+
* complete. Carries the pending parent override (archive + slot + fee header) for pipelining,
|
|
116
|
+
* or the invalidation pending override when rolling back. Consumed by
|
|
117
|
+
* publisher.validateBlockHeader before broadcast.
|
|
118
|
+
*/
|
|
119
|
+
private checkpointSimulationOverridesPlan?: SimulationOverridesPlan;
|
|
120
|
+
|
|
121
|
+
private getSignatureContext(): CoordinationSignatureContext {
|
|
122
|
+
return this.signatureContext;
|
|
123
|
+
}
|
|
73
124
|
|
|
74
125
|
constructor(
|
|
75
|
-
private readonly
|
|
76
|
-
private readonly
|
|
126
|
+
private readonly slotNow: SlotNumber,
|
|
127
|
+
private readonly targetSlot: SlotNumber,
|
|
128
|
+
private readonly targetEpoch: EpochNumber,
|
|
77
129
|
private readonly checkpointNumber: CheckpointNumber,
|
|
78
130
|
private readonly syncedToBlockNumber: BlockNumber,
|
|
131
|
+
private readonly checkpointedCheckpointNumber: CheckpointNumber,
|
|
79
132
|
// TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
|
|
80
133
|
private readonly proposer: EthAddress | undefined,
|
|
81
134
|
private readonly publisher: SequencerPublisher,
|
|
@@ -88,25 +141,75 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
88
141
|
private readonly l1ToL2MessageSource: L1ToL2MessageSource,
|
|
89
142
|
private readonly l2BlockSource: L2BlockSource,
|
|
90
143
|
private readonly checkpointsBuilder: FullNodeCheckpointsBuilder,
|
|
91
|
-
private readonly blockSink: L2BlockSink,
|
|
144
|
+
private readonly blockSink: L2BlockSink & ProposedCheckpointSink,
|
|
92
145
|
private readonly l1Constants: SequencerRollupConstants,
|
|
146
|
+
private readonly signatureContext: CoordinationSignatureContext,
|
|
93
147
|
protected config: ResolvedSequencerConfig,
|
|
94
148
|
protected timetable: SequencerTimetable,
|
|
95
149
|
private readonly slasherClient: SlasherClientInterface | undefined,
|
|
96
150
|
private readonly epochCache: EpochCache,
|
|
97
151
|
private readonly dateProvider: DateProvider,
|
|
98
152
|
private readonly metrics: SequencerMetrics,
|
|
99
|
-
private readonly
|
|
100
|
-
|
|
153
|
+
private readonly checkpointMetrics: CheckpointProposalJobMetricsRecorder,
|
|
154
|
+
protected readonly eventEmitter: TypedEventEmitter<SequencerEvents>,
|
|
155
|
+
private readonly setStateFn: (state: SequencerState, slot?: SlotNumber, timeReferenceSlot?: SlotNumber) => void,
|
|
101
156
|
public readonly tracer: Tracer,
|
|
102
157
|
bindings?: LoggerBindings,
|
|
158
|
+
private readonly proposedCheckpointData?: ProposedCheckpointData,
|
|
103
159
|
) {
|
|
104
|
-
this.log = createLogger('sequencer:checkpoint-proposal', {
|
|
160
|
+
this.log = createLogger('sequencer:checkpoint-proposal', {
|
|
161
|
+
...bindings,
|
|
162
|
+
instanceId: `slot-${this.slotNow}`,
|
|
163
|
+
});
|
|
164
|
+
this.checkpointEventLog = createLogger('sequencer:checkpoint-events', {
|
|
165
|
+
...bindings,
|
|
166
|
+
instanceId: `slot-${this.slotNow}`,
|
|
167
|
+
});
|
|
168
|
+
}
|
|
169
|
+
|
|
170
|
+
/** Awaits the pending L1 submission if one is in progress. Call during shutdown. */
|
|
171
|
+
public async awaitPendingSubmission(): Promise<void> {
|
|
172
|
+
this.log.info('Awaiting pending L1 payload submission');
|
|
173
|
+
await this.pendingL1Submission;
|
|
174
|
+
}
|
|
175
|
+
|
|
176
|
+
/** Interrupts job-owned waits, including the publisher's send-at-slot sleep, so shutdown can finish. */
|
|
177
|
+
public interrupt(): void {
|
|
178
|
+
this.interrupted = true;
|
|
179
|
+
this.interruptibleSleep.interrupt(true);
|
|
180
|
+
this.publisher.interrupt();
|
|
181
|
+
}
|
|
182
|
+
|
|
183
|
+
private async awaitInterruptibleSleep(ms: number): Promise<void> {
|
|
184
|
+
if (this.interrupted) {
|
|
185
|
+
throw new SequencerInterruptedError();
|
|
186
|
+
}
|
|
187
|
+
if (ms <= 0) {
|
|
188
|
+
return;
|
|
189
|
+
}
|
|
190
|
+
try {
|
|
191
|
+
await this.interruptibleSleep.sleep(ms);
|
|
192
|
+
} catch (err) {
|
|
193
|
+
if (err instanceof InterruptError) {
|
|
194
|
+
throw new SequencerInterruptedError();
|
|
195
|
+
}
|
|
196
|
+
throw err;
|
|
197
|
+
}
|
|
198
|
+
}
|
|
199
|
+
|
|
200
|
+
private logCheckpointEvent(eventName: string, message: string, fields: Record<string, unknown>): void {
|
|
201
|
+
this.checkpointEventLog.debug(message, {
|
|
202
|
+
eventName: `sequencer-checkpoint-${eventName}`,
|
|
203
|
+
...fields,
|
|
204
|
+
});
|
|
105
205
|
}
|
|
106
206
|
|
|
107
207
|
/**
|
|
108
208
|
* Executes the checkpoint proposal job.
|
|
109
|
-
*
|
|
209
|
+
* Builds blocks, assembles checkpoint, and broadcasts the proposal (blocking).
|
|
210
|
+
* Attestation collection, signing, and L1 submission are backgrounded so the
|
|
211
|
+
* work loop can return to IDLE immediately for consecutive slot proposals.
|
|
212
|
+
* Returns the built checkpoint if successful, undefined otherwise.
|
|
110
213
|
*/
|
|
111
214
|
@trackSpan('CheckpointProposalJob.execute')
|
|
112
215
|
public async execute(): Promise<Checkpoint | undefined> {
|
|
@@ -114,7 +217,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
114
217
|
// In fisherman mode, we simulate slashing but don't actually publish to L1
|
|
115
218
|
// These are constant for the whole slot, so we only enqueue them once
|
|
116
219
|
const votesPromises = new CheckpointVoter(
|
|
117
|
-
this.
|
|
220
|
+
this.targetSlot,
|
|
118
221
|
this.publisher,
|
|
119
222
|
this.attestorAddress,
|
|
120
223
|
this.validatorClient,
|
|
@@ -125,33 +228,302 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
125
228
|
this.log,
|
|
126
229
|
).enqueueVotes();
|
|
127
230
|
|
|
128
|
-
// Build
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
135
|
-
|
|
231
|
+
// Build blocks, assemble checkpoint, and broadcast proposal (BLOCKING).
|
|
232
|
+
// Returns after broadcast — attestation collection is deferred.
|
|
233
|
+
const broadcast = await this.proposeCheckpoint();
|
|
234
|
+
|
|
235
|
+
if (!broadcast) {
|
|
236
|
+
await Promise.all(votesPromises);
|
|
237
|
+
// Still submit votes even without a checkpoint.
|
|
238
|
+
// Under proposer pipelining, vote-offenses signatures are EIP-712-bound to `targetSlot`
|
|
239
|
+
// (the pipelined slot in which the multicall is expected to mine). Submitting at the
|
|
240
|
+
// wall-clock time would let the multicall mine in a different L2 slot, causing
|
|
241
|
+
// signature verification to fail silently inside Multicall3. Delay submission to the
|
|
242
|
+
// start of `targetSlot` so the tx mines in the slot the vote was signed for.
|
|
243
|
+
if (!this.config.fishermanMode) {
|
|
244
|
+
this.pendingL1Submission = this.publisher.sendRequestsAt(this.targetSlot).then(() => {});
|
|
245
|
+
}
|
|
246
|
+
return undefined;
|
|
136
247
|
}
|
|
137
248
|
|
|
249
|
+
const { checkpoint } = broadcast;
|
|
250
|
+
this.metrics.recordCheckpointProposalSuccess();
|
|
251
|
+
|
|
138
252
|
// Do not post anything to L1 if we are fishermen, but do perform L1 fee analysis
|
|
139
253
|
if (this.config.fishermanMode) {
|
|
140
254
|
await this.handleCheckpointEndAsFisherman(checkpoint);
|
|
141
|
-
return;
|
|
255
|
+
return checkpoint;
|
|
142
256
|
}
|
|
143
257
|
|
|
144
|
-
//
|
|
145
|
-
|
|
146
|
-
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
258
|
+
// Background the attestation → signing → L1 pipeline so the work loop is unblocked
|
|
259
|
+
this.pendingL1Submission = this.waitForAttestationsAndEnqueueSubmissionAsync(broadcast, votesPromises);
|
|
260
|
+
|
|
261
|
+
// Return the built checkpoint immediately — the work loop is now unblocked
|
|
262
|
+
return checkpoint;
|
|
263
|
+
}
|
|
264
|
+
|
|
265
|
+
/**
|
|
266
|
+
* Background pipeline: collects attestations, signs them, enqueues the checkpoint, and submits to L1.
|
|
267
|
+
* Runs as a fire-and-forget task stored in `pendingL1Submission` so the work loop is unblocked.
|
|
268
|
+
*/
|
|
269
|
+
private async waitForAttestationsAndEnqueueSubmissionAsync(
|
|
270
|
+
broadcast: CheckpointProposalBroadcast,
|
|
271
|
+
votesPromises: Promise<unknown>[],
|
|
272
|
+
): Promise<void> {
|
|
273
|
+
const { checkpoint } = broadcast;
|
|
274
|
+
|
|
275
|
+
try {
|
|
276
|
+
// Wait for all votes actions, enqueued at the beginning, to resolve
|
|
277
|
+
await Promise.all(votesPromises);
|
|
278
|
+
|
|
279
|
+
// Try to collect attestations from the committee
|
|
280
|
+
const signedAttestations = await this.getSignedCommitteeAttestations(broadcast);
|
|
281
|
+
|
|
282
|
+
// Wait for the previous checkpoint to land on L1 before submitting, so we can check it
|
|
283
|
+
// matches the proposed checkpoint we used as parent, and has valid attestations.
|
|
284
|
+
if (signedAttestations && (await this.waitForValidParentCheckpointOnL1())) {
|
|
285
|
+
await this.enqueueCheckpointForSubmission({ checkpoint, ...signedAttestations });
|
|
286
|
+
}
|
|
287
|
+
|
|
288
|
+
// If we failed to collect attestations, at least check if we need to issue an invalidation
|
|
289
|
+
if (!signedAttestations && (await this.waitForSyncedL2SlotNumber(this.slotNow))) {
|
|
290
|
+
const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
|
|
291
|
+
if (!validationStatus.valid) {
|
|
292
|
+
this.log.warn(
|
|
293
|
+
`Checkpoint ${validationStatus.checkpoint.checkpointNumber} has invalid attestations, enqueuing invalidation in spite of attestation collection failure`,
|
|
294
|
+
{ checkpoint: validationStatus.checkpoint, reason: validationStatus.reason },
|
|
295
|
+
);
|
|
296
|
+
await this.enqueueInvalidation(validationStatus);
|
|
297
|
+
}
|
|
298
|
+
}
|
|
299
|
+
|
|
300
|
+
// Send whatever was enqueued: votes + (propose | invalidation | nothing).
|
|
301
|
+
const l1Response = await this.publisher.sendRequestsAt(this.targetSlot);
|
|
302
|
+
const proposedAction = l1Response?.successfulActions.find(a => a === 'propose');
|
|
303
|
+
if (proposedAction) {
|
|
304
|
+
this.logCheckpointEvent('published', `Checkpoint published for slot ${this.targetSlot}`, {
|
|
305
|
+
slot: this.targetSlot,
|
|
306
|
+
checkpointNumber: this.checkpointNumber,
|
|
307
|
+
successfulActions: l1Response?.successfulActions,
|
|
308
|
+
sentActions: l1Response?.sentActions,
|
|
309
|
+
});
|
|
310
|
+
this.eventEmitter.emit('checkpoint-published', { checkpoint: this.checkpointNumber, slot: this.targetSlot });
|
|
311
|
+
const coinbase = checkpoint.header.coinbase;
|
|
312
|
+
await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
|
|
313
|
+
} else {
|
|
314
|
+
this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
|
|
315
|
+
slot: this.targetSlot,
|
|
316
|
+
checkpointNumber: this.checkpointNumber,
|
|
317
|
+
successfulActions: l1Response?.successfulActions,
|
|
318
|
+
failedActions: l1Response?.failedActions,
|
|
319
|
+
sentActions: l1Response?.sentActions,
|
|
320
|
+
expiredActions: l1Response?.expiredActions,
|
|
321
|
+
reason: 'propose_action_not_successful',
|
|
322
|
+
});
|
|
323
|
+
this.log.warn(`Checkpoint publish failed for slot ${this.targetSlot}`, {
|
|
324
|
+
slot: this.targetSlot,
|
|
325
|
+
checkpointNumber: this.checkpointNumber,
|
|
326
|
+
successfulActions: l1Response?.successfulActions,
|
|
327
|
+
failedActions: l1Response?.failedActions,
|
|
328
|
+
sentActions: l1Response?.sentActions,
|
|
329
|
+
expiredActions: l1Response?.expiredActions,
|
|
330
|
+
reason: 'propose_action_not_successful',
|
|
331
|
+
});
|
|
332
|
+
this.eventEmitter.emit('checkpoint-publish-failed', { ...l1Response, slot: this.targetSlot });
|
|
333
|
+
this.metrics.recordPipelineDiscard();
|
|
334
|
+
}
|
|
335
|
+
} catch (err) {
|
|
336
|
+
if (err instanceof SequencerInterruptedError) {
|
|
337
|
+
return;
|
|
338
|
+
}
|
|
339
|
+
this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
|
|
340
|
+
slot: this.targetSlot,
|
|
341
|
+
checkpointNumber: this.checkpointNumber,
|
|
342
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
343
|
+
});
|
|
344
|
+
this.log.error(`Background attestation/L1 pipeline failed for slot ${this.targetSlot}`, err, {
|
|
345
|
+
slot: this.targetSlot,
|
|
346
|
+
checkpointNumber: this.checkpointNumber,
|
|
347
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
348
|
+
});
|
|
349
|
+
this.eventEmitter.emit('checkpoint-publish-failed', { slot: this.targetSlot });
|
|
350
|
+
this.metrics.recordPipelineDiscard();
|
|
351
|
+
}
|
|
352
|
+
}
|
|
353
|
+
|
|
354
|
+
/** Enqueues the checkpoint for L1 submission. Called after pipeline sleep in execute(). */
|
|
355
|
+
private async enqueueCheckpointForSubmission(result: CheckpointProposalResult): Promise<void> {
|
|
356
|
+
const { checkpoint, attestations, attestationsSignature } = result;
|
|
357
|
+
|
|
358
|
+
// Build-frame deadlines are measured against `slotNow`; observers still see `targetSlot`.
|
|
359
|
+
this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.targetSlot, this.slotNow);
|
|
360
|
+
const aztecSlotDuration = this.l1Constants.slotDuration;
|
|
361
|
+
const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
|
|
362
|
+
const txTimeoutAt = new Date((submissionSlotStart + aztecSlotDuration) * 1000);
|
|
363
|
+
|
|
364
|
+
// If we have been configured to potentially skip publishing checkpoint then roll the dice here
|
|
365
|
+
if (
|
|
366
|
+
this.config.skipPublishingCheckpointsPercent !== undefined &&
|
|
367
|
+
this.config.skipPublishingCheckpointsPercent > 0
|
|
368
|
+
) {
|
|
369
|
+
const roll = Math.max(0, randomInt(100));
|
|
370
|
+
if (roll < this.config.skipPublishingCheckpointsPercent) {
|
|
371
|
+
this.log.warn(
|
|
372
|
+
`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${roll}`,
|
|
373
|
+
);
|
|
374
|
+
return;
|
|
375
|
+
}
|
|
376
|
+
}
|
|
377
|
+
|
|
378
|
+
await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
|
|
379
|
+
txTimeoutAt,
|
|
380
|
+
});
|
|
381
|
+
}
|
|
382
|
+
|
|
383
|
+
/**
|
|
384
|
+
* Wait until the archiver syncs past the given L2 slot number.
|
|
385
|
+
* The deadline is the end of `this.targetSlot`, beyond which any pipelined work would miss its
|
|
386
|
+
* L1 submission window and is no longer useful.
|
|
387
|
+
*/
|
|
388
|
+
private async waitForSyncedL2SlotNumber(waitForSlot: SlotNumber): Promise<boolean> {
|
|
389
|
+
const targetSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
|
|
390
|
+
const targetSlotEndMs = (targetSlotStart + this.l1Constants.slotDuration) * 1000;
|
|
391
|
+
const syncDelayTolerance = this.l1Constants.ethereumSlotDuration * 2 * 1000;
|
|
392
|
+
const timeoutSeconds = Math.max(0.1, (targetSlotEndMs + syncDelayTolerance - this.dateProvider.now()) / 1000);
|
|
393
|
+
|
|
394
|
+
try {
|
|
395
|
+
const timer = new Timer();
|
|
396
|
+
while (true) {
|
|
397
|
+
const syncedSlot = await this.l2BlockSource.getSyncedL2SlotNumber();
|
|
398
|
+
if (syncedSlot !== undefined && syncedSlot >= waitForSlot) {
|
|
399
|
+
return true;
|
|
400
|
+
}
|
|
401
|
+
if (timeoutSeconds && timer.s() > timeoutSeconds) {
|
|
402
|
+
throw new TimeoutError(`Timeout awaiting archiver sync past slot ${waitForSlot}`);
|
|
403
|
+
}
|
|
404
|
+
await this.awaitInterruptibleSleep(ARCHIVER_SYNC_POLLING_MS);
|
|
405
|
+
}
|
|
406
|
+
} catch (err) {
|
|
407
|
+
if (err instanceof SequencerInterruptedError) {
|
|
408
|
+
throw err;
|
|
409
|
+
}
|
|
410
|
+
this.log.warn(
|
|
411
|
+
`Archiver did not sync L1 past slot ${waitForSlot} before slot ${this.targetSlot} expired, discarding pipelined work`,
|
|
412
|
+
{ checkpointNumber: this.checkpointNumber },
|
|
413
|
+
);
|
|
414
|
+
this.emitPipelinedCheckpointDiscarded('archiver-sync-timeout');
|
|
415
|
+
return false;
|
|
416
|
+
}
|
|
417
|
+
}
|
|
418
|
+
|
|
419
|
+
/**
|
|
420
|
+
* Waits for the parent checkpoint to land on L1 before submitting a pipelined checkpoint.
|
|
421
|
+
* Polls until the archiver has synced L1 past the parent's slot, then verifies:
|
|
422
|
+
* - If we built on a proposed parent: it must have landed on L1 with matching hash and valid attestations.
|
|
423
|
+
* - If we built without a proposed parent: no new checkpoint must have appeared for that slot.
|
|
424
|
+
* If the parent has invalid attestations, enqueues an invalidation. Returns whether to proceed with the proposal.
|
|
425
|
+
*/
|
|
426
|
+
protected async waitForValidParentCheckpointOnL1(): Promise<boolean> {
|
|
427
|
+
if (this.config.skipWaitForValidParentCheckpointOnL1) {
|
|
428
|
+
this.log.warn(`Skipping waitForValidParentCheckpointOnL1 due to test configuration`, {
|
|
429
|
+
checkpointNumber: this.checkpointNumber,
|
|
430
|
+
});
|
|
431
|
+
return true;
|
|
432
|
+
}
|
|
433
|
+
|
|
434
|
+
const parentCheckpointNumber = CheckpointNumber(this.checkpointNumber - 1);
|
|
435
|
+
|
|
436
|
+
// Wait until archiver has synced L1 past the parent's slot (slotNow)
|
|
437
|
+
if (!(await this.waitForSyncedL2SlotNumber(this.slotNow))) {
|
|
438
|
+
return false;
|
|
439
|
+
}
|
|
440
|
+
|
|
441
|
+
const tips = await this.l2BlockSource.getL2Tips();
|
|
442
|
+
const checkpointedNumber = tips.checkpointed.checkpoint.number;
|
|
443
|
+
|
|
444
|
+
// We built on top of a proposed checkpoint. Verify it landed on L1 as expected.
|
|
445
|
+
if (this.proposedCheckpointData) {
|
|
446
|
+
// After syncing from L1 we see the chain tip has invalid attestations. This means the parent checkpoint was posted
|
|
447
|
+
// with invalid attestations, or it built on top of something with invalid attestations and didnt invalidate them.
|
|
448
|
+
// Either way, we thought our parent would be valid, so we have to throw away our work. But at least we'll try and
|
|
449
|
+
// invalidate on L1 so we clean up the chain for the next proposer. And we'll slash them, but that's handled elsewhere.
|
|
450
|
+
const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
|
|
451
|
+
if (!validationStatus.valid) {
|
|
452
|
+
this.log.warn(
|
|
453
|
+
`Parent checkpoint ${parentCheckpointNumber} has invalid attestations, discarding pipelined work`,
|
|
454
|
+
{ checkpointNumber: this.checkpointNumber, reason: validationStatus.reason },
|
|
455
|
+
);
|
|
456
|
+
this.emitPipelinedCheckpointDiscarded('parent-invalid-attestations');
|
|
457
|
+
await this.enqueueInvalidation(validationStatus);
|
|
458
|
+
return false;
|
|
459
|
+
}
|
|
460
|
+
|
|
461
|
+
// The pending chain is valid. But did the parent checkpoint land on L1 at all?
|
|
462
|
+
if (checkpointedNumber < parentCheckpointNumber) {
|
|
463
|
+
this.log.warn(`Parent checkpoint ${parentCheckpointNumber} did not land on L1, discarding pipelined work`, {
|
|
464
|
+
checkpointNumber: this.checkpointNumber,
|
|
465
|
+
checkpointedNumber,
|
|
466
|
+
});
|
|
467
|
+
this.emitPipelinedCheckpointDiscarded('parent-not-on-l1');
|
|
468
|
+
return false;
|
|
469
|
+
}
|
|
470
|
+
|
|
471
|
+
// It landed. But is it the one we were expecting?
|
|
472
|
+
const expectedHash = this.proposedCheckpointData.header.hash().toString();
|
|
473
|
+
if (tips.checkpointed.checkpoint.hash !== expectedHash) {
|
|
474
|
+
this.log.warn(`Parent checkpoint ${parentCheckpointNumber} hash mismatch on L1, discarding pipelined work`, {
|
|
475
|
+
checkpointNumber: this.checkpointNumber,
|
|
476
|
+
expectedHash,
|
|
477
|
+
actualHash: tips.checkpointed.checkpoint.hash,
|
|
478
|
+
});
|
|
479
|
+
this.emitPipelinedCheckpointDiscarded('parent-hash-mismatch');
|
|
480
|
+
return false;
|
|
481
|
+
}
|
|
482
|
+
|
|
483
|
+
return true;
|
|
484
|
+
} else {
|
|
485
|
+
// We didn't see a proposed checkpoint at build time, so we built on checkpointed parent from two slots ago.
|
|
486
|
+
// But if a new checkpoint for the previous slot appeared on L1 in the meantime, our checkpoint assumed the wrong parent,
|
|
487
|
+
// so we have to discard our work. This can happen if we're somehow cut off from p2p and fail to see the checkpoint
|
|
488
|
+
// proposal for the previous slot.
|
|
489
|
+
if (checkpointedNumber > parentCheckpointNumber) {
|
|
490
|
+
this.log.warn(
|
|
491
|
+
`Unexpected checkpoint ${checkpointedNumber} landed on L1 after we built on top of parent ${parentCheckpointNumber}, discarding pipelined work`,
|
|
492
|
+
{ checkpointNumber: this.checkpointNumber, checkpointedNumber },
|
|
493
|
+
);
|
|
494
|
+
this.emitPipelinedCheckpointDiscarded('unexpected-parent-appeared');
|
|
495
|
+
return false;
|
|
496
|
+
}
|
|
497
|
+
|
|
498
|
+
return true;
|
|
499
|
+
}
|
|
500
|
+
}
|
|
501
|
+
|
|
502
|
+
/** Emits the pipelined-checkpoint-discarded event and records the metric. */
|
|
503
|
+
private emitPipelinedCheckpointDiscarded(reason: string): void {
|
|
504
|
+
this.metrics.recordPipelineParentCheckpointMismatch(reason);
|
|
505
|
+
this.eventEmitter.emit('pipelined-checkpoint-discarded', {
|
|
506
|
+
slot: this.targetSlot,
|
|
507
|
+
checkpointNumber: this.checkpointNumber,
|
|
508
|
+
reason,
|
|
509
|
+
});
|
|
510
|
+
}
|
|
511
|
+
|
|
512
|
+
/** Simulates and enqueues an invalidation request for the invalid parent checkpoint. */
|
|
513
|
+
private async enqueueInvalidation(validationStatus: ValidateCheckpointResult): Promise<void> {
|
|
514
|
+
if (this.config.skipInvalidateBlockAsProposer) {
|
|
515
|
+
this.log.warn(`Skipping checkpoint invalidation as proposer due to test configuration`);
|
|
516
|
+
return;
|
|
517
|
+
}
|
|
518
|
+
const invalidateRequest = await this.publisher.simulateInvalidateCheckpoint(validationStatus);
|
|
519
|
+
if (invalidateRequest) {
|
|
520
|
+
const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
|
|
521
|
+
const txTimeoutAt = new Date((submissionSlotStart + this.l1Constants.slotDuration) * 1000);
|
|
522
|
+
this.publisher.enqueueInvalidateCheckpoint(invalidateRequest, { txTimeoutAt });
|
|
523
|
+
} else {
|
|
524
|
+
this.log.info(`Invalidation simulation returned undefined, checkpoint may have been removed already`, {
|
|
525
|
+
checkpointNumber: this.checkpointNumber,
|
|
526
|
+
});
|
|
155
527
|
}
|
|
156
528
|
}
|
|
157
529
|
|
|
@@ -159,42 +531,89 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
159
531
|
return {
|
|
160
532
|
// nullish operator needed for tests
|
|
161
533
|
[Attributes.COINBASE]: this.validatorClient.getCoinbaseForAttestor(this.attestorAddress)?.toString(),
|
|
162
|
-
[Attributes.SLOT_NUMBER]: this.
|
|
534
|
+
[Attributes.SLOT_NUMBER]: this.targetSlot,
|
|
163
535
|
};
|
|
164
536
|
})
|
|
165
|
-
private async proposeCheckpoint(): Promise<
|
|
537
|
+
private async proposeCheckpoint(): Promise<CheckpointProposalBroadcast | undefined> {
|
|
166
538
|
try {
|
|
539
|
+
const now = this.dateProvider.now();
|
|
540
|
+
if (this.proposedCheckpointData) {
|
|
541
|
+
// Measure against the wall-clock slot whose build window we are currently using.
|
|
542
|
+
// In pipelining mode `targetSlot` is intentionally one slot ahead, which makes the
|
|
543
|
+
// target-slot boundary a full slot away from the actual build start time.
|
|
544
|
+
const slotBoundaryMs = Number(getTimestampForSlot(this.slotNow, this.l1Constants)) * 1000;
|
|
545
|
+
this.checkpointMetrics.recordPipelinedCheckpointBuildStartOffsetFromSlotBoundary(now - slotBoundaryMs);
|
|
546
|
+
}
|
|
547
|
+
this.checkpointMetrics.startCheckpointTiming(now);
|
|
548
|
+
|
|
167
549
|
// Get operator configured coinbase and fee recipient for this attestor
|
|
168
550
|
const coinbase = this.validatorClient.getCoinbaseForAttestor(this.attestorAddress);
|
|
169
551
|
const feeRecipient = this.validatorClient.getFeeRecipientForAttestor(this.attestorAddress);
|
|
170
552
|
|
|
171
553
|
// Start the checkpoint
|
|
172
|
-
this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.
|
|
173
|
-
this.
|
|
554
|
+
this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.targetSlot, this.slotNow);
|
|
555
|
+
this.logCheckpointEvent('slot-started', `Starting checkpoint proposal for slot ${this.targetSlot}`, {
|
|
556
|
+
buildSlot: this.slotNow,
|
|
557
|
+
submissionSlot: this.targetSlot,
|
|
558
|
+
slot: this.targetSlot,
|
|
559
|
+
checkpointNumber: this.checkpointNumber,
|
|
560
|
+
proposer: this.proposer?.toString(),
|
|
561
|
+
attestorAddress: this.attestorAddress.toString(),
|
|
562
|
+
publisherAddress: this.publisher.getSenderAddress().toString(),
|
|
563
|
+
coinbase: coinbase.toString(),
|
|
564
|
+
});
|
|
565
|
+
this.metrics.incOpenSlot(this.targetSlot, this.proposer?.toString() ?? 'unknown');
|
|
174
566
|
|
|
175
567
|
// Enqueues checkpoint invalidation (constant for the whole slot)
|
|
176
568
|
if (this.invalidateCheckpoint && !this.config.skipInvalidateBlockAsProposer) {
|
|
177
569
|
this.publisher.enqueueInvalidateCheckpoint(this.invalidateCheckpoint);
|
|
178
570
|
}
|
|
179
571
|
|
|
180
|
-
//
|
|
572
|
+
// Build the simulation plan for this slot. When pipelining, this overrides L1's view of
|
|
573
|
+
// pending/archive/fee-header to "as if the proposed parent had landed", so both the
|
|
574
|
+
// mana-min-fee simulation (in the globals builder) and the pre-broadcast
|
|
575
|
+
// validateBlockHeader see the chain tip the eventual L1 send will see.
|
|
576
|
+
this.checkpointSimulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
|
|
577
|
+
checkpointNumber: this.checkpointNumber,
|
|
578
|
+
proposedCheckpointData: this.proposedCheckpointData,
|
|
579
|
+
invalidateToPendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
|
|
580
|
+
checkpointedCheckpointNumber: this.checkpointedCheckpointNumber,
|
|
581
|
+
rollup: this.publisher.rollupContract,
|
|
582
|
+
signatureContext: this.signatureContext,
|
|
583
|
+
log: this.log,
|
|
584
|
+
});
|
|
585
|
+
|
|
181
586
|
const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(
|
|
182
587
|
coinbase,
|
|
183
588
|
feeRecipient,
|
|
184
|
-
this.
|
|
589
|
+
this.targetSlot,
|
|
590
|
+
this.checkpointSimulationOverridesPlan,
|
|
185
591
|
);
|
|
186
592
|
|
|
187
593
|
// Collect L1 to L2 messages for the checkpoint and compute their hash
|
|
188
594
|
const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(this.checkpointNumber);
|
|
189
595
|
const inHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
|
|
190
596
|
|
|
191
|
-
// Collect the out hashes of all the checkpoints before this one in the same epoch
|
|
192
|
-
|
|
193
|
-
|
|
194
|
-
|
|
597
|
+
// Collect the out hashes of all the checkpoints before this one in the same epoch.
|
|
598
|
+
// Under pipelining the parent checkpoint may not be on L1 yet at build time, so the helper
|
|
599
|
+
// splices in the parent's checkpointOutHash from the locally-known proposed checkpoint so
|
|
600
|
+
// the resulting `epochOutHash` matches what validators (and L1) compute once the parent
|
|
601
|
+
// lands on L1.
|
|
602
|
+
const previousCheckpointOutHashes = await getPreviousCheckpointOutHashes({
|
|
603
|
+
blockSource: this.l2BlockSource,
|
|
604
|
+
epoch: this.targetEpoch,
|
|
605
|
+
checkpointNumber: this.checkpointNumber,
|
|
606
|
+
l1Constants: this.epochCache.getL1Constants(),
|
|
607
|
+
pipeliningEnabled: true,
|
|
608
|
+
proposedCheckpointData: this.proposedCheckpointData,
|
|
609
|
+
log: this.log,
|
|
610
|
+
});
|
|
195
611
|
|
|
196
|
-
//
|
|
197
|
-
|
|
612
|
+
// Anchor the modifier to the predicted parent fee header: L1 will apply it against
|
|
613
|
+
// that, not against the latest published checkpoint (which lags by one under pipelining).
|
|
614
|
+
const predictedParentEthPerFeeAssetE12 =
|
|
615
|
+
this.checkpointSimulationOverridesPlan?.pendingCheckpointState?.feeHeader?.ethPerFeeAsset;
|
|
616
|
+
const feeAssetPriceModifier = await this.publisher.getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12);
|
|
198
617
|
|
|
199
618
|
// Create a long-lived forked world state for the checkpoint builder
|
|
200
619
|
await using fork = await this.worldState.fork(this.syncedToBlockNumber, { closeDelayMs: 12_000 });
|
|
@@ -218,11 +637,12 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
218
637
|
|
|
219
638
|
const checkpointProposalOptions: CheckpointProposalOptions = {
|
|
220
639
|
publishFullTxs: !!this.config.publishTxsWithProposals,
|
|
221
|
-
broadcastInvalidCheckpointProposal:
|
|
640
|
+
broadcastInvalidCheckpointProposal:
|
|
641
|
+
this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal,
|
|
222
642
|
};
|
|
223
643
|
|
|
224
644
|
let blocksInCheckpoint: L2Block[] = [];
|
|
225
|
-
let blockPendingBroadcast:
|
|
645
|
+
let blockPendingBroadcast: BlockProposal | undefined = undefined;
|
|
226
646
|
const checkpointBuildTimer = new Timer();
|
|
227
647
|
|
|
228
648
|
try {
|
|
@@ -246,26 +666,49 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
246
666
|
}
|
|
247
667
|
|
|
248
668
|
if (blocksInCheckpoint.length === 0) {
|
|
249
|
-
this.
|
|
250
|
-
|
|
669
|
+
this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
|
|
670
|
+
slot: this.targetSlot,
|
|
671
|
+
checkpointNumber: this.checkpointNumber,
|
|
672
|
+
reason: 'no_blocks_built',
|
|
673
|
+
});
|
|
674
|
+
this.log.warn(`No blocks were built for slot ${this.targetSlot}`, {
|
|
675
|
+
slot: this.targetSlot,
|
|
676
|
+
checkpointNumber: this.checkpointNumber,
|
|
677
|
+
reason: 'no_blocks_built',
|
|
678
|
+
});
|
|
679
|
+
this.eventEmitter.emit('checkpoint-empty', { slot: this.targetSlot });
|
|
251
680
|
return undefined;
|
|
252
681
|
}
|
|
253
682
|
|
|
254
683
|
const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
|
|
255
684
|
if (minBlocksForCheckpoint !== undefined && blocksInCheckpoint.length < minBlocksForCheckpoint) {
|
|
685
|
+
this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
|
|
686
|
+
slot: this.targetSlot,
|
|
687
|
+
checkpointNumber: this.checkpointNumber,
|
|
688
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
689
|
+
minBlocksForCheckpoint,
|
|
690
|
+
reason: 'min_blocks_not_met',
|
|
691
|
+
});
|
|
256
692
|
this.log.warn(
|
|
257
693
|
`Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`,
|
|
258
|
-
{
|
|
694
|
+
{
|
|
695
|
+
slot: this.targetSlot,
|
|
696
|
+
checkpointNumber: this.checkpointNumber,
|
|
697
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
698
|
+
minBlocksForCheckpoint,
|
|
699
|
+
reason: 'min_blocks_not_met',
|
|
700
|
+
},
|
|
259
701
|
);
|
|
260
702
|
return undefined;
|
|
261
703
|
}
|
|
262
704
|
|
|
263
705
|
// Assemble and broadcast the checkpoint proposal, including the last block that was not
|
|
264
706
|
// broadcasted yet, and wait to collect the committee attestations.
|
|
265
|
-
this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.
|
|
707
|
+
this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.targetSlot, this.slotNow);
|
|
266
708
|
const checkpoint = await checkpointBuilder.completeCheckpoint();
|
|
267
709
|
|
|
268
|
-
// Final validation
|
|
710
|
+
// Final validation: per-block limits are only checked if the operator set them explicitly.
|
|
711
|
+
// Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
|
|
269
712
|
try {
|
|
270
713
|
validateCheckpoint(checkpoint, {
|
|
271
714
|
rollupManaLimit: this.l1Constants.rollupManaLimit,
|
|
@@ -275,113 +718,118 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
275
718
|
maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint,
|
|
276
719
|
});
|
|
277
720
|
} catch (err) {
|
|
278
|
-
this.
|
|
721
|
+
this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
|
|
722
|
+
slot: this.targetSlot,
|
|
723
|
+
checkpointNumber: this.checkpointNumber,
|
|
724
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
725
|
+
reason: 'invalid_checkpoint',
|
|
726
|
+
checkpoint: checkpoint.header.toInspect(),
|
|
727
|
+
});
|
|
728
|
+
this.log.error(`Built an invalid checkpoint at slot ${this.slotNow} (skipping proposal)`, err, {
|
|
729
|
+
slot: this.targetSlot,
|
|
730
|
+
checkpointNumber: this.checkpointNumber,
|
|
731
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
732
|
+
reason: 'invalid_checkpoint',
|
|
279
733
|
checkpoint: checkpoint.header.toInspect(),
|
|
280
734
|
});
|
|
281
735
|
return undefined;
|
|
282
736
|
}
|
|
283
737
|
|
|
284
738
|
// Record checkpoint-level build metrics
|
|
285
|
-
this.
|
|
739
|
+
this.checkpointMetrics.recordCheckpointBuild(
|
|
286
740
|
checkpointBuildTimer.ms(),
|
|
287
741
|
blocksInCheckpoint.length,
|
|
288
742
|
checkpoint.getStats().txCount,
|
|
289
743
|
Number(checkpoint.header.totalManaUsed.toBigInt()),
|
|
290
744
|
);
|
|
745
|
+
this.logCheckpointEvent('built', `Checkpoint built for slot ${this.targetSlot}`, {
|
|
746
|
+
slot: this.targetSlot,
|
|
747
|
+
buildSlot: this.slotNow,
|
|
748
|
+
checkpointNumber: this.checkpointNumber,
|
|
749
|
+
proposer: this.proposer?.toString(),
|
|
750
|
+
attestorAddress: this.attestorAddress.toString(),
|
|
751
|
+
publisherAddress: this.publisher.getSenderAddress().toString(),
|
|
752
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
753
|
+
txCount: checkpoint.getStats().txCount,
|
|
754
|
+
totalMana: Number(checkpoint.header.totalManaUsed.toBigInt()),
|
|
755
|
+
});
|
|
291
756
|
|
|
292
|
-
//
|
|
757
|
+
// In fisherman mode, return the checkpoint without broadcasting or collecting attestations
|
|
293
758
|
if (this.config.fishermanMode) {
|
|
294
759
|
this.log.info(
|
|
295
|
-
`Built checkpoint for slot ${this.
|
|
760
|
+
`Built checkpoint for slot ${this.targetSlot} with ${blocksInCheckpoint.length} blocks. ` +
|
|
296
761
|
`Skipping proposal in fisherman mode.`,
|
|
297
762
|
{
|
|
298
|
-
slot: this.
|
|
763
|
+
slot: this.targetSlot,
|
|
299
764
|
checkpoint: checkpoint.header.toInspect(),
|
|
300
765
|
blocksBuilt: blocksInCheckpoint.length,
|
|
301
766
|
},
|
|
302
767
|
);
|
|
303
768
|
this.metrics.recordCheckpointSuccess();
|
|
304
|
-
|
|
769
|
+
// Return a broadcast result with a dummy proposal — fisherman mode skips attestation collection
|
|
770
|
+
return { checkpoint, proposal: undefined!, blockProposedAt: this.dateProvider.now() };
|
|
305
771
|
}
|
|
306
772
|
|
|
307
|
-
//
|
|
308
|
-
|
|
309
|
-
|
|
310
|
-
|
|
311
|
-
|
|
312
|
-
}
|
|
773
|
+
// Validate the header against L1 state before broadcasting.
|
|
774
|
+
// If this fails the slot is aborted before any gossip work; state drift between here
|
|
775
|
+
// and the eventual L1 send is caught by the bundle simulate at send time.
|
|
776
|
+
try {
|
|
777
|
+
await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
|
|
778
|
+
} catch (err) {
|
|
779
|
+
this.log.error(`Pre-broadcast header validation failed for slot ${this.targetSlot}; aborting`, err, {
|
|
780
|
+
slot: this.targetSlot,
|
|
781
|
+
checkpointNumber: this.checkpointNumber,
|
|
782
|
+
});
|
|
783
|
+
this.metrics.recordCheckpointProposalFailed('header_validation_failed');
|
|
784
|
+
this.eventEmitter.emit('header-validation-failed', {
|
|
785
|
+
slot: this.targetSlot,
|
|
786
|
+
checkpointNumber: this.checkpointNumber,
|
|
787
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
788
|
+
});
|
|
789
|
+
return undefined;
|
|
790
|
+
}
|
|
313
791
|
|
|
314
792
|
// Create the checkpoint proposal and broadcast it
|
|
315
793
|
const proposal = await this.validatorClient.createCheckpointProposal(
|
|
316
794
|
checkpoint.header,
|
|
317
795
|
checkpoint.archive.root,
|
|
796
|
+
this.checkpointNumber,
|
|
318
797
|
feeAssetPriceModifier,
|
|
319
|
-
|
|
798
|
+
blockPendingBroadcast,
|
|
320
799
|
this.proposer,
|
|
321
800
|
checkpointProposalOptions,
|
|
322
801
|
);
|
|
323
802
|
|
|
324
|
-
|
|
325
|
-
|
|
326
|
-
|
|
327
|
-
|
|
328
|
-
|
|
329
|
-
|
|
330
|
-
|
|
331
|
-
this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
|
|
332
|
-
|
|
333
|
-
// Proposer must sign over the attestations before pushing them to L1
|
|
334
|
-
const signer = this.proposer ?? this.publisher.getSenderAddress();
|
|
335
|
-
let attestationsSignature: Signature;
|
|
336
|
-
try {
|
|
337
|
-
attestationsSignature = await this.validatorClient.signAttestationsAndSigners(
|
|
338
|
-
attestations,
|
|
339
|
-
signer,
|
|
340
|
-
this.slot,
|
|
341
|
-
this.checkpointNumber,
|
|
342
|
-
);
|
|
343
|
-
} catch (err) {
|
|
344
|
-
// We shouldn't really get here since we yield to another HA node
|
|
345
|
-
// as soon as we see these errors when creating block or checkpoint proposals.
|
|
346
|
-
if (this.handleHASigningError(err, 'Attestations signature')) {
|
|
347
|
-
return undefined;
|
|
348
|
-
}
|
|
349
|
-
throw err;
|
|
350
|
-
}
|
|
351
|
-
|
|
352
|
-
// Enqueue publishing the checkpoint to L1
|
|
353
|
-
this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.slot);
|
|
354
|
-
const aztecSlotDuration = this.l1Constants.slotDuration;
|
|
355
|
-
const slotStartBuildTimestamp = this.getSlotStartBuildTimestamp();
|
|
356
|
-
const txTimeoutAt = new Date((slotStartBuildTimestamp + aztecSlotDuration) * 1000);
|
|
803
|
+
// Advance our own optimistic proposed-checkpoint tip locally before gossiping. Gossipsub
|
|
804
|
+
// doesn't echo our own messages back, so this is how the proposer makes its own proposed
|
|
805
|
+
// checkpoint visible for pipelining the next slot. Built from local checkpoint data — never
|
|
806
|
+
// from the broadcast proposal archive, which may be deliberately corrupted under test flags.
|
|
807
|
+
// Fail closed: if this throws, the outer catch aborts the slot before gossiping.
|
|
808
|
+
await this.syncProposedCheckpointToArchiver(checkpoint, blocksInCheckpoint.length, feeAssetPriceModifier);
|
|
357
809
|
|
|
358
|
-
|
|
359
|
-
if (
|
|
360
|
-
|
|
361
|
-
this
|
|
362
|
-
|
|
363
|
-
|
|
364
|
-
if (
|
|
365
|
-
this.
|
|
366
|
-
`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${result}`,
|
|
367
|
-
);
|
|
368
|
-
return checkpoint;
|
|
810
|
+
const blockProposedAt = this.dateProvider.now();
|
|
811
|
+
if (this.config.skipBroadcastCheckpointProposal) {
|
|
812
|
+
// Test-only: suppress the CheckpointProposal so peers never see a proposed checkpoint for
|
|
813
|
+
// this slot, but still broadcast the held last block standalone so peers' archivers ingest
|
|
814
|
+
// it as a proposed-but-uncheckpointed tip — the exact orphan-block state that
|
|
815
|
+
// pruneOrphanProposedBlocks / checkSync must handle.
|
|
816
|
+
if (blockPendingBroadcast && !this.config.skipBroadcastProposals) {
|
|
817
|
+
await this.p2pClient.broadcastProposal(blockPendingBroadcast);
|
|
369
818
|
}
|
|
819
|
+
} else if (!this.config.skipBroadcastProposals) {
|
|
820
|
+
await this.p2pClient.broadcastCheckpointProposal(proposal);
|
|
821
|
+
this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
|
|
370
822
|
}
|
|
371
823
|
|
|
372
|
-
|
|
373
|
-
|
|
374
|
-
forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
|
|
375
|
-
});
|
|
376
|
-
|
|
377
|
-
return checkpoint;
|
|
824
|
+
// Return immediately after broadcast — attestation collection happens in the background
|
|
825
|
+
return { checkpoint, proposal, blockProposedAt };
|
|
378
826
|
} catch (err) {
|
|
379
827
|
if (err && (err instanceof DutyAlreadySignedError || err instanceof SlashingProtectionError)) {
|
|
380
828
|
// swallow this error. It's already been logged by a function deeper in the stack
|
|
381
829
|
return undefined;
|
|
382
830
|
}
|
|
383
831
|
|
|
384
|
-
this.log.error(`Error building checkpoint at slot ${this.
|
|
832
|
+
this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
|
|
385
833
|
return undefined;
|
|
386
834
|
}
|
|
387
835
|
}
|
|
@@ -397,26 +845,35 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
397
845
|
blockProposalOptions: BlockProposalOptions,
|
|
398
846
|
): Promise<{
|
|
399
847
|
blocksInCheckpoint: L2Block[];
|
|
400
|
-
blockPendingBroadcast:
|
|
848
|
+
blockPendingBroadcast: BlockProposal | undefined;
|
|
401
849
|
}> {
|
|
402
850
|
const blocksInCheckpoint: L2Block[] = [];
|
|
403
851
|
const txHashesAlreadyIncluded = new Set<string>();
|
|
404
852
|
const initialBlockNumber = BlockNumber(this.syncedToBlockNumber + 1);
|
|
405
853
|
|
|
406
854
|
// Last block in the checkpoint will usually be flagged as pending broadcast, so we send it along with the checkpoint proposal
|
|
407
|
-
let blockPendingBroadcast:
|
|
855
|
+
let blockPendingBroadcast: BlockProposal | undefined = undefined;
|
|
408
856
|
|
|
409
857
|
while (true) {
|
|
410
858
|
const blocksBuilt = blocksInCheckpoint.length;
|
|
411
859
|
const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
|
|
412
860
|
const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
|
|
413
861
|
|
|
862
|
+
if (blocksBuilt >= this.config.maxBlocksPerCheckpoint) {
|
|
863
|
+
this.log.debug(`Reached max blocks per checkpoint`, {
|
|
864
|
+
slot: this.targetSlot,
|
|
865
|
+
blocksBuilt,
|
|
866
|
+
maxBlocksPerCheckpoint: this.config.maxBlocksPerCheckpoint,
|
|
867
|
+
});
|
|
868
|
+
break;
|
|
869
|
+
}
|
|
870
|
+
|
|
414
871
|
const secondsIntoSlot = this.getSecondsIntoSlot();
|
|
415
872
|
const timingInfo = this.timetable.canStartNextBlock(secondsIntoSlot);
|
|
416
873
|
|
|
417
874
|
if (!timingInfo.canStart) {
|
|
418
875
|
this.log.debug(`Not enough time left in slot to start another block`, {
|
|
419
|
-
slot: this.
|
|
876
|
+
slot: this.targetSlot,
|
|
420
877
|
blocksBuilt,
|
|
421
878
|
secondsIntoSlot,
|
|
422
879
|
});
|
|
@@ -437,22 +894,23 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
437
894
|
txHashesAlreadyIncluded,
|
|
438
895
|
});
|
|
439
896
|
|
|
440
|
-
//
|
|
441
|
-
if (
|
|
442
|
-
// If
|
|
443
|
-
|
|
444
|
-
|
|
445
|
-
|
|
897
|
+
// If we failed to build the block due to insufficient txs, we try again if there is still time left in the slot
|
|
898
|
+
if ('failure' in buildResult) {
|
|
899
|
+
// If this was the last subslot, or we're running with a single block per slot, we're done
|
|
900
|
+
if (timingInfo.isLastBlock || timingInfo.deadline === undefined) {
|
|
901
|
+
break;
|
|
902
|
+
}
|
|
903
|
+
// Otherwise, if there is still time for more blocks, we wait until the next subslot and try again
|
|
446
904
|
await this.waitUntilNextSubslot(timingInfo.deadline);
|
|
447
905
|
continue;
|
|
448
|
-
}
|
|
449
|
-
|
|
450
|
-
|
|
451
|
-
|
|
452
|
-
|
|
906
|
+
}
|
|
907
|
+
|
|
908
|
+
// If there was an error building the block, we just exit the loop and give up the rest of the slot.
|
|
909
|
+
// We don't want to risk building more blocks if something went wrong.
|
|
910
|
+
if ('error' in buildResult) {
|
|
453
911
|
if (!(buildResult.error instanceof SequencerInterruptedError)) {
|
|
454
|
-
this.log.warn(`Halting block building for slot ${this.
|
|
455
|
-
slot: this.
|
|
912
|
+
this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
|
|
913
|
+
slot: this.targetSlot,
|
|
456
914
|
blocksBuilt,
|
|
457
915
|
error: buildResult.error,
|
|
458
916
|
});
|
|
@@ -461,41 +919,42 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
461
919
|
}
|
|
462
920
|
|
|
463
921
|
const { block, usedTxs } = buildResult;
|
|
922
|
+
this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
|
|
923
|
+
isFirstBlock: blocksBuilt === 0,
|
|
924
|
+
isLastBlock: timingInfo.isLastBlock,
|
|
925
|
+
});
|
|
926
|
+
|
|
464
927
|
blocksInCheckpoint.push(block);
|
|
928
|
+
usedTxs.forEach(tx => txHashesAlreadyIncluded.add(tx.txHash.toString()));
|
|
465
929
|
|
|
466
|
-
//
|
|
467
|
-
|
|
468
|
-
|
|
469
|
-
|
|
470
|
-
|
|
471
|
-
|
|
930
|
+
// Sign the block proposal. This will throw if HA signing fails.
|
|
931
|
+
const proposal = await this.createBlockProposal(block, inHash, usedTxs, {
|
|
932
|
+
...blockProposalOptions,
|
|
933
|
+
broadcastInvalidBlockProposal:
|
|
934
|
+
blockProposalOptions.broadcastInvalidBlockProposal ||
|
|
935
|
+
block.indexWithinCheckpoint === this.config.invalidBlockProposalIndexWithinCheckpoint,
|
|
472
936
|
});
|
|
473
937
|
|
|
474
|
-
|
|
938
|
+
// Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal,
|
|
939
|
+
// so we avoid polluting our archive with a block that would fail.
|
|
940
|
+
// We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
|
|
941
|
+
// If this throws, we abort the entire checkpoint.
|
|
942
|
+
await this.syncProposedBlockToArchiver(block);
|
|
475
943
|
|
|
476
|
-
// If this is the last block,
|
|
944
|
+
// If this is the last block, do not broadcast it, since it will be included in the checkpoint proposal.
|
|
477
945
|
if (timingInfo.isLastBlock) {
|
|
478
|
-
this.log.verbose(`Completed final block ${blockNumber} for slot ${this.
|
|
479
|
-
slot: this.
|
|
946
|
+
this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
|
|
947
|
+
slot: this.targetSlot,
|
|
480
948
|
blockNumber,
|
|
481
949
|
blocksBuilt,
|
|
482
950
|
});
|
|
483
|
-
|
|
951
|
+
|
|
952
|
+
blockPendingBroadcast = proposal;
|
|
484
953
|
break;
|
|
485
954
|
}
|
|
486
955
|
|
|
487
|
-
//
|
|
488
|
-
|
|
489
|
-
if (!this.config.fishermanMode) {
|
|
490
|
-
const proposal = await this.validatorClient.createBlockProposal(
|
|
491
|
-
block.header,
|
|
492
|
-
block.indexWithinCheckpoint,
|
|
493
|
-
inHash,
|
|
494
|
-
block.archive.root,
|
|
495
|
-
usedTxs,
|
|
496
|
-
this.proposer,
|
|
497
|
-
blockProposalOptions,
|
|
498
|
-
);
|
|
956
|
+
// Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
|
|
957
|
+
if (proposal && !this.config.skipBroadcastProposals) {
|
|
499
958
|
await this.p2pClient.broadcastProposal(proposal);
|
|
500
959
|
}
|
|
501
960
|
|
|
@@ -503,19 +962,44 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
503
962
|
await this.waitUntilNextSubslot(timingInfo.deadline);
|
|
504
963
|
}
|
|
505
964
|
|
|
506
|
-
this.log.verbose(`Block building loop completed for slot ${this.
|
|
507
|
-
slot: this.
|
|
965
|
+
this.log.verbose(`Block building loop completed for slot ${this.targetSlot}`, {
|
|
966
|
+
slot: this.targetSlot,
|
|
508
967
|
blocksBuilt: blocksInCheckpoint.length,
|
|
509
968
|
});
|
|
510
969
|
|
|
511
970
|
return { blocksInCheckpoint, blockPendingBroadcast };
|
|
512
971
|
}
|
|
513
972
|
|
|
973
|
+
/** Creates a block proposal for a given block via the validator client (unless in fisherman mode) */
|
|
974
|
+
private createBlockProposal(
|
|
975
|
+
block: L2Block,
|
|
976
|
+
inHash: Fr,
|
|
977
|
+
usedTxs: Tx[],
|
|
978
|
+
blockProposalOptions: BlockProposalOptions,
|
|
979
|
+
): Promise<BlockProposal | undefined> {
|
|
980
|
+
if (this.config.fishermanMode) {
|
|
981
|
+
this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
|
|
982
|
+
return Promise.resolve(undefined);
|
|
983
|
+
}
|
|
984
|
+
return this.validatorClient.createBlockProposal(
|
|
985
|
+
block.header,
|
|
986
|
+
this.checkpointNumber,
|
|
987
|
+
block.indexWithinCheckpoint,
|
|
988
|
+
inHash,
|
|
989
|
+
block.archive.root,
|
|
990
|
+
usedTxs,
|
|
991
|
+
this.proposer,
|
|
992
|
+
blockProposalOptions,
|
|
993
|
+
);
|
|
994
|
+
}
|
|
995
|
+
|
|
514
996
|
/** Sleeps until it is time to produce the next block in the slot */
|
|
515
997
|
@trackSpan('CheckpointProposalJob.waitUntilNextSubslot')
|
|
516
998
|
private async waitUntilNextSubslot(nextSubslotStart: number) {
|
|
517
|
-
this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.
|
|
518
|
-
this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, {
|
|
999
|
+
this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.targetSlot, this.slotNow);
|
|
1000
|
+
this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, {
|
|
1001
|
+
slot: this.targetSlot,
|
|
1002
|
+
});
|
|
519
1003
|
await this.waitUntilTimeInSlot(nextSubslotStart);
|
|
520
1004
|
}
|
|
521
1005
|
|
|
@@ -531,12 +1015,14 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
531
1015
|
buildDeadline: Date | undefined;
|
|
532
1016
|
txHashesAlreadyIncluded: Set<string>;
|
|
533
1017
|
},
|
|
534
|
-
): Promise<
|
|
1018
|
+
): Promise<
|
|
1019
|
+
{ block: L2Block; usedTxs: Tx[] } | { failure: 'insufficient-txs' | 'insufficient-valid-txs' } | { error: Error }
|
|
1020
|
+
> {
|
|
535
1021
|
const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } =
|
|
536
1022
|
opts;
|
|
537
1023
|
|
|
538
1024
|
this.log.verbose(
|
|
539
|
-
`Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.
|
|
1025
|
+
`Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot}`,
|
|
540
1026
|
{ ...checkpointBuilder.getConstantData(), ...opts },
|
|
541
1027
|
);
|
|
542
1028
|
|
|
@@ -544,13 +1030,30 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
544
1030
|
// Wait until we have enough txs to build the block
|
|
545
1031
|
const { availableTxs, canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
|
|
546
1032
|
if (!canStartBuilding) {
|
|
1033
|
+
this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
|
|
1034
|
+
reason: 'insufficient_txs',
|
|
1035
|
+
blockNumber,
|
|
1036
|
+
slot: this.targetSlot,
|
|
1037
|
+
checkpointNumber: this.checkpointNumber,
|
|
1038
|
+
indexWithinCheckpoint,
|
|
1039
|
+
availableTxs,
|
|
1040
|
+
minTxs,
|
|
1041
|
+
});
|
|
547
1042
|
this.log.warn(
|
|
548
|
-
`Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.
|
|
549
|
-
{
|
|
1043
|
+
`Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (got ${availableTxs} txs but needs ${minTxs})`,
|
|
1044
|
+
{
|
|
1045
|
+
reason: 'insufficient_txs',
|
|
1046
|
+
blockNumber,
|
|
1047
|
+
slot: this.targetSlot,
|
|
1048
|
+
checkpointNumber: this.checkpointNumber,
|
|
1049
|
+
indexWithinCheckpoint,
|
|
1050
|
+
availableTxs,
|
|
1051
|
+
minTxs,
|
|
1052
|
+
},
|
|
550
1053
|
);
|
|
551
|
-
this.eventEmitter.emit('block-tx-count-check-failed', { minTxs, availableTxs, slot: this.
|
|
1054
|
+
this.eventEmitter.emit('block-tx-count-check-failed', { minTxs, availableTxs, slot: this.targetSlot });
|
|
552
1055
|
this.metrics.recordBlockProposalFailed('insufficient_txs');
|
|
553
|
-
return
|
|
1056
|
+
return { failure: 'insufficient-txs' };
|
|
554
1057
|
}
|
|
555
1058
|
|
|
556
1059
|
// Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
|
|
@@ -561,14 +1064,16 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
561
1064
|
);
|
|
562
1065
|
|
|
563
1066
|
this.log.debug(
|
|
564
|
-
`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.
|
|
565
|
-
{ slot: this.
|
|
1067
|
+
`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.targetSlot} with ${availableTxs} available txs`,
|
|
1068
|
+
{ slot: this.targetSlot, blockNumber, indexWithinCheckpoint },
|
|
566
1069
|
);
|
|
567
|
-
this.setStateFn(SequencerState.CREATING_BLOCK, this.
|
|
1070
|
+
this.setStateFn(SequencerState.CREATING_BLOCK, this.targetSlot, this.slotNow);
|
|
568
1071
|
|
|
569
|
-
// Per-block limits
|
|
1072
|
+
// Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
|
|
570
1073
|
// by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
|
|
571
|
-
|
|
1074
|
+
// minValidTxs is passed into the builder so it can reject the block *before* updating state.
|
|
1075
|
+
const minValidTxs = forceCreate ? 0 : (this.config.minValidTxsPerBlock ?? minTxs);
|
|
1076
|
+
const blockBuilderOptions: BlockBuilderOptions = {
|
|
572
1077
|
maxTransactions: this.config.maxTxsPerBlock,
|
|
573
1078
|
maxBlockGas:
|
|
574
1079
|
this.config.maxL2BlockGas !== undefined || this.config.maxDABlockGas !== undefined
|
|
@@ -576,9 +1081,14 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
576
1081
|
: undefined,
|
|
577
1082
|
deadline: buildDeadline,
|
|
578
1083
|
isBuildingProposal: true,
|
|
1084
|
+
minValidTxs,
|
|
1085
|
+
maxBlocksPerCheckpoint: this.timetable.maxNumberOfBlocks,
|
|
1086
|
+
perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier,
|
|
579
1087
|
};
|
|
580
1088
|
|
|
581
|
-
// Actually build the block by executing txs
|
|
1089
|
+
// Actually build the block by executing txs. The builder throws InsufficientValidTxsError
|
|
1090
|
+
// if the number of successfully processed txs is below minValidTxs, ensuring state is not
|
|
1091
|
+
// updated for blocks that will be discarded.
|
|
582
1092
|
const buildResult = await this.buildSingleBlockWithCheckpointBuilder(
|
|
583
1093
|
checkpointBuilder,
|
|
584
1094
|
pendingTxs,
|
|
@@ -590,22 +1100,38 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
590
1100
|
// If any txs failed during execution, drop them from the mempool so we don't pick them up again
|
|
591
1101
|
await this.dropFailedTxsFromP2P(buildResult.failedTxs);
|
|
592
1102
|
|
|
593
|
-
|
|
594
|
-
|
|
595
|
-
|
|
596
|
-
|
|
597
|
-
|
|
1103
|
+
if (buildResult.status === 'insufficient-valid-txs') {
|
|
1104
|
+
this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
|
|
1105
|
+
reason: 'insufficient_valid_txs',
|
|
1106
|
+
slot: this.targetSlot,
|
|
1107
|
+
checkpointNumber: this.checkpointNumber,
|
|
1108
|
+
blockNumber,
|
|
1109
|
+
numTxs: buildResult.processedCount,
|
|
1110
|
+
indexWithinCheckpoint,
|
|
1111
|
+
minValidTxs,
|
|
1112
|
+
});
|
|
598
1113
|
this.log.warn(
|
|
599
|
-
`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.
|
|
600
|
-
{
|
|
1114
|
+
`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.targetSlot} has too few valid txs to be proposed`,
|
|
1115
|
+
{
|
|
1116
|
+
reason: 'insufficient_valid_txs',
|
|
1117
|
+
slot: this.targetSlot,
|
|
1118
|
+
checkpointNumber: this.checkpointNumber,
|
|
1119
|
+
blockNumber,
|
|
1120
|
+
numTxs: buildResult.processedCount,
|
|
1121
|
+
indexWithinCheckpoint,
|
|
1122
|
+
minValidTxs,
|
|
1123
|
+
},
|
|
601
1124
|
);
|
|
602
|
-
this.eventEmitter.emit('block-build-failed', {
|
|
1125
|
+
this.eventEmitter.emit('block-build-failed', {
|
|
1126
|
+
reason: `Insufficient valid txs`,
|
|
1127
|
+
slot: this.targetSlot,
|
|
1128
|
+
});
|
|
603
1129
|
this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
|
|
604
|
-
return
|
|
1130
|
+
return { failure: 'insufficient-valid-txs' };
|
|
605
1131
|
}
|
|
606
1132
|
|
|
607
1133
|
// Block creation succeeded, emit stats and metrics
|
|
608
|
-
const { block, publicProcessorDuration, usedTxs, blockBuildDuration } = buildResult;
|
|
1134
|
+
const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
|
|
609
1135
|
|
|
610
1136
|
const blockStats = {
|
|
611
1137
|
eventName: 'l2-block-built',
|
|
@@ -619,30 +1145,53 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
619
1145
|
const manaPerSec = block.header.totalManaUsed.toNumberUnsafe() / (blockBuildDuration / 1000);
|
|
620
1146
|
|
|
621
1147
|
this.log.info(
|
|
622
|
-
`Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.
|
|
1148
|
+
`Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot} with ${numTxs} txs`,
|
|
623
1149
|
{ blockHash, txHashes, manaPerSec, ...blockStats },
|
|
624
1150
|
);
|
|
625
1151
|
|
|
626
|
-
|
|
627
|
-
|
|
1152
|
+
// `slot` is the target/submission slot (may be one ahead when pipelining),
|
|
1153
|
+
// `buildSlot` is the wall-clock slot during which the block was actually built.
|
|
1154
|
+
this.eventEmitter.emit('block-proposed', {
|
|
1155
|
+
blockNumber: block.number,
|
|
1156
|
+
blockHash,
|
|
1157
|
+
checkpointNumber: this.checkpointNumber,
|
|
1158
|
+
indexWithinCheckpoint: block.indexWithinCheckpoint,
|
|
1159
|
+
slot: this.targetSlot,
|
|
1160
|
+
buildSlot: this.slotNow,
|
|
1161
|
+
});
|
|
1162
|
+
this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
|
|
628
1163
|
|
|
629
1164
|
return { block, usedTxs };
|
|
630
1165
|
} catch (err: any) {
|
|
631
|
-
this.eventEmitter.emit('block-build-failed', {
|
|
632
|
-
|
|
1166
|
+
this.eventEmitter.emit('block-build-failed', {
|
|
1167
|
+
reason: err.message,
|
|
1168
|
+
slot: this.targetSlot,
|
|
1169
|
+
});
|
|
1170
|
+
this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
|
|
1171
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
1172
|
+
slot: this.targetSlot,
|
|
1173
|
+
checkpointNumber: this.checkpointNumber,
|
|
1174
|
+
blockNumber,
|
|
1175
|
+
});
|
|
1176
|
+
this.log.error(`Error building block`, err, {
|
|
1177
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
1178
|
+
slot: this.targetSlot,
|
|
1179
|
+
checkpointNumber: this.checkpointNumber,
|
|
1180
|
+
blockNumber,
|
|
1181
|
+
});
|
|
633
1182
|
this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
|
|
634
1183
|
this.metrics.recordFailedBlock();
|
|
635
1184
|
return { error: err };
|
|
636
1185
|
}
|
|
637
1186
|
}
|
|
638
1187
|
|
|
639
|
-
/** Uses the checkpoint builder to build a block, catching
|
|
1188
|
+
/** Uses the checkpoint builder to build a block, catching InsufficientValidTxsError. */
|
|
640
1189
|
private async buildSingleBlockWithCheckpointBuilder(
|
|
641
1190
|
checkpointBuilder: CheckpointBuilder,
|
|
642
1191
|
pendingTxs: AsyncIterable<Tx>,
|
|
643
1192
|
blockNumber: BlockNumber,
|
|
644
1193
|
blockTimestamp: bigint,
|
|
645
|
-
blockBuilderOptions:
|
|
1194
|
+
blockBuilderOptions: BlockBuilderOptions,
|
|
646
1195
|
) {
|
|
647
1196
|
try {
|
|
648
1197
|
const workTimer = new Timer();
|
|
@@ -650,8 +1199,12 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
650
1199
|
const blockBuildDuration = workTimer.ms();
|
|
651
1200
|
return { ...result, blockBuildDuration, status: 'success' as const };
|
|
652
1201
|
} catch (err: unknown) {
|
|
653
|
-
if (isErrorClass(err,
|
|
654
|
-
return {
|
|
1202
|
+
if (isErrorClass(err, InsufficientValidTxsError)) {
|
|
1203
|
+
return {
|
|
1204
|
+
failedTxs: err.failedTxs,
|
|
1205
|
+
processedCount: err.processedCount,
|
|
1206
|
+
status: 'insufficient-valid-txs' as const,
|
|
1207
|
+
};
|
|
655
1208
|
}
|
|
656
1209
|
throw err;
|
|
657
1210
|
}
|
|
@@ -685,10 +1238,10 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
685
1238
|
}
|
|
686
1239
|
|
|
687
1240
|
// Wait a bit before checking again
|
|
688
|
-
this.setStateFn(SequencerState.WAITING_FOR_TXS, this.
|
|
1241
|
+
this.setStateFn(SequencerState.WAITING_FOR_TXS, this.targetSlot, this.slotNow);
|
|
689
1242
|
this.log.verbose(
|
|
690
|
-
`Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.
|
|
691
|
-
{ blockNumber, slot: this.
|
|
1243
|
+
`Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (have ${availableTxs} but need ${minTxs})`,
|
|
1244
|
+
{ blockNumber, slot: this.targetSlot, indexWithinCheckpoint },
|
|
692
1245
|
);
|
|
693
1246
|
await this.waitForTxsPollingInterval();
|
|
694
1247
|
availableTxs = await this.p2pClient.getPendingTxCount();
|
|
@@ -697,15 +1250,47 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
697
1250
|
return { canStartBuilding: true, availableTxs, minTxs };
|
|
698
1251
|
}
|
|
699
1252
|
|
|
1253
|
+
private async getSignedCommitteeAttestations(
|
|
1254
|
+
broadcast: CheckpointProposalBroadcast,
|
|
1255
|
+
): Promise<{ attestations: CommitteeAttestationsAndSigners; attestationsSignature: Signature } | undefined> {
|
|
1256
|
+
const { proposal, blockProposedAt } = broadcast;
|
|
1257
|
+
this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.targetSlot, this.slotNow);
|
|
1258
|
+
const attestations = await this.waitForAttestations(proposal);
|
|
1259
|
+
if (!attestations) {
|
|
1260
|
+
return undefined;
|
|
1261
|
+
}
|
|
1262
|
+
this.checkpointMetrics.recordCheckpointAttestationDelay(this.dateProvider.now() - blockProposedAt);
|
|
1263
|
+
|
|
1264
|
+
// Proposer must sign over the attestations before pushing them to L1
|
|
1265
|
+
const signer = this.proposer ?? this.publisher.getSenderAddress();
|
|
1266
|
+
try {
|
|
1267
|
+
const attestationsSignature = await this.validatorClient.signAttestationsAndSigners(
|
|
1268
|
+
attestations,
|
|
1269
|
+
signer,
|
|
1270
|
+
this.targetSlot,
|
|
1271
|
+
this.checkpointNumber,
|
|
1272
|
+
);
|
|
1273
|
+
return { attestations, attestationsSignature };
|
|
1274
|
+
} catch (err) {
|
|
1275
|
+
if (this.handleHASigningError(err, 'Attestations signature')) {
|
|
1276
|
+
return;
|
|
1277
|
+
}
|
|
1278
|
+
this.log.error(`Error signing attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
|
|
1279
|
+
return undefined;
|
|
1280
|
+
}
|
|
1281
|
+
}
|
|
1282
|
+
|
|
700
1283
|
/**
|
|
701
1284
|
* Waits for enough attestations to be collected via p2p.
|
|
702
1285
|
* This is run after all blocks for the checkpoint have been built.
|
|
703
1286
|
*/
|
|
704
1287
|
@trackSpan('CheckpointProposalJob.waitForAttestations')
|
|
705
|
-
private async waitForAttestations(
|
|
1288
|
+
private async waitForAttestations(
|
|
1289
|
+
proposal: CheckpointProposal,
|
|
1290
|
+
): Promise<CommitteeAttestationsAndSigners | undefined> {
|
|
706
1291
|
if (this.config.fishermanMode) {
|
|
707
1292
|
this.log.debug('Skipping attestation collection in fisherman mode');
|
|
708
|
-
return CommitteeAttestationsAndSigners.empty();
|
|
1293
|
+
return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
|
|
709
1294
|
}
|
|
710
1295
|
|
|
711
1296
|
const slotNumber = proposal.slotNumber;
|
|
@@ -715,21 +1300,27 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
715
1300
|
throw new Error('No committee when collecting attestations');
|
|
716
1301
|
} else if (committee.length === 0) {
|
|
717
1302
|
this.log.verbose(`Attesting committee is empty`);
|
|
718
|
-
return CommitteeAttestationsAndSigners.empty();
|
|
1303
|
+
return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
|
|
719
1304
|
} else {
|
|
720
1305
|
this.log.debug(`Attesting committee length is ${committee.length}`, { committee });
|
|
721
1306
|
}
|
|
722
1307
|
|
|
723
|
-
const numberOfRequiredAttestations =
|
|
1308
|
+
const numberOfRequiredAttestations = computeQuorum(committee.length);
|
|
724
1309
|
|
|
725
1310
|
if (this.config.skipCollectingAttestations) {
|
|
726
1311
|
this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
|
|
727
|
-
const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
|
|
728
|
-
|
|
1312
|
+
const attestations = await this.validatorClient?.collectOwnAttestations(proposal, this.checkpointNumber);
|
|
1313
|
+
this.logCheckpointAttestations('collected', committee, attestations ?? [], numberOfRequiredAttestations, {
|
|
1314
|
+
reason: 'collect_own_only',
|
|
1315
|
+
});
|
|
1316
|
+
return new CommitteeAttestationsAndSigners(
|
|
1317
|
+
orderAttestations(attestations ?? [], committee),
|
|
1318
|
+
this.getSignatureContext(),
|
|
1319
|
+
);
|
|
729
1320
|
}
|
|
730
1321
|
|
|
731
1322
|
const attestationTimeAllowed = this.config.enforceTimeTable
|
|
732
|
-
? this.timetable.
|
|
1323
|
+
? this.timetable.getCheckpointAttestationDeadline()
|
|
733
1324
|
: this.l1Constants.slotDuration;
|
|
734
1325
|
const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
|
|
735
1326
|
|
|
@@ -742,6 +1333,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
742
1333
|
proposal,
|
|
743
1334
|
numberOfRequiredAttestations,
|
|
744
1335
|
attestationDeadline,
|
|
1336
|
+
this.checkpointNumber,
|
|
745
1337
|
);
|
|
746
1338
|
|
|
747
1339
|
collectedAttestationsCount = attestations.length;
|
|
@@ -760,6 +1352,9 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
760
1352
|
|
|
761
1353
|
// Rollup contract requires that the signatures are provided in the order of the committee
|
|
762
1354
|
const sorted = orderAttestations(trimmed, committee);
|
|
1355
|
+
this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
|
|
1356
|
+
submittedCount: trimmed.length,
|
|
1357
|
+
});
|
|
763
1358
|
|
|
764
1359
|
// Manipulate the attestations if we've been configured to do so
|
|
765
1360
|
if (
|
|
@@ -771,17 +1366,56 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
771
1366
|
return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
|
|
772
1367
|
}
|
|
773
1368
|
|
|
774
|
-
return new CommitteeAttestationsAndSigners(sorted);
|
|
1369
|
+
return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
|
|
775
1370
|
} catch (err) {
|
|
776
1371
|
if (err && err instanceof AttestationTimeoutError) {
|
|
777
1372
|
collectedAttestationsCount = err.collectedCount;
|
|
1373
|
+
this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
|
|
1374
|
+
collectedCount: collectedAttestationsCount,
|
|
1375
|
+
reason: 'timeout',
|
|
1376
|
+
});
|
|
1377
|
+
this.log.error(
|
|
1378
|
+
`Timeout while waiting for attestations for checkpoint proposal at slot ${proposal.slotNumber} (collected ${collectedAttestationsCount}/${numberOfRequiredAttestations})`,
|
|
1379
|
+
err,
|
|
1380
|
+
);
|
|
1381
|
+
} else {
|
|
1382
|
+
this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
|
|
1383
|
+
collectedCount: collectedAttestationsCount,
|
|
1384
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
1385
|
+
});
|
|
1386
|
+
this.log.error(`Error collecting attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
|
|
778
1387
|
}
|
|
779
|
-
|
|
1388
|
+
return undefined;
|
|
780
1389
|
} finally {
|
|
781
1390
|
this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
|
|
782
1391
|
}
|
|
783
1392
|
}
|
|
784
1393
|
|
|
1394
|
+
private logCheckpointAttestations(
|
|
1395
|
+
status: 'collected' | 'failed',
|
|
1396
|
+
committee: EthAddress[],
|
|
1397
|
+
attestations: CheckpointAttestation[] | undefined,
|
|
1398
|
+
requiredAttestations: number,
|
|
1399
|
+
opts: { collectedCount?: number; submittedCount?: number; reason?: string } = {},
|
|
1400
|
+
) {
|
|
1401
|
+
const signedValidators =
|
|
1402
|
+
attestations
|
|
1403
|
+
?.map(attestation => attestation.getSender()?.toString())
|
|
1404
|
+
.filter((address): address is `0x${string}` => address !== undefined) ?? [];
|
|
1405
|
+
const collectedCount = opts.collectedCount ?? new Set(signedValidators).size;
|
|
1406
|
+
const missingValidatorCount = status === 'failed' ? Math.max(0, requiredAttestations - collectedCount) : undefined;
|
|
1407
|
+
this.logCheckpointEvent(`attestations-${status}`, `Checkpoint attestations ${status} for slot ${this.targetSlot}`, {
|
|
1408
|
+
slot: this.targetSlot,
|
|
1409
|
+
checkpointNumber: this.checkpointNumber,
|
|
1410
|
+
committeeSize: committee.length,
|
|
1411
|
+
requiredAttestations,
|
|
1412
|
+
collectedAttestations: collectedCount,
|
|
1413
|
+
...(opts.submittedCount !== undefined && { submittedAttestations: opts.submittedCount }),
|
|
1414
|
+
...(missingValidatorCount !== undefined && { missingValidatorCount }),
|
|
1415
|
+
...(opts.reason !== undefined && { reason: opts.reason }),
|
|
1416
|
+
});
|
|
1417
|
+
}
|
|
1418
|
+
|
|
785
1419
|
/** Breaks the attestations before publishing based on attack configs */
|
|
786
1420
|
private manipulateAttestations(
|
|
787
1421
|
slotNumber: SlotNumber,
|
|
@@ -825,7 +1459,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
825
1459
|
unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
|
|
826
1460
|
}
|
|
827
1461
|
}
|
|
828
|
-
return new CommitteeAttestationsAndSigners(attestations);
|
|
1462
|
+
return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
|
|
829
1463
|
}
|
|
830
1464
|
|
|
831
1465
|
if (this.config.shuffleAttestationOrdering) {
|
|
@@ -847,11 +1481,11 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
847
1481
|
[shuffled[i], shuffled[j]] = [shuffled[j], shuffled[i]];
|
|
848
1482
|
}
|
|
849
1483
|
|
|
850
|
-
const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
|
|
851
|
-
return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
|
|
1484
|
+
const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
|
|
1485
|
+
return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
|
|
852
1486
|
}
|
|
853
1487
|
|
|
854
|
-
return new CommitteeAttestationsAndSigners(attestations);
|
|
1488
|
+
return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
|
|
855
1489
|
}
|
|
856
1490
|
|
|
857
1491
|
private async dropFailedTxsFromP2P(failedTxs: FailedTx[]) {
|
|
@@ -868,9 +1502,13 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
868
1502
|
* Adds the proposed block to the archiver so it's available via P2P.
|
|
869
1503
|
* Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
|
|
870
1504
|
* would never receive its own block without this explicit sync.
|
|
1505
|
+
*
|
|
1506
|
+
* In fisherman mode we skip this push: the fisherman builds blocks locally for validation
|
|
1507
|
+
* and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
|
|
1508
|
+
* whenever the real proposer's block arrives from L1.
|
|
871
1509
|
*/
|
|
872
1510
|
private async syncProposedBlockToArchiver(block: L2Block): Promise<void> {
|
|
873
|
-
if (this.config.skipPushProposedBlocksToArchiver
|
|
1511
|
+
if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
|
|
874
1512
|
this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
|
|
875
1513
|
blockNumber: block.number,
|
|
876
1514
|
slot: block.header.globalVariables.slotNumber,
|
|
@@ -884,23 +1522,58 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
884
1522
|
await this.blockSink.addBlock(block);
|
|
885
1523
|
}
|
|
886
1524
|
|
|
1525
|
+
/**
|
|
1526
|
+
* Adds the proposed checkpoint to the archiver so the proposer's optimistic proposed-checkpoint
|
|
1527
|
+
* tip advances locally. Gossip doesn't echo our own messages back, so without this the proposer
|
|
1528
|
+
* would never see its own proposed checkpoint and couldn't pipeline the next slot.
|
|
1529
|
+
*
|
|
1530
|
+
* Skipped whenever proposed blocks aren't pushed (`skipPushProposedBlocksToArchiver`, fisherman
|
|
1531
|
+
* mode): the archiver derives the checkpoint archive from its stored blocks, so without them the
|
|
1532
|
+
* push would fail. All blocks were already added (and awaited) during block building, so this
|
|
1533
|
+
* needs no retry — they are guaranteed present by the time we get here.
|
|
1534
|
+
*/
|
|
1535
|
+
private async syncProposedCheckpointToArchiver(
|
|
1536
|
+
checkpoint: Checkpoint,
|
|
1537
|
+
blockCount: number,
|
|
1538
|
+
feeAssetPriceModifier: bigint,
|
|
1539
|
+
): Promise<void> {
|
|
1540
|
+
if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
|
|
1541
|
+
return;
|
|
1542
|
+
}
|
|
1543
|
+
const startBlock = BlockNumber(this.syncedToBlockNumber + 1);
|
|
1544
|
+
this.log.debug(`Syncing proposed checkpoint ${this.checkpointNumber} to archiver`, {
|
|
1545
|
+
checkpointNumber: this.checkpointNumber,
|
|
1546
|
+
slot: this.targetSlot,
|
|
1547
|
+
startBlock,
|
|
1548
|
+
blockCount,
|
|
1549
|
+
});
|
|
1550
|
+
await this.blockSink.addProposedCheckpoint({
|
|
1551
|
+
header: checkpoint.header,
|
|
1552
|
+
checkpointNumber: this.checkpointNumber,
|
|
1553
|
+
startBlock,
|
|
1554
|
+
blockCount,
|
|
1555
|
+
totalManaUsed: checkpoint.header.totalManaUsed.toBigInt(),
|
|
1556
|
+
feeAssetPriceModifier,
|
|
1557
|
+
});
|
|
1558
|
+
}
|
|
1559
|
+
|
|
887
1560
|
/** Runs fee analysis and logs checkpoint outcome as fisherman */
|
|
888
1561
|
private async handleCheckpointEndAsFisherman(checkpoint: Checkpoint | undefined) {
|
|
889
1562
|
// Perform L1 fee analysis before clearing requests
|
|
890
1563
|
// The callback is invoked asynchronously after the next block is mined
|
|
891
|
-
const feeAnalysis = await this.publisher.analyzeL1Fees(this.
|
|
1564
|
+
const feeAnalysis = await this.publisher.analyzeL1Fees(this.targetSlot, analysis =>
|
|
892
1565
|
this.metrics.recordFishermanFeeAnalysis(analysis),
|
|
893
1566
|
);
|
|
894
1567
|
|
|
895
1568
|
if (checkpoint) {
|
|
896
|
-
this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.
|
|
1569
|
+
this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.targetSlot}`, {
|
|
897
1570
|
...checkpoint.toCheckpointInfo(),
|
|
898
1571
|
...checkpoint.getStats(),
|
|
899
1572
|
feeAnalysisId: feeAnalysis?.id,
|
|
900
1573
|
});
|
|
901
1574
|
} else {
|
|
902
|
-
this.log.warn(`Validation block building FAILED for slot ${this.
|
|
903
|
-
slot: this.
|
|
1575
|
+
this.log.warn(`Validation block building FAILED for slot ${this.targetSlot}`, {
|
|
1576
|
+
slot: this.targetSlot,
|
|
904
1577
|
feeAnalysisId: feeAnalysis?.id,
|
|
905
1578
|
});
|
|
906
1579
|
this.metrics.recordCheckpointProposalFailed('block_build_failed');
|
|
@@ -914,15 +1587,15 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
914
1587
|
*/
|
|
915
1588
|
private handleHASigningError(err: any, errorContext: string): boolean {
|
|
916
1589
|
if (err instanceof DutyAlreadySignedError) {
|
|
917
|
-
this.log.info(`${errorContext} for slot ${this.
|
|
918
|
-
slot: this.
|
|
1590
|
+
this.log.info(`${errorContext} for slot ${this.targetSlot} already signed by another HA node, yielding`, {
|
|
1591
|
+
slot: this.targetSlot,
|
|
919
1592
|
signedByNode: err.signedByNode,
|
|
920
1593
|
});
|
|
921
1594
|
return true;
|
|
922
1595
|
}
|
|
923
1596
|
if (err instanceof SlashingProtectionError) {
|
|
924
|
-
this.log.info(`${errorContext} for slot ${this.
|
|
925
|
-
slot: this.
|
|
1597
|
+
this.log.info(`${errorContext} for slot ${this.targetSlot} blocked by slashing protection, yielding`, {
|
|
1598
|
+
slot: this.targetSlot,
|
|
926
1599
|
existingMessageHash: err.existingMessageHash,
|
|
927
1600
|
attemptedMessageHash: err.attemptedMessageHash,
|
|
928
1601
|
});
|
|
@@ -936,16 +1609,17 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
936
1609
|
protected async waitUntilTimeInSlot(targetSecondsIntoSlot: number): Promise<void> {
|
|
937
1610
|
const slotStartTimestamp = this.getSlotStartBuildTimestamp();
|
|
938
1611
|
const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
|
|
939
|
-
|
|
1612
|
+
const delayMs = targetTimestamp * 1000 - this.dateProvider.nowAsDate().getTime();
|
|
1613
|
+
await this.awaitInterruptibleSleep(delayMs);
|
|
940
1614
|
}
|
|
941
1615
|
|
|
942
|
-
/**
|
|
1616
|
+
/** Pause between mempool polls in waitForMinTxs. Extracted for test overriding. */
|
|
943
1617
|
protected async waitForTxsPollingInterval(): Promise<void> {
|
|
944
|
-
await
|
|
1618
|
+
await this.awaitInterruptibleSleep(TXS_POLLING_MS);
|
|
945
1619
|
}
|
|
946
1620
|
|
|
947
1621
|
private getSlotStartBuildTimestamp(): number {
|
|
948
|
-
return getSlotStartBuildTimestamp(this.
|
|
1622
|
+
return getSlotStartBuildTimestamp(this.slotNow, this.l1Constants);
|
|
949
1623
|
}
|
|
950
1624
|
|
|
951
1625
|
private getSecondsIntoSlot(): number {
|