@aztec/sequencer-client 0.0.1-commit.181e2d196 → 0.0.1-commit.189eedb3
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 -3
- package/dest/client/sequencer-client.d.ts.map +1 -1
- package/dest/client/sequencer-client.js +51 -29
- package/dest/config.d.ts +26 -3
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +51 -22
- 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 -12
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_proposal_job.js +774 -219
- 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 +68 -17
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +359 -119
- package/dest/sequencer/timetable.d.ts +14 -3
- package/dest/sequencer/timetable.d.ts.map +1 -1
- package/dest/sequencer/timetable.js +47 -43
- 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 +7 -9
- package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
- package/dest/test/mock_checkpoint_builder.js +39 -30
- 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 +71 -33
- package/src/config.ts +61 -22
- 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 +923 -251
- 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 +440 -134
- package/src/sequencer/timetable.ts +56 -51
- package/src/sequencer/types.ts +1 -1
- package/src/test/mock_checkpoint_builder.ts +51 -48
- package/src/test/utils.ts +28 -10
|
@@ -1,6 +1,5 @@
|
|
|
1
|
-
import { NUM_CHECKPOINT_END_MARKER_FIELDS, getNumBlockEndBlobFields } from '@aztec/blob-lib/encoding';
|
|
2
|
-
import { BLOBS_PER_CHECKPOINT, FIELDS_PER_BLOB } from '@aztec/constants';
|
|
3
1
|
import type { EpochCache } from '@aztec/epoch-cache';
|
|
2
|
+
import type { SimulationOverridesPlan } from '@aztec/ethereum/contracts';
|
|
4
3
|
import {
|
|
5
4
|
BlockNumber,
|
|
6
5
|
CheckpointNumber,
|
|
@@ -15,11 +14,12 @@ import {
|
|
|
15
14
|
generateUnrecoverableSignature,
|
|
16
15
|
} from '@aztec/foundation/crypto/secp256k1-signer';
|
|
17
16
|
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
17
|
+
import { InterruptError, TimeoutError } from '@aztec/foundation/error';
|
|
18
18
|
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
19
19
|
import { Signature } from '@aztec/foundation/eth-signature';
|
|
20
20
|
import { filter } from '@aztec/foundation/iterator';
|
|
21
21
|
import { type Logger, type LoggerBindings, createLogger } from '@aztec/foundation/log';
|
|
22
|
-
import {
|
|
22
|
+
import { InterruptibleSleep } from '@aztec/foundation/sleep';
|
|
23
23
|
import { type DateProvider, Timer } from '@aztec/foundation/timer';
|
|
24
24
|
import { type TypedEventEmitter, isErrorClass, unfreeze } from '@aztec/foundation/types';
|
|
25
25
|
import type { P2P } from '@aztec/p2p';
|
|
@@ -31,18 +31,32 @@ import {
|
|
|
31
31
|
type L2BlockSink,
|
|
32
32
|
type L2BlockSource,
|
|
33
33
|
MaliciousCommitteeAttestationsAndSigners,
|
|
34
|
+
type ProposedCheckpointSink,
|
|
35
|
+
type ValidateCheckpointResult,
|
|
34
36
|
} from '@aztec/stdlib/block';
|
|
35
|
-
import
|
|
36
|
-
|
|
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';
|
|
37
44
|
import { Gas } from '@aztec/stdlib/gas';
|
|
38
45
|
import {
|
|
39
|
-
|
|
40
|
-
|
|
46
|
+
type BlockBuilderOptions,
|
|
47
|
+
InsufficientValidTxsError,
|
|
41
48
|
type ResolvedSequencerConfig,
|
|
42
49
|
type WorldStateSynchronizer,
|
|
43
50
|
} from '@aztec/stdlib/interfaces/server';
|
|
44
51
|
import { type L1ToL2MessageSource, computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
|
|
45
|
-
import type {
|
|
52
|
+
import type {
|
|
53
|
+
BlockProposal,
|
|
54
|
+
BlockProposalOptions,
|
|
55
|
+
CheckpointAttestation,
|
|
56
|
+
CheckpointProposal,
|
|
57
|
+
CheckpointProposalOptions,
|
|
58
|
+
CoordinationSignatureContext,
|
|
59
|
+
} from '@aztec/stdlib/p2p';
|
|
46
60
|
import { orderAttestations, trimAttestations } from '@aztec/stdlib/p2p';
|
|
47
61
|
import type { L2BlockBuiltStats } from '@aztec/stdlib/stats';
|
|
48
62
|
import { type FailedTx, Tx } from '@aztec/stdlib/tx';
|
|
@@ -53,6 +67,8 @@ import { DutyAlreadySignedError, SlashingProtectionError } from '@aztec/validato
|
|
|
53
67
|
|
|
54
68
|
import type { GlobalVariableBuilder } from '../global_variable_builder/global_builder.js';
|
|
55
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';
|
|
56
72
|
import { CheckpointVoter } from './checkpoint_voter.js';
|
|
57
73
|
import { SequencerInterruptedError } from './errors.js';
|
|
58
74
|
import type { SequencerEvents } from './events.js';
|
|
@@ -63,6 +79,21 @@ import { SequencerState } from './utils.js';
|
|
|
63
79
|
|
|
64
80
|
/** How much time to sleep while waiting for min transactions to accumulate for a block */
|
|
65
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
|
+
};
|
|
66
97
|
|
|
67
98
|
/**
|
|
68
99
|
* Handles the execution of a checkpoint proposal after the initial preparation phase.
|
|
@@ -72,12 +103,32 @@ const TXS_POLLING_MS = 500;
|
|
|
72
103
|
*/
|
|
73
104
|
export class CheckpointProposalJob implements Traceable {
|
|
74
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
|
+
}
|
|
75
124
|
|
|
76
125
|
constructor(
|
|
77
|
-
private readonly
|
|
78
|
-
private readonly
|
|
126
|
+
private readonly slotNow: SlotNumber,
|
|
127
|
+
private readonly targetSlot: SlotNumber,
|
|
128
|
+
private readonly targetEpoch: EpochNumber,
|
|
79
129
|
private readonly checkpointNumber: CheckpointNumber,
|
|
80
130
|
private readonly syncedToBlockNumber: BlockNumber,
|
|
131
|
+
private readonly checkpointedCheckpointNumber: CheckpointNumber,
|
|
81
132
|
// TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
|
|
82
133
|
private readonly proposer: EthAddress | undefined,
|
|
83
134
|
private readonly publisher: SequencerPublisher,
|
|
@@ -90,25 +141,75 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
90
141
|
private readonly l1ToL2MessageSource: L1ToL2MessageSource,
|
|
91
142
|
private readonly l2BlockSource: L2BlockSource,
|
|
92
143
|
private readonly checkpointsBuilder: FullNodeCheckpointsBuilder,
|
|
93
|
-
private readonly blockSink: L2BlockSink,
|
|
144
|
+
private readonly blockSink: L2BlockSink & ProposedCheckpointSink,
|
|
94
145
|
private readonly l1Constants: SequencerRollupConstants,
|
|
146
|
+
private readonly signatureContext: CoordinationSignatureContext,
|
|
95
147
|
protected config: ResolvedSequencerConfig,
|
|
96
148
|
protected timetable: SequencerTimetable,
|
|
97
149
|
private readonly slasherClient: SlasherClientInterface | undefined,
|
|
98
150
|
private readonly epochCache: EpochCache,
|
|
99
151
|
private readonly dateProvider: DateProvider,
|
|
100
152
|
private readonly metrics: SequencerMetrics,
|
|
101
|
-
private readonly
|
|
102
|
-
|
|
153
|
+
private readonly checkpointMetrics: CheckpointProposalJobMetricsRecorder,
|
|
154
|
+
protected readonly eventEmitter: TypedEventEmitter<SequencerEvents>,
|
|
155
|
+
private readonly setStateFn: (state: SequencerState, slot?: SlotNumber, timeReferenceSlot?: SlotNumber) => void,
|
|
103
156
|
public readonly tracer: Tracer,
|
|
104
157
|
bindings?: LoggerBindings,
|
|
158
|
+
private readonly proposedCheckpointData?: ProposedCheckpointData,
|
|
105
159
|
) {
|
|
106
|
-
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
|
+
});
|
|
107
205
|
}
|
|
108
206
|
|
|
109
207
|
/**
|
|
110
208
|
* Executes the checkpoint proposal job.
|
|
111
|
-
*
|
|
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.
|
|
112
213
|
*/
|
|
113
214
|
@trackSpan('CheckpointProposalJob.execute')
|
|
114
215
|
public async execute(): Promise<Checkpoint | undefined> {
|
|
@@ -116,7 +217,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
116
217
|
// In fisherman mode, we simulate slashing but don't actually publish to L1
|
|
117
218
|
// These are constant for the whole slot, so we only enqueue them once
|
|
118
219
|
const votesPromises = new CheckpointVoter(
|
|
119
|
-
this.
|
|
220
|
+
this.targetSlot,
|
|
120
221
|
this.publisher,
|
|
121
222
|
this.attestorAddress,
|
|
122
223
|
this.validatorClient,
|
|
@@ -127,33 +228,302 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
127
228
|
this.log,
|
|
128
229
|
).enqueueVotes();
|
|
129
230
|
|
|
130
|
-
// Build
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
135
|
-
|
|
136
|
-
|
|
137
|
-
|
|
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;
|
|
138
247
|
}
|
|
139
248
|
|
|
249
|
+
const { checkpoint } = broadcast;
|
|
250
|
+
this.metrics.recordCheckpointProposalSuccess();
|
|
251
|
+
|
|
140
252
|
// Do not post anything to L1 if we are fishermen, but do perform L1 fee analysis
|
|
141
253
|
if (this.config.fishermanMode) {
|
|
142
254
|
await this.handleCheckpointEndAsFisherman(checkpoint);
|
|
143
|
-
return;
|
|
255
|
+
return checkpoint;
|
|
144
256
|
}
|
|
145
257
|
|
|
146
|
-
//
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
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
|
+
});
|
|
157
527
|
}
|
|
158
528
|
}
|
|
159
529
|
|
|
@@ -161,42 +531,89 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
161
531
|
return {
|
|
162
532
|
// nullish operator needed for tests
|
|
163
533
|
[Attributes.COINBASE]: this.validatorClient.getCoinbaseForAttestor(this.attestorAddress)?.toString(),
|
|
164
|
-
[Attributes.SLOT_NUMBER]: this.
|
|
534
|
+
[Attributes.SLOT_NUMBER]: this.targetSlot,
|
|
165
535
|
};
|
|
166
536
|
})
|
|
167
|
-
private async proposeCheckpoint(): Promise<
|
|
537
|
+
private async proposeCheckpoint(): Promise<CheckpointProposalBroadcast | undefined> {
|
|
168
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
|
+
|
|
169
549
|
// Get operator configured coinbase and fee recipient for this attestor
|
|
170
550
|
const coinbase = this.validatorClient.getCoinbaseForAttestor(this.attestorAddress);
|
|
171
551
|
const feeRecipient = this.validatorClient.getFeeRecipientForAttestor(this.attestorAddress);
|
|
172
552
|
|
|
173
553
|
// Start the checkpoint
|
|
174
|
-
this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.
|
|
175
|
-
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');
|
|
176
566
|
|
|
177
567
|
// Enqueues checkpoint invalidation (constant for the whole slot)
|
|
178
568
|
if (this.invalidateCheckpoint && !this.config.skipInvalidateBlockAsProposer) {
|
|
179
569
|
this.publisher.enqueueInvalidateCheckpoint(this.invalidateCheckpoint);
|
|
180
570
|
}
|
|
181
571
|
|
|
182
|
-
//
|
|
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
|
+
|
|
183
586
|
const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(
|
|
184
587
|
coinbase,
|
|
185
588
|
feeRecipient,
|
|
186
|
-
this.
|
|
589
|
+
this.targetSlot,
|
|
590
|
+
this.checkpointSimulationOverridesPlan,
|
|
187
591
|
);
|
|
188
592
|
|
|
189
593
|
// Collect L1 to L2 messages for the checkpoint and compute their hash
|
|
190
594
|
const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(this.checkpointNumber);
|
|
191
595
|
const inHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
|
|
192
596
|
|
|
193
|
-
// Collect the out hashes of all the checkpoints before this one in the same epoch
|
|
194
|
-
|
|
195
|
-
|
|
196
|
-
|
|
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
|
+
});
|
|
197
611
|
|
|
198
|
-
//
|
|
199
|
-
|
|
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);
|
|
200
617
|
|
|
201
618
|
// Create a long-lived forked world state for the checkpoint builder
|
|
202
619
|
await using fork = await this.worldState.fork(this.syncedToBlockNumber, { closeDelayMs: 12_000 });
|
|
@@ -220,11 +637,12 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
220
637
|
|
|
221
638
|
const checkpointProposalOptions: CheckpointProposalOptions = {
|
|
222
639
|
publishFullTxs: !!this.config.publishTxsWithProposals,
|
|
223
|
-
broadcastInvalidCheckpointProposal:
|
|
640
|
+
broadcastInvalidCheckpointProposal:
|
|
641
|
+
this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal,
|
|
224
642
|
};
|
|
225
643
|
|
|
226
644
|
let blocksInCheckpoint: L2Block[] = [];
|
|
227
|
-
let blockPendingBroadcast:
|
|
645
|
+
let blockPendingBroadcast: BlockProposal | undefined = undefined;
|
|
228
646
|
const checkpointBuildTimer = new Timer();
|
|
229
647
|
|
|
230
648
|
try {
|
|
@@ -248,126 +666,170 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
248
666
|
}
|
|
249
667
|
|
|
250
668
|
if (blocksInCheckpoint.length === 0) {
|
|
251
|
-
this.
|
|
252
|
-
|
|
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 });
|
|
253
680
|
return undefined;
|
|
254
681
|
}
|
|
255
682
|
|
|
256
683
|
const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
|
|
257
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
|
+
});
|
|
258
692
|
this.log.warn(
|
|
259
693
|
`Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`,
|
|
260
|
-
{
|
|
694
|
+
{
|
|
695
|
+
slot: this.targetSlot,
|
|
696
|
+
checkpointNumber: this.checkpointNumber,
|
|
697
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
698
|
+
minBlocksForCheckpoint,
|
|
699
|
+
reason: 'min_blocks_not_met',
|
|
700
|
+
},
|
|
261
701
|
);
|
|
262
702
|
return undefined;
|
|
263
703
|
}
|
|
264
704
|
|
|
265
705
|
// Assemble and broadcast the checkpoint proposal, including the last block that was not
|
|
266
706
|
// broadcasted yet, and wait to collect the committee attestations.
|
|
267
|
-
this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.
|
|
707
|
+
this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.targetSlot, this.slotNow);
|
|
268
708
|
const checkpoint = await checkpointBuilder.completeCheckpoint();
|
|
269
709
|
|
|
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.
|
|
712
|
+
try {
|
|
713
|
+
validateCheckpoint(checkpoint, {
|
|
714
|
+
rollupManaLimit: this.l1Constants.rollupManaLimit,
|
|
715
|
+
maxL2BlockGas: this.config.maxL2BlockGas,
|
|
716
|
+
maxDABlockGas: this.config.maxDABlockGas,
|
|
717
|
+
maxTxsPerBlock: this.config.maxTxsPerBlock,
|
|
718
|
+
maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint,
|
|
719
|
+
});
|
|
720
|
+
} catch (err) {
|
|
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',
|
|
733
|
+
checkpoint: checkpoint.header.toInspect(),
|
|
734
|
+
});
|
|
735
|
+
return undefined;
|
|
736
|
+
}
|
|
737
|
+
|
|
270
738
|
// Record checkpoint-level build metrics
|
|
271
|
-
this.
|
|
739
|
+
this.checkpointMetrics.recordCheckpointBuild(
|
|
272
740
|
checkpointBuildTimer.ms(),
|
|
273
741
|
blocksInCheckpoint.length,
|
|
274
742
|
checkpoint.getStats().txCount,
|
|
275
743
|
Number(checkpoint.header.totalManaUsed.toBigInt()),
|
|
276
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
|
+
});
|
|
277
756
|
|
|
278
|
-
//
|
|
757
|
+
// In fisherman mode, return the checkpoint without broadcasting or collecting attestations
|
|
279
758
|
if (this.config.fishermanMode) {
|
|
280
759
|
this.log.info(
|
|
281
|
-
`Built checkpoint for slot ${this.
|
|
760
|
+
`Built checkpoint for slot ${this.targetSlot} with ${blocksInCheckpoint.length} blocks. ` +
|
|
282
761
|
`Skipping proposal in fisherman mode.`,
|
|
283
762
|
{
|
|
284
|
-
slot: this.
|
|
763
|
+
slot: this.targetSlot,
|
|
285
764
|
checkpoint: checkpoint.header.toInspect(),
|
|
286
765
|
blocksBuilt: blocksInCheckpoint.length,
|
|
287
766
|
},
|
|
288
767
|
);
|
|
289
768
|
this.metrics.recordCheckpointSuccess();
|
|
290
|
-
|
|
769
|
+
// Return a broadcast result with a dummy proposal — fisherman mode skips attestation collection
|
|
770
|
+
return { checkpoint, proposal: undefined!, blockProposedAt: this.dateProvider.now() };
|
|
291
771
|
}
|
|
292
772
|
|
|
293
|
-
//
|
|
294
|
-
|
|
295
|
-
|
|
296
|
-
|
|
297
|
-
|
|
298
|
-
}
|
|
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
|
+
}
|
|
299
791
|
|
|
300
792
|
// Create the checkpoint proposal and broadcast it
|
|
301
793
|
const proposal = await this.validatorClient.createCheckpointProposal(
|
|
302
794
|
checkpoint.header,
|
|
303
795
|
checkpoint.archive.root,
|
|
796
|
+
this.checkpointNumber,
|
|
304
797
|
feeAssetPriceModifier,
|
|
305
|
-
|
|
798
|
+
blockPendingBroadcast,
|
|
306
799
|
this.proposer,
|
|
307
800
|
checkpointProposalOptions,
|
|
308
801
|
);
|
|
309
802
|
|
|
310
|
-
|
|
311
|
-
|
|
312
|
-
|
|
313
|
-
|
|
314
|
-
|
|
315
|
-
|
|
316
|
-
|
|
317
|
-
this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
|
|
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);
|
|
318
809
|
|
|
319
|
-
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
325
|
-
|
|
326
|
-
this.
|
|
327
|
-
this.checkpointNumber,
|
|
328
|
-
);
|
|
329
|
-
} catch (err) {
|
|
330
|
-
// We shouldn't really get here since we yield to another HA node
|
|
331
|
-
// as soon as we see these errors when creating block or checkpoint proposals.
|
|
332
|
-
if (this.handleHASigningError(err, 'Attestations signature')) {
|
|
333
|
-
return undefined;
|
|
334
|
-
}
|
|
335
|
-
throw err;
|
|
336
|
-
}
|
|
337
|
-
|
|
338
|
-
// Enqueue publishing the checkpoint to L1
|
|
339
|
-
this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.slot);
|
|
340
|
-
const aztecSlotDuration = this.l1Constants.slotDuration;
|
|
341
|
-
const slotStartBuildTimestamp = this.getSlotStartBuildTimestamp();
|
|
342
|
-
const txTimeoutAt = new Date((slotStartBuildTimestamp + aztecSlotDuration) * 1000);
|
|
343
|
-
|
|
344
|
-
// If we have been configured to potentially skip publishing checkpoint then roll the dice here
|
|
345
|
-
if (
|
|
346
|
-
this.config.skipPublishingCheckpointsPercent !== undefined &&
|
|
347
|
-
this.config.skipPublishingCheckpointsPercent > 0
|
|
348
|
-
) {
|
|
349
|
-
const result = Math.max(0, randomInt(100));
|
|
350
|
-
if (result < this.config.skipPublishingCheckpointsPercent) {
|
|
351
|
-
this.log.warn(
|
|
352
|
-
`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${result}`,
|
|
353
|
-
);
|
|
354
|
-
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);
|
|
355
818
|
}
|
|
819
|
+
} else if (!this.config.skipBroadcastProposals) {
|
|
820
|
+
await this.p2pClient.broadcastCheckpointProposal(proposal);
|
|
821
|
+
this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
|
|
356
822
|
}
|
|
357
823
|
|
|
358
|
-
|
|
359
|
-
|
|
360
|
-
forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
|
|
361
|
-
});
|
|
362
|
-
|
|
363
|
-
return checkpoint;
|
|
824
|
+
// Return immediately after broadcast — attestation collection happens in the background
|
|
825
|
+
return { checkpoint, proposal, blockProposedAt };
|
|
364
826
|
} catch (err) {
|
|
365
827
|
if (err && (err instanceof DutyAlreadySignedError || err instanceof SlashingProtectionError)) {
|
|
366
828
|
// swallow this error. It's already been logged by a function deeper in the stack
|
|
367
829
|
return undefined;
|
|
368
830
|
}
|
|
369
831
|
|
|
370
|
-
this.log.error(`Error building checkpoint at slot ${this.
|
|
832
|
+
this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
|
|
371
833
|
return undefined;
|
|
372
834
|
}
|
|
373
835
|
}
|
|
@@ -383,29 +845,35 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
383
845
|
blockProposalOptions: BlockProposalOptions,
|
|
384
846
|
): Promise<{
|
|
385
847
|
blocksInCheckpoint: L2Block[];
|
|
386
|
-
blockPendingBroadcast:
|
|
848
|
+
blockPendingBroadcast: BlockProposal | undefined;
|
|
387
849
|
}> {
|
|
388
850
|
const blocksInCheckpoint: L2Block[] = [];
|
|
389
851
|
const txHashesAlreadyIncluded = new Set<string>();
|
|
390
852
|
const initialBlockNumber = BlockNumber(this.syncedToBlockNumber + 1);
|
|
391
853
|
|
|
392
|
-
// Remaining blob fields available for blocks (checkpoint end marker already subtracted)
|
|
393
|
-
let remainingBlobFields = BLOBS_PER_CHECKPOINT * FIELDS_PER_BLOB - NUM_CHECKPOINT_END_MARKER_FIELDS;
|
|
394
|
-
|
|
395
854
|
// Last block in the checkpoint will usually be flagged as pending broadcast, so we send it along with the checkpoint proposal
|
|
396
|
-
let blockPendingBroadcast:
|
|
855
|
+
let blockPendingBroadcast: BlockProposal | undefined = undefined;
|
|
397
856
|
|
|
398
857
|
while (true) {
|
|
399
858
|
const blocksBuilt = blocksInCheckpoint.length;
|
|
400
859
|
const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
|
|
401
860
|
const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
|
|
402
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
|
+
|
|
403
871
|
const secondsIntoSlot = this.getSecondsIntoSlot();
|
|
404
872
|
const timingInfo = this.timetable.canStartNextBlock(secondsIntoSlot);
|
|
405
873
|
|
|
406
874
|
if (!timingInfo.canStart) {
|
|
407
875
|
this.log.debug(`Not enough time left in slot to start another block`, {
|
|
408
|
-
slot: this.
|
|
876
|
+
slot: this.targetSlot,
|
|
409
877
|
blocksBuilt,
|
|
410
878
|
secondsIntoSlot,
|
|
411
879
|
});
|
|
@@ -424,25 +892,25 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
424
892
|
blockNumber,
|
|
425
893
|
indexWithinCheckpoint,
|
|
426
894
|
txHashesAlreadyIncluded,
|
|
427
|
-
remainingBlobFields,
|
|
428
895
|
});
|
|
429
896
|
|
|
430
|
-
//
|
|
431
|
-
if (
|
|
432
|
-
// If
|
|
433
|
-
|
|
434
|
-
|
|
435
|
-
|
|
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
|
|
436
904
|
await this.waitUntilNextSubslot(timingInfo.deadline);
|
|
437
905
|
continue;
|
|
438
|
-
}
|
|
439
|
-
|
|
440
|
-
|
|
441
|
-
|
|
442
|
-
|
|
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) {
|
|
443
911
|
if (!(buildResult.error instanceof SequencerInterruptedError)) {
|
|
444
|
-
this.log.warn(`Halting block building for slot ${this.
|
|
445
|
-
slot: this.
|
|
912
|
+
this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
|
|
913
|
+
slot: this.targetSlot,
|
|
446
914
|
blocksBuilt,
|
|
447
915
|
error: buildResult.error,
|
|
448
916
|
});
|
|
@@ -450,45 +918,43 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
450
918
|
break;
|
|
451
919
|
}
|
|
452
920
|
|
|
453
|
-
const { block, usedTxs
|
|
454
|
-
|
|
921
|
+
const { block, usedTxs } = buildResult;
|
|
922
|
+
this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
|
|
923
|
+
isFirstBlock: blocksBuilt === 0,
|
|
924
|
+
isLastBlock: timingInfo.isLastBlock,
|
|
925
|
+
});
|
|
455
926
|
|
|
456
|
-
|
|
457
|
-
|
|
927
|
+
blocksInCheckpoint.push(block);
|
|
928
|
+
usedTxs.forEach(tx => txHashesAlreadyIncluded.add(tx.txHash.toString()));
|
|
458
929
|
|
|
459
|
-
//
|
|
460
|
-
|
|
461
|
-
|
|
462
|
-
|
|
463
|
-
|
|
464
|
-
|
|
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,
|
|
465
936
|
});
|
|
466
937
|
|
|
467
|
-
|
|
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);
|
|
468
943
|
|
|
469
|
-
// 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.
|
|
470
945
|
if (timingInfo.isLastBlock) {
|
|
471
|
-
this.log.verbose(`Completed final block ${blockNumber} for slot ${this.
|
|
472
|
-
slot: this.
|
|
946
|
+
this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
|
|
947
|
+
slot: this.targetSlot,
|
|
473
948
|
blockNumber,
|
|
474
949
|
blocksBuilt,
|
|
475
950
|
});
|
|
476
|
-
|
|
951
|
+
|
|
952
|
+
blockPendingBroadcast = proposal;
|
|
477
953
|
break;
|
|
478
954
|
}
|
|
479
955
|
|
|
480
|
-
//
|
|
481
|
-
|
|
482
|
-
if (!this.config.fishermanMode) {
|
|
483
|
-
const proposal = await this.validatorClient.createBlockProposal(
|
|
484
|
-
block.header,
|
|
485
|
-
block.indexWithinCheckpoint,
|
|
486
|
-
inHash,
|
|
487
|
-
block.archive.root,
|
|
488
|
-
usedTxs,
|
|
489
|
-
this.proposer,
|
|
490
|
-
blockProposalOptions,
|
|
491
|
-
);
|
|
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) {
|
|
492
958
|
await this.p2pClient.broadcastProposal(proposal);
|
|
493
959
|
}
|
|
494
960
|
|
|
@@ -496,19 +962,44 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
496
962
|
await this.waitUntilNextSubslot(timingInfo.deadline);
|
|
497
963
|
}
|
|
498
964
|
|
|
499
|
-
this.log.verbose(`Block building loop completed for slot ${this.
|
|
500
|
-
slot: this.
|
|
965
|
+
this.log.verbose(`Block building loop completed for slot ${this.targetSlot}`, {
|
|
966
|
+
slot: this.targetSlot,
|
|
501
967
|
blocksBuilt: blocksInCheckpoint.length,
|
|
502
968
|
});
|
|
503
969
|
|
|
504
970
|
return { blocksInCheckpoint, blockPendingBroadcast };
|
|
505
971
|
}
|
|
506
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
|
+
|
|
507
996
|
/** Sleeps until it is time to produce the next block in the slot */
|
|
508
997
|
@trackSpan('CheckpointProposalJob.waitUntilNextSubslot')
|
|
509
998
|
private async waitUntilNextSubslot(nextSubslotStart: number) {
|
|
510
|
-
this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.
|
|
511
|
-
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
|
+
});
|
|
512
1003
|
await this.waitUntilTimeInSlot(nextSubslotStart);
|
|
513
1004
|
}
|
|
514
1005
|
|
|
@@ -523,36 +1014,46 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
523
1014
|
indexWithinCheckpoint: IndexWithinCheckpoint;
|
|
524
1015
|
buildDeadline: Date | undefined;
|
|
525
1016
|
txHashesAlreadyIncluded: Set<string>;
|
|
526
|
-
remainingBlobFields: number;
|
|
527
1017
|
},
|
|
528
|
-
): Promise<
|
|
529
|
-
|
|
530
|
-
|
|
531
|
-
|
|
532
|
-
|
|
533
|
-
indexWithinCheckpoint,
|
|
534
|
-
buildDeadline,
|
|
535
|
-
txHashesAlreadyIncluded,
|
|
536
|
-
remainingBlobFields,
|
|
537
|
-
} = opts;
|
|
1018
|
+
): Promise<
|
|
1019
|
+
{ block: L2Block; usedTxs: Tx[] } | { failure: 'insufficient-txs' | 'insufficient-valid-txs' } | { error: Error }
|
|
1020
|
+
> {
|
|
1021
|
+
const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } =
|
|
1022
|
+
opts;
|
|
538
1023
|
|
|
539
1024
|
this.log.verbose(
|
|
540
|
-
`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}`,
|
|
541
1026
|
{ ...checkpointBuilder.getConstantData(), ...opts },
|
|
542
1027
|
);
|
|
543
1028
|
|
|
544
1029
|
try {
|
|
545
1030
|
// Wait until we have enough txs to build the block
|
|
546
|
-
const minTxs = this.
|
|
547
|
-
const { availableTxs, canStartBuilding } = await this.waitForMinTxs(opts);
|
|
1031
|
+
const { availableTxs, canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
|
|
548
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
|
+
});
|
|
549
1042
|
this.log.warn(
|
|
550
|
-
`Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.
|
|
551
|
-
{
|
|
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
|
+
},
|
|
552
1053
|
);
|
|
553
|
-
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 });
|
|
554
1055
|
this.metrics.recordBlockProposalFailed('insufficient_txs');
|
|
555
|
-
return
|
|
1056
|
+
return { failure: 'insufficient-txs' };
|
|
556
1057
|
}
|
|
557
1058
|
|
|
558
1059
|
// Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
|
|
@@ -563,24 +1064,31 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
563
1064
|
);
|
|
564
1065
|
|
|
565
1066
|
this.log.debug(
|
|
566
|
-
`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.
|
|
567
|
-
{ slot: this.
|
|
1067
|
+
`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.targetSlot} with ${availableTxs} available txs`,
|
|
1068
|
+
{ slot: this.targetSlot, blockNumber, indexWithinCheckpoint },
|
|
568
1069
|
);
|
|
569
|
-
this.setStateFn(SequencerState.CREATING_BLOCK, this.
|
|
570
|
-
|
|
571
|
-
// Calculate blob fields limit for txs (remaining capacity - this block's end overhead)
|
|
572
|
-
const blockEndOverhead = getNumBlockEndBlobFields(indexWithinCheckpoint === 0);
|
|
573
|
-
const maxBlobFieldsForTxs = remainingBlobFields - blockEndOverhead;
|
|
1070
|
+
this.setStateFn(SequencerState.CREATING_BLOCK, this.targetSlot, this.slotNow);
|
|
574
1071
|
|
|
575
|
-
|
|
1072
|
+
// Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
|
|
1073
|
+
// by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
|
|
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 = {
|
|
576
1077
|
maxTransactions: this.config.maxTxsPerBlock,
|
|
577
|
-
|
|
578
|
-
|
|
579
|
-
|
|
1078
|
+
maxBlockGas:
|
|
1079
|
+
this.config.maxL2BlockGas !== undefined || this.config.maxDABlockGas !== undefined
|
|
1080
|
+
? new Gas(this.config.maxDABlockGas ?? Infinity, this.config.maxL2BlockGas ?? Infinity)
|
|
1081
|
+
: undefined,
|
|
580
1082
|
deadline: buildDeadline,
|
|
1083
|
+
isBuildingProposal: true,
|
|
1084
|
+
minValidTxs,
|
|
1085
|
+
maxBlocksPerCheckpoint: this.timetable.maxNumberOfBlocks,
|
|
1086
|
+
perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier,
|
|
581
1087
|
};
|
|
582
1088
|
|
|
583
|
-
// 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.
|
|
584
1092
|
const buildResult = await this.buildSingleBlockWithCheckpointBuilder(
|
|
585
1093
|
checkpointBuilder,
|
|
586
1094
|
pendingTxs,
|
|
@@ -592,22 +1100,38 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
592
1100
|
// If any txs failed during execution, drop them from the mempool so we don't pick them up again
|
|
593
1101
|
await this.dropFailedTxsFromP2P(buildResult.failedTxs);
|
|
594
1102
|
|
|
595
|
-
|
|
596
|
-
|
|
597
|
-
|
|
598
|
-
|
|
599
|
-
|
|
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
|
+
});
|
|
600
1113
|
this.log.warn(
|
|
601
|
-
`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.
|
|
602
|
-
{
|
|
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
|
+
},
|
|
603
1124
|
);
|
|
604
|
-
this.eventEmitter.emit('block-build-failed', {
|
|
1125
|
+
this.eventEmitter.emit('block-build-failed', {
|
|
1126
|
+
reason: `Insufficient valid txs`,
|
|
1127
|
+
slot: this.targetSlot,
|
|
1128
|
+
});
|
|
605
1129
|
this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
|
|
606
|
-
return
|
|
1130
|
+
return { failure: 'insufficient-valid-txs' };
|
|
607
1131
|
}
|
|
608
1132
|
|
|
609
1133
|
// Block creation succeeded, emit stats and metrics
|
|
610
|
-
const {
|
|
1134
|
+
const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
|
|
611
1135
|
|
|
612
1136
|
const blockStats = {
|
|
613
1137
|
eventName: 'l2-block-built',
|
|
@@ -618,33 +1142,56 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
618
1142
|
|
|
619
1143
|
const blockHash = await block.hash();
|
|
620
1144
|
const txHashes = block.body.txEffects.map(tx => tx.txHash);
|
|
621
|
-
const manaPerSec =
|
|
1145
|
+
const manaPerSec = block.header.totalManaUsed.toNumberUnsafe() / (blockBuildDuration / 1000);
|
|
622
1146
|
|
|
623
1147
|
this.log.info(
|
|
624
|
-
`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`,
|
|
625
1149
|
{ blockHash, txHashes, manaPerSec, ...blockStats },
|
|
626
1150
|
);
|
|
627
1151
|
|
|
628
|
-
|
|
629
|
-
|
|
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);
|
|
630
1163
|
|
|
631
|
-
return { block, usedTxs
|
|
1164
|
+
return { block, usedTxs };
|
|
632
1165
|
} catch (err: any) {
|
|
633
|
-
this.eventEmitter.emit('block-build-failed', {
|
|
634
|
-
|
|
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
|
+
});
|
|
635
1182
|
this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
|
|
636
1183
|
this.metrics.recordFailedBlock();
|
|
637
1184
|
return { error: err };
|
|
638
1185
|
}
|
|
639
1186
|
}
|
|
640
1187
|
|
|
641
|
-
/** Uses the checkpoint builder to build a block, catching
|
|
1188
|
+
/** Uses the checkpoint builder to build a block, catching InsufficientValidTxsError. */
|
|
642
1189
|
private async buildSingleBlockWithCheckpointBuilder(
|
|
643
1190
|
checkpointBuilder: CheckpointBuilder,
|
|
644
1191
|
pendingTxs: AsyncIterable<Tx>,
|
|
645
1192
|
blockNumber: BlockNumber,
|
|
646
1193
|
blockTimestamp: bigint,
|
|
647
|
-
blockBuilderOptions:
|
|
1194
|
+
blockBuilderOptions: BlockBuilderOptions,
|
|
648
1195
|
) {
|
|
649
1196
|
try {
|
|
650
1197
|
const workTimer = new Timer();
|
|
@@ -652,8 +1199,12 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
652
1199
|
const blockBuildDuration = workTimer.ms();
|
|
653
1200
|
return { ...result, blockBuildDuration, status: 'success' as const };
|
|
654
1201
|
} catch (err: unknown) {
|
|
655
|
-
if (isErrorClass(err,
|
|
656
|
-
return {
|
|
1202
|
+
if (isErrorClass(err, InsufficientValidTxsError)) {
|
|
1203
|
+
return {
|
|
1204
|
+
failedTxs: err.failedTxs,
|
|
1205
|
+
processedCount: err.processedCount,
|
|
1206
|
+
status: 'insufficient-valid-txs' as const,
|
|
1207
|
+
};
|
|
657
1208
|
}
|
|
658
1209
|
throw err;
|
|
659
1210
|
}
|
|
@@ -666,7 +1217,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
666
1217
|
blockNumber: BlockNumber;
|
|
667
1218
|
indexWithinCheckpoint: IndexWithinCheckpoint;
|
|
668
1219
|
buildDeadline: Date | undefined;
|
|
669
|
-
}): Promise<{ canStartBuilding: boolean; availableTxs: number }> {
|
|
1220
|
+
}): Promise<{ canStartBuilding: boolean; availableTxs: number; minTxs: number }> {
|
|
670
1221
|
const { indexWithinCheckpoint, blockNumber, buildDeadline, forceCreate } = opts;
|
|
671
1222
|
|
|
672
1223
|
// We only allow a block with 0 txs in the first block of the checkpoint
|
|
@@ -683,20 +1234,50 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
683
1234
|
// If we're past deadline, or we have no deadline, give up
|
|
684
1235
|
const now = this.dateProvider.nowAsDate();
|
|
685
1236
|
if (startBuildingDeadline === undefined || now >= startBuildingDeadline) {
|
|
686
|
-
return { canStartBuilding: false, availableTxs
|
|
1237
|
+
return { canStartBuilding: false, availableTxs, minTxs };
|
|
687
1238
|
}
|
|
688
1239
|
|
|
689
1240
|
// Wait a bit before checking again
|
|
690
|
-
this.setStateFn(SequencerState.WAITING_FOR_TXS, this.
|
|
1241
|
+
this.setStateFn(SequencerState.WAITING_FOR_TXS, this.targetSlot, this.slotNow);
|
|
691
1242
|
this.log.verbose(
|
|
692
|
-
`Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.
|
|
693
|
-
{ 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 },
|
|
694
1245
|
);
|
|
695
1246
|
await this.waitForTxsPollingInterval();
|
|
696
1247
|
availableTxs = await this.p2pClient.getPendingTxCount();
|
|
697
1248
|
}
|
|
698
1249
|
|
|
699
|
-
return { canStartBuilding: true, availableTxs };
|
|
1250
|
+
return { canStartBuilding: true, availableTxs, minTxs };
|
|
1251
|
+
}
|
|
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
|
+
}
|
|
700
1281
|
}
|
|
701
1282
|
|
|
702
1283
|
/**
|
|
@@ -704,10 +1285,12 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
704
1285
|
* This is run after all blocks for the checkpoint have been built.
|
|
705
1286
|
*/
|
|
706
1287
|
@trackSpan('CheckpointProposalJob.waitForAttestations')
|
|
707
|
-
private async waitForAttestations(
|
|
1288
|
+
private async waitForAttestations(
|
|
1289
|
+
proposal: CheckpointProposal,
|
|
1290
|
+
): Promise<CommitteeAttestationsAndSigners | undefined> {
|
|
708
1291
|
if (this.config.fishermanMode) {
|
|
709
1292
|
this.log.debug('Skipping attestation collection in fisherman mode');
|
|
710
|
-
return CommitteeAttestationsAndSigners.empty();
|
|
1293
|
+
return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
|
|
711
1294
|
}
|
|
712
1295
|
|
|
713
1296
|
const slotNumber = proposal.slotNumber;
|
|
@@ -717,21 +1300,27 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
717
1300
|
throw new Error('No committee when collecting attestations');
|
|
718
1301
|
} else if (committee.length === 0) {
|
|
719
1302
|
this.log.verbose(`Attesting committee is empty`);
|
|
720
|
-
return CommitteeAttestationsAndSigners.empty();
|
|
1303
|
+
return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
|
|
721
1304
|
} else {
|
|
722
1305
|
this.log.debug(`Attesting committee length is ${committee.length}`, { committee });
|
|
723
1306
|
}
|
|
724
1307
|
|
|
725
|
-
const numberOfRequiredAttestations =
|
|
1308
|
+
const numberOfRequiredAttestations = computeQuorum(committee.length);
|
|
726
1309
|
|
|
727
1310
|
if (this.config.skipCollectingAttestations) {
|
|
728
1311
|
this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
|
|
729
|
-
const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
|
|
730
|
-
|
|
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
|
+
);
|
|
731
1320
|
}
|
|
732
1321
|
|
|
733
1322
|
const attestationTimeAllowed = this.config.enforceTimeTable
|
|
734
|
-
? this.timetable.
|
|
1323
|
+
? this.timetable.getCheckpointAttestationDeadline()
|
|
735
1324
|
: this.l1Constants.slotDuration;
|
|
736
1325
|
const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
|
|
737
1326
|
|
|
@@ -744,6 +1333,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
744
1333
|
proposal,
|
|
745
1334
|
numberOfRequiredAttestations,
|
|
746
1335
|
attestationDeadline,
|
|
1336
|
+
this.checkpointNumber,
|
|
747
1337
|
);
|
|
748
1338
|
|
|
749
1339
|
collectedAttestationsCount = attestations.length;
|
|
@@ -762,6 +1352,9 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
762
1352
|
|
|
763
1353
|
// Rollup contract requires that the signatures are provided in the order of the committee
|
|
764
1354
|
const sorted = orderAttestations(trimmed, committee);
|
|
1355
|
+
this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
|
|
1356
|
+
submittedCount: trimmed.length,
|
|
1357
|
+
});
|
|
765
1358
|
|
|
766
1359
|
// Manipulate the attestations if we've been configured to do so
|
|
767
1360
|
if (
|
|
@@ -773,17 +1366,56 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
773
1366
|
return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
|
|
774
1367
|
}
|
|
775
1368
|
|
|
776
|
-
return new CommitteeAttestationsAndSigners(sorted);
|
|
1369
|
+
return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
|
|
777
1370
|
} catch (err) {
|
|
778
1371
|
if (err && err instanceof AttestationTimeoutError) {
|
|
779
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);
|
|
780
1387
|
}
|
|
781
|
-
|
|
1388
|
+
return undefined;
|
|
782
1389
|
} finally {
|
|
783
1390
|
this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
|
|
784
1391
|
}
|
|
785
1392
|
}
|
|
786
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
|
+
|
|
787
1419
|
/** Breaks the attestations before publishing based on attack configs */
|
|
788
1420
|
private manipulateAttestations(
|
|
789
1421
|
slotNumber: SlotNumber,
|
|
@@ -827,7 +1459,7 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
827
1459
|
unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
|
|
828
1460
|
}
|
|
829
1461
|
}
|
|
830
|
-
return new CommitteeAttestationsAndSigners(attestations);
|
|
1462
|
+
return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
|
|
831
1463
|
}
|
|
832
1464
|
|
|
833
1465
|
if (this.config.shuffleAttestationOrdering) {
|
|
@@ -849,11 +1481,11 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
849
1481
|
[shuffled[i], shuffled[j]] = [shuffled[j], shuffled[i]];
|
|
850
1482
|
}
|
|
851
1483
|
|
|
852
|
-
const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
|
|
853
|
-
return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
|
|
1484
|
+
const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
|
|
1485
|
+
return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
|
|
854
1486
|
}
|
|
855
1487
|
|
|
856
|
-
return new CommitteeAttestationsAndSigners(attestations);
|
|
1488
|
+
return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
|
|
857
1489
|
}
|
|
858
1490
|
|
|
859
1491
|
private async dropFailedTxsFromP2P(failedTxs: FailedTx[]) {
|
|
@@ -870,9 +1502,13 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
870
1502
|
* Adds the proposed block to the archiver so it's available via P2P.
|
|
871
1503
|
* Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
|
|
872
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.
|
|
873
1509
|
*/
|
|
874
1510
|
private async syncProposedBlockToArchiver(block: L2Block): Promise<void> {
|
|
875
|
-
if (this.config.skipPushProposedBlocksToArchiver
|
|
1511
|
+
if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
|
|
876
1512
|
this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
|
|
877
1513
|
blockNumber: block.number,
|
|
878
1514
|
slot: block.header.globalVariables.slotNumber,
|
|
@@ -886,23 +1522,58 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
886
1522
|
await this.blockSink.addBlock(block);
|
|
887
1523
|
}
|
|
888
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
|
+
|
|
889
1560
|
/** Runs fee analysis and logs checkpoint outcome as fisherman */
|
|
890
1561
|
private async handleCheckpointEndAsFisherman(checkpoint: Checkpoint | undefined) {
|
|
891
1562
|
// Perform L1 fee analysis before clearing requests
|
|
892
1563
|
// The callback is invoked asynchronously after the next block is mined
|
|
893
|
-
const feeAnalysis = await this.publisher.analyzeL1Fees(this.
|
|
1564
|
+
const feeAnalysis = await this.publisher.analyzeL1Fees(this.targetSlot, analysis =>
|
|
894
1565
|
this.metrics.recordFishermanFeeAnalysis(analysis),
|
|
895
1566
|
);
|
|
896
1567
|
|
|
897
1568
|
if (checkpoint) {
|
|
898
|
-
this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.
|
|
1569
|
+
this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.targetSlot}`, {
|
|
899
1570
|
...checkpoint.toCheckpointInfo(),
|
|
900
1571
|
...checkpoint.getStats(),
|
|
901
1572
|
feeAnalysisId: feeAnalysis?.id,
|
|
902
1573
|
});
|
|
903
1574
|
} else {
|
|
904
|
-
this.log.warn(`Validation block building FAILED for slot ${this.
|
|
905
|
-
slot: this.
|
|
1575
|
+
this.log.warn(`Validation block building FAILED for slot ${this.targetSlot}`, {
|
|
1576
|
+
slot: this.targetSlot,
|
|
906
1577
|
feeAnalysisId: feeAnalysis?.id,
|
|
907
1578
|
});
|
|
908
1579
|
this.metrics.recordCheckpointProposalFailed('block_build_failed');
|
|
@@ -916,15 +1587,15 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
916
1587
|
*/
|
|
917
1588
|
private handleHASigningError(err: any, errorContext: string): boolean {
|
|
918
1589
|
if (err instanceof DutyAlreadySignedError) {
|
|
919
|
-
this.log.info(`${errorContext} for slot ${this.
|
|
920
|
-
slot: this.
|
|
1590
|
+
this.log.info(`${errorContext} for slot ${this.targetSlot} already signed by another HA node, yielding`, {
|
|
1591
|
+
slot: this.targetSlot,
|
|
921
1592
|
signedByNode: err.signedByNode,
|
|
922
1593
|
});
|
|
923
1594
|
return true;
|
|
924
1595
|
}
|
|
925
1596
|
if (err instanceof SlashingProtectionError) {
|
|
926
|
-
this.log.info(`${errorContext} for slot ${this.
|
|
927
|
-
slot: this.
|
|
1597
|
+
this.log.info(`${errorContext} for slot ${this.targetSlot} blocked by slashing protection, yielding`, {
|
|
1598
|
+
slot: this.targetSlot,
|
|
928
1599
|
existingMessageHash: err.existingMessageHash,
|
|
929
1600
|
attemptedMessageHash: err.attemptedMessageHash,
|
|
930
1601
|
});
|
|
@@ -938,16 +1609,17 @@ export class CheckpointProposalJob implements Traceable {
|
|
|
938
1609
|
protected async waitUntilTimeInSlot(targetSecondsIntoSlot: number): Promise<void> {
|
|
939
1610
|
const slotStartTimestamp = this.getSlotStartBuildTimestamp();
|
|
940
1611
|
const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
|
|
941
|
-
|
|
1612
|
+
const delayMs = targetTimestamp * 1000 - this.dateProvider.nowAsDate().getTime();
|
|
1613
|
+
await this.awaitInterruptibleSleep(delayMs);
|
|
942
1614
|
}
|
|
943
1615
|
|
|
944
|
-
/**
|
|
1616
|
+
/** Pause between mempool polls in waitForMinTxs. Extracted for test overriding. */
|
|
945
1617
|
protected async waitForTxsPollingInterval(): Promise<void> {
|
|
946
|
-
await
|
|
1618
|
+
await this.awaitInterruptibleSleep(TXS_POLLING_MS);
|
|
947
1619
|
}
|
|
948
1620
|
|
|
949
1621
|
private getSlotStartBuildTimestamp(): number {
|
|
950
|
-
return getSlotStartBuildTimestamp(this.
|
|
1622
|
+
return getSlotStartBuildTimestamp(this.slotNow, this.l1Constants);
|
|
951
1623
|
}
|
|
952
1624
|
|
|
953
1625
|
private getSecondsIntoSlot(): number {
|