@aztec/sequencer-client 0.0.1-commit.4d3c002 → 0.0.1-commit.4d9804df
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 +20 -5
- package/dest/client/sequencer-client.d.ts.map +1 -1
- package/dest/client/sequencer-client.js +26 -22
- package/dest/config.d.ts +11 -4
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +55 -26
- 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 +138 -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 +80 -0
- package/dest/global_variable_builder/global_builder.d.ts +7 -26
- package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
- package/dest/global_variable_builder/global_builder.js +6 -67
- package/dest/global_variable_builder/index.d.ts +3 -1
- package/dest/global_variable_builder/index.d.ts.map +1 -1
- package/dest/global_variable_builder/index.js +2 -0
- package/dest/publisher/config.d.ts +7 -3
- package/dest/publisher/config.d.ts.map +1 -1
- package/dest/publisher/config.js +13 -3
- package/dest/publisher/l1_to_l2_messaging.d.ts +21 -0
- package/dest/publisher/l1_to_l2_messaging.d.ts.map +1 -0
- package/dest/publisher/l1_to_l2_messaging.js +70 -0
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +50 -11
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.js +68 -1
- package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts +2 -2
- package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts.map +1 -1
- package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.js +4 -2
- package/dest/publisher/l1_tx_failed_store/index.d.ts +2 -2
- package/dest/publisher/l1_tx_failed_store/index.d.ts.map +1 -1
- package/dest/publisher/l1_tx_failed_store/index.js +1 -0
- 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 +1 -3
- package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher-factory.js +0 -1
- package/dest/publisher/sequencer-publisher.d.ts +77 -68
- package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher.js +593 -553
- package/dest/publisher/write_json.d.ts +11 -0
- package/dest/publisher/write_json.d.ts.map +1 -0
- package/dest/publisher/write_json.js +57 -0
- package/dest/sequencer/automine/automine_factory.d.ts +56 -0
- package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
- package/dest/sequencer/automine/automine_factory.js +85 -0
- package/dest/sequencer/automine/automine_sequencer.d.ts +189 -0
- package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
- package/dest/sequencer/automine/automine_sequencer.js +690 -0
- package/dest/sequencer/automine/index.d.ts +3 -0
- package/dest/sequencer/automine/index.d.ts.map +1 -0
- package/dest/sequencer/automine/index.js +2 -0
- package/dest/sequencer/checkpoint_proposal_job.d.ts +67 -36
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_proposal_job.js +686 -253
- 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/events.d.ts +61 -5
- package/dest/sequencer/events.d.ts.map +1 -1
- package/dest/sequencer/metrics.d.ts +9 -10
- package/dest/sequencer/metrics.d.ts.map +1 -1
- package/dest/sequencer/metrics.js +34 -20
- package/dest/sequencer/requests_tracker.d.ts +22 -0
- package/dest/sequencer/requests_tracker.d.ts.map +1 -0
- package/dest/sequencer/requests_tracker.js +33 -0
- package/dest/sequencer/sequencer.d.ts +145 -33
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +520 -180
- package/dest/sequencer/types.d.ts +2 -2
- package/dest/sequencer/types.d.ts.map +1 -1
- package/dest/test/index.d.ts +3 -3
- package/dest/test/index.d.ts.map +1 -1
- package/dest/test/utils.d.ts +15 -1
- package/dest/test/utils.d.ts.map +1 -1
- package/dest/test/utils.js +25 -7
- package/package.json +30 -28
- package/src/client/sequencer-client.ts +37 -27
- package/src/config.ts +64 -26
- package/src/global_variable_builder/README.md +44 -0
- package/src/global_variable_builder/fee_predictor.ts +182 -0
- package/src/global_variable_builder/fee_provider.ts +97 -0
- package/src/global_variable_builder/global_builder.ts +11 -91
- package/src/global_variable_builder/index.ts +2 -0
- package/src/publisher/config.ts +30 -7
- package/src/publisher/l1_to_l2_messaging.ts +85 -0
- package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +117 -8
- package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +4 -3
- package/src/publisher/l1_tx_failed_store/index.ts +1 -1
- package/src/publisher/sequencer-bundle-simulator.ts +254 -0
- package/src/publisher/sequencer-publisher-factory.ts +0 -3
- package/src/publisher/sequencer-publisher.ts +671 -641
- package/src/publisher/write_json.ts +78 -0
- package/src/sequencer/README.md +162 -450
- package/src/sequencer/automine/README.md +60 -0
- package/src/sequencer/automine/automine_factory.ts +152 -0
- package/src/sequencer/automine/automine_sequencer.ts +796 -0
- package/src/sequencer/automine/index.ts +6 -0
- package/src/sequencer/checkpoint_proposal_job.ts +808 -313
- package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
- package/src/sequencer/events.ts +66 -5
- package/src/sequencer/metrics.ts +43 -24
- package/src/sequencer/requests_tracker.ts +43 -0
- package/src/sequencer/sequencer.ts +586 -206
- package/src/sequencer/types.ts +1 -1
- package/src/test/index.ts +2 -2
- package/src/test/utils.ts +61 -10
- package/dest/sequencer/timetable.d.ts +0 -88
- package/dest/sequencer/timetable.d.ts.map +0 -1
- package/dest/sequencer/timetable.js +0 -222
- package/src/sequencer/timetable.ts +0 -283
|
@@ -1,20 +1,35 @@
|
|
|
1
1
|
import { getKzg } from '@aztec/blob-lib';
|
|
2
|
-
import {
|
|
3
|
-
import type { EpochCache } from '@aztec/epoch-cache';
|
|
2
|
+
import { type EpochCache, PROPOSER_PIPELINING_SLOT_OFFSET } from '@aztec/epoch-cache';
|
|
4
3
|
import { NoCommitteeError, type RollupContract } from '@aztec/ethereum/contracts';
|
|
5
4
|
import { BlockNumber, CheckpointNumber, EpochNumber, SlotNumber } from '@aztec/foundation/branded-types';
|
|
6
5
|
import { merge, omit, pick } from '@aztec/foundation/collection';
|
|
7
6
|
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
8
7
|
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
9
|
-
import { createLogger } from '@aztec/foundation/log';
|
|
8
|
+
import { type Logger, createLogger } from '@aztec/foundation/log';
|
|
10
9
|
import { RunningPromise } from '@aztec/foundation/running-promise';
|
|
11
10
|
import type { DateProvider } from '@aztec/foundation/timer';
|
|
12
11
|
import type { TypedEventEmitter } from '@aztec/foundation/types';
|
|
13
12
|
import type { P2P } from '@aztec/p2p';
|
|
14
13
|
import type { SlasherClientInterface } from '@aztec/slasher';
|
|
15
|
-
import type {
|
|
16
|
-
|
|
17
|
-
|
|
14
|
+
import type {
|
|
15
|
+
BlockData,
|
|
16
|
+
L2BlockSink,
|
|
17
|
+
L2BlockSource,
|
|
18
|
+
ProposedCheckpointSink,
|
|
19
|
+
ValidateCheckpointResult,
|
|
20
|
+
} from '@aztec/stdlib/block';
|
|
21
|
+
import {
|
|
22
|
+
type Checkpoint,
|
|
23
|
+
type ProposedCheckpointData,
|
|
24
|
+
buildCheckpointSimulationOverridesPlan,
|
|
25
|
+
} from '@aztec/stdlib/checkpoint';
|
|
26
|
+
import type { ChainConfig } from '@aztec/stdlib/config';
|
|
27
|
+
import { getEpochAtSlot } from '@aztec/stdlib/epoch-helpers';
|
|
28
|
+
import {
|
|
29
|
+
MIN_PER_BLOCK_ALLOCATION_MULTIPLIER,
|
|
30
|
+
MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER,
|
|
31
|
+
computeNetworkTxGasLimits,
|
|
32
|
+
} from '@aztec/stdlib/gas';
|
|
18
33
|
import {
|
|
19
34
|
type ResolvedSequencerConfig,
|
|
20
35
|
type SequencerConfig,
|
|
@@ -22,8 +37,9 @@ import {
|
|
|
22
37
|
type WorldStateSynchronizer,
|
|
23
38
|
} from '@aztec/stdlib/interfaces/server';
|
|
24
39
|
import type { L1ToL2MessageSource } from '@aztec/stdlib/messaging';
|
|
40
|
+
import type { CoordinationSignatureContext } from '@aztec/stdlib/p2p';
|
|
25
41
|
import { pickFromSchema } from '@aztec/stdlib/schemas';
|
|
26
|
-
import {
|
|
42
|
+
import { ProposerTimetable, buildProposerTimetable } from '@aztec/stdlib/timetable';
|
|
27
43
|
import { Attributes, type TelemetryClient, type Tracer, getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
|
|
28
44
|
import { FullNodeCheckpointsBuilder, NodeKeystoreAdapter, type ValidatorClient } from '@aztec/validator-client';
|
|
29
45
|
|
|
@@ -34,16 +50,27 @@ import type { GlobalVariableBuilder } from '../global_variable_builder/global_bu
|
|
|
34
50
|
import type { SequencerPublisherFactory } from '../publisher/sequencer-publisher-factory.js';
|
|
35
51
|
import type { InvalidateCheckpointRequest, SequencerPublisher } from '../publisher/sequencer-publisher.js';
|
|
36
52
|
import { CheckpointProposalJob } from './checkpoint_proposal_job.js';
|
|
53
|
+
import { CheckpointProposalJobMetrics } from './checkpoint_proposal_job_metrics.js';
|
|
37
54
|
import { CheckpointVoter } from './checkpoint_voter.js';
|
|
38
|
-
import { SequencerInterruptedError
|
|
55
|
+
import { SequencerInterruptedError } from './errors.js';
|
|
39
56
|
import type { SequencerEvents } from './events.js';
|
|
40
57
|
import { SequencerMetrics } from './metrics.js';
|
|
41
|
-
import {
|
|
58
|
+
import { RequestsTracker } from './requests_tracker.js';
|
|
42
59
|
import type { SequencerRollupConstants } from './types.js';
|
|
43
60
|
import { SequencerState } from './utils.js';
|
|
44
61
|
|
|
45
62
|
export { SequencerState };
|
|
46
63
|
|
|
64
|
+
/** Slot snapshot used to prepare a checkpoint proposal. */
|
|
65
|
+
type SequencerSlotContext = {
|
|
66
|
+
slot: SlotNumber;
|
|
67
|
+
targetSlot: SlotNumber;
|
|
68
|
+
epoch: EpochNumber;
|
|
69
|
+
targetEpoch: EpochNumber;
|
|
70
|
+
ts: bigint;
|
|
71
|
+
nowSeconds: bigint;
|
|
72
|
+
};
|
|
73
|
+
|
|
47
74
|
/**
|
|
48
75
|
* Sequencer client
|
|
49
76
|
* - Checks whether it is elected as proposer for the next slot
|
|
@@ -53,18 +80,27 @@ export { SequencerState };
|
|
|
53
80
|
* - Votes for proposals and slashes on L1
|
|
54
81
|
*/
|
|
55
82
|
export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<SequencerEvents>) {
|
|
56
|
-
|
|
83
|
+
protected runningPromise?: RunningPromise;
|
|
57
84
|
private state = SequencerState.STOPPED;
|
|
85
|
+
private stateSlotNumber: SlotNumber | undefined;
|
|
86
|
+
/** Wall-clock time (ms, via the date provider) at which the current state was entered. */
|
|
87
|
+
private stateEnteredAtMs: number;
|
|
58
88
|
private metrics: SequencerMetrics;
|
|
89
|
+
private checkpointProposalJobMetrics: CheckpointProposalJobMetrics;
|
|
90
|
+
private readonly stateLog: Logger;
|
|
91
|
+
|
|
92
|
+
/** The last slot for which we attempted to perform our fallback duties (votes and/or prune) with degraded block production */
|
|
93
|
+
private lastSlotForFallbackAction: SlotNumber | undefined;
|
|
59
94
|
|
|
60
|
-
/** The
|
|
61
|
-
|
|
95
|
+
/** The (checkpoint, slot) of the last invalidation request we successfully simulated, to prevent
|
|
96
|
+
* re-simulating and re-submitting the same invalidation across the many ticks within a single slot. */
|
|
97
|
+
private lastInvalidationAttempt: { slot: SlotNumber; checkpointNumber: CheckpointNumber } | undefined;
|
|
62
98
|
|
|
63
99
|
/** The last slot for which we logged "no committee" warning, to avoid spam */
|
|
64
100
|
private lastSlotForNoCommitteeWarning: SlotNumber | undefined;
|
|
65
101
|
|
|
66
102
|
/** The last slot for which we triggered a checkpoint proposal job, to prevent duplicate attempts. */
|
|
67
|
-
|
|
103
|
+
protected lastSlotForCheckpointProposalJob: SlotNumber | undefined;
|
|
68
104
|
|
|
69
105
|
/** Last successful checkpoint proposed */
|
|
70
106
|
private lastCheckpointProposed: Checkpoint | undefined;
|
|
@@ -73,13 +109,25 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
73
109
|
private lastEpochForStrategyComparison: EpochNumber | undefined;
|
|
74
110
|
|
|
75
111
|
/** The last checkpoint proposal job, tracked so we can await its pending L1 submission during shutdown. */
|
|
76
|
-
|
|
112
|
+
protected lastCheckpointProposalJob: CheckpointProposalJob | undefined;
|
|
113
|
+
|
|
114
|
+
/**
|
|
115
|
+
* In-flight fire-and-forget requests that {@link stop} interrupts and drains, and {@link pause} awaits
|
|
116
|
+
* untouched: the checkpoint proposal jobs' backgrounded L1 submissions (each job is handed this shared
|
|
117
|
+
* tracker) plus the sequencer's own fallback submissions (votes/prune when we cannot build, or
|
|
118
|
+
* escape-hatch votes). Each fallback send is gated by its wrapper publisher's interruptible target-slot
|
|
119
|
+
* sleep; the tracked interrupt wakes that sleep so the send short-circuits without publishing. A wrapper
|
|
120
|
+
* is created for a single send and is never restarted, so once interrupted its sleeper can never publish
|
|
121
|
+
* a stale-slot tx, even after the pooled publishers are restarted by a later {@link start}.
|
|
122
|
+
*/
|
|
123
|
+
protected readonly pendingRequests = new RequestsTracker();
|
|
77
124
|
|
|
78
|
-
/**
|
|
79
|
-
protected timetable!:
|
|
125
|
+
/** Proposer schedule and block sub-slot timetable for the sequencer, rebuilt on every config update. */
|
|
126
|
+
protected timetable!: ProposerTimetable;
|
|
80
127
|
|
|
81
128
|
/** Config for the sequencer */
|
|
82
129
|
protected config: ResolvedSequencerConfig = DefaultSequencerConfig;
|
|
130
|
+
private readonly signatureContext: CoordinationSignatureContext;
|
|
83
131
|
|
|
84
132
|
constructor(
|
|
85
133
|
protected publisherFactory: SequencerPublisherFactory,
|
|
@@ -88,45 +136,148 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
88
136
|
protected p2pClient: P2P,
|
|
89
137
|
protected worldState: WorldStateSynchronizer,
|
|
90
138
|
protected slasherClient: SlasherClientInterface | undefined,
|
|
91
|
-
protected l2BlockSource: L2BlockSource & L2BlockSink,
|
|
139
|
+
protected l2BlockSource: L2BlockSource & L2BlockSink & ProposedCheckpointSink,
|
|
92
140
|
protected l1ToL2MessageSource: L1ToL2MessageSource,
|
|
93
141
|
protected checkpointsBuilder: FullNodeCheckpointsBuilder,
|
|
94
142
|
protected l1Constants: SequencerRollupConstants,
|
|
95
143
|
protected dateProvider: DateProvider,
|
|
96
144
|
protected epochCache: EpochCache,
|
|
97
145
|
protected rollupContract: RollupContract,
|
|
98
|
-
config: SequencerConfig,
|
|
146
|
+
config: SequencerConfig & Pick<ChainConfig, 'l1ChainId' | 'rollupAddress'>,
|
|
99
147
|
protected telemetry: TelemetryClient = getTelemetryClient(),
|
|
100
148
|
protected log = createLogger('sequencer'),
|
|
101
149
|
) {
|
|
102
150
|
super();
|
|
151
|
+
this.stateLog = log.createChild('state');
|
|
152
|
+
this.stateEnteredAtMs = this.dateProvider.now();
|
|
103
153
|
|
|
104
154
|
// Add [FISHERMAN] prefix to logger if in fisherman mode
|
|
105
155
|
if (config.fishermanMode) {
|
|
106
156
|
this.log = log.createChild('[FISHERMAN]');
|
|
107
157
|
}
|
|
108
158
|
|
|
159
|
+
this.signatureContext = {
|
|
160
|
+
chainId: config.l1ChainId,
|
|
161
|
+
rollupAddress: config.rollupAddress,
|
|
162
|
+
};
|
|
109
163
|
this.metrics = new SequencerMetrics(telemetry, this.rollupContract, 'Sequencer');
|
|
164
|
+
this.checkpointProposalJobMetrics = new CheckpointProposalJobMetrics(telemetry);
|
|
110
165
|
this.updateConfig(config);
|
|
111
166
|
}
|
|
112
167
|
|
|
113
|
-
/**
|
|
168
|
+
/**
|
|
169
|
+
* Updates sequencer config by the defined values and rebuilds the timetable.
|
|
170
|
+
*
|
|
171
|
+
* The merged config is validated against a candidate before being committed: {@link buildTimetable} may
|
|
172
|
+
* reject the candidate (invalid timing geometry, or per-block allocation multipliers below the network
|
|
173
|
+
* minimums). On rejection we leave `this.config` and `this.timetable` untouched and rethrow, so a bad update
|
|
174
|
+
* never leaves the sequencer running with a rejected config and a stale timetable.
|
|
175
|
+
*/
|
|
114
176
|
public updateConfig(config: Partial<SequencerConfig>) {
|
|
115
177
|
const filteredConfig = pickFromSchema(config, SequencerConfigSchema);
|
|
178
|
+
const candidate = merge(this.config, filteredConfig);
|
|
179
|
+
let timetable: ProposerTimetable;
|
|
180
|
+
try {
|
|
181
|
+
timetable = this.buildTimetable(candidate);
|
|
182
|
+
} catch (err) {
|
|
183
|
+
this.log.warn(`Rejecting sequencer config update: ${(err as Error).message}`, {
|
|
184
|
+
rejectedConfig: omit(filteredConfig, 'txPublicSetupAllowListExtend'),
|
|
185
|
+
});
|
|
186
|
+
throw err;
|
|
187
|
+
}
|
|
188
|
+
this.config = candidate;
|
|
189
|
+
this.timetable = timetable;
|
|
116
190
|
this.log.info(`Updated sequencer config`, omit(filteredConfig, 'txPublicSetupAllowListExtend'));
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
);
|
|
191
|
+
}
|
|
192
|
+
|
|
193
|
+
/**
|
|
194
|
+
* Builds the proposer timetable from the given config and L1 constants via the shared
|
|
195
|
+
* {@link buildProposerTimetable} helper, so the sequencer derives the same blocks-per-checkpoint as the p2p
|
|
196
|
+
* layer and `getNodeInfo`. The fast local/e2e profile and budget clamping happen inside
|
|
197
|
+
* {@link ProposerTimetable}.
|
|
198
|
+
*
|
|
199
|
+
* Throws if the timing geometry is invalid or the per-block allocation multipliers are below the network
|
|
200
|
+
* minimums; callers must treat a throw as a rejected config and not commit it.
|
|
201
|
+
*/
|
|
202
|
+
private buildTimetable(config: ResolvedSequencerConfig): ProposerTimetable {
|
|
203
|
+
const timetable = buildProposerTimetable(config, this.l1Constants);
|
|
204
|
+
|
|
205
|
+
const maxNumberOfBlocks = timetable.getMaxBlocksPerCheckpoint();
|
|
206
|
+
this.log.info(`Sequencer timetable initialized with ${maxNumberOfBlocks} blocks per slot`, {
|
|
207
|
+
aztecSlotDuration: timetable.aztecSlotDuration,
|
|
208
|
+
ethereumSlotDuration: timetable.ethereumSlotDuration,
|
|
209
|
+
blockDuration: timetable.blockDuration,
|
|
210
|
+
minBlockDuration: timetable.minBlockDuration,
|
|
211
|
+
p2pPropagationTime: timetable.p2pPropagationTime,
|
|
212
|
+
checkpointProposalPrepareTime: timetable.checkpointProposalPrepareTime,
|
|
213
|
+
maxNumberOfBlocks,
|
|
214
|
+
});
|
|
215
|
+
|
|
216
|
+
this.assertConfigMeetsNetworkTxLimits(config, maxNumberOfBlocks);
|
|
217
|
+
|
|
218
|
+
return timetable;
|
|
219
|
+
}
|
|
220
|
+
|
|
221
|
+
/**
|
|
222
|
+
* Checks this node's configured per-block allocation against the network admission limit. A node
|
|
223
|
+
* advertises and admits txs up to the limit derived from the network-minimum multipliers (see
|
|
224
|
+
* {@link computeNetworkTxGasLimits}).
|
|
225
|
+
*
|
|
226
|
+
* Fails startup (and runtime config updates) only when the configured per-block allocation *multipliers*
|
|
227
|
+
* (`perBlockAllocationMultiplier` / `perBlockDAAllocationMultiplier`) are below the network minimums: such
|
|
228
|
+
* a node would accept txs over RPC/gossip that its builder can never pack into a block regardless of block
|
|
229
|
+
* size. Operators may configure a higher (more generous) multiplier, but not a lower one.
|
|
230
|
+
*
|
|
231
|
+
* When the multipliers meet the floor but an absolute per-block gas cap (`maxDABlockGas` / `maxL2BlockGas`)
|
|
232
|
+
* shrinks the builder's effective grant below the network limit, this is legitimate operator
|
|
233
|
+
* restrictiveness — the node simply builds smaller blocks and such txs stay in the pool for other
|
|
234
|
+
* proposers — so we only log a warning rather than failing startup. Restrictive tx-count caps
|
|
235
|
+
* (`maxTxsPerBlock` / `maxTxsPerCheckpoint`) can likewise make the builder skip admitted txs; they are
|
|
236
|
+
* intentionally not modeled here for the same reason.
|
|
237
|
+
*/
|
|
238
|
+
private assertConfigMeetsNetworkTxLimits(config: ResolvedSequencerConfig, maxBlocksPerCheckpoint: number) {
|
|
239
|
+
// Mirror CheckpointBuilder.capLimitsByCheckpointBudgets: DA falls back to the general multiplier.
|
|
240
|
+
const l2Multiplier = config.perBlockAllocationMultiplier;
|
|
241
|
+
const daMultiplier = config.perBlockDAAllocationMultiplier ?? l2Multiplier;
|
|
242
|
+
|
|
243
|
+
// The allocation is monotonic in the multiplier, so a multiplier at or above the network minimum
|
|
244
|
+
// guarantees the builder grants at least the network admission limit. Checking the multipliers directly
|
|
245
|
+
// is sufficient (and strictly more conservative than modeling the resulting gas grant).
|
|
246
|
+
if (daMultiplier < MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER) {
|
|
247
|
+
throw new Error(
|
|
248
|
+
`perBlockDAAllocationMultiplier (${daMultiplier}) is below the network minimum ` +
|
|
249
|
+
`${MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER}; the node would admit txs its own builder can never include.`,
|
|
250
|
+
);
|
|
251
|
+
}
|
|
252
|
+
if (l2Multiplier < MIN_PER_BLOCK_ALLOCATION_MULTIPLIER) {
|
|
253
|
+
throw new Error(
|
|
254
|
+
`perBlockAllocationMultiplier (${l2Multiplier}) is below the network minimum ` +
|
|
255
|
+
`${MIN_PER_BLOCK_ALLOCATION_MULTIPLIER}; the node would admit txs its own builder can never include.`,
|
|
256
|
+
);
|
|
257
|
+
}
|
|
258
|
+
|
|
259
|
+
// Absolute per-block gas caps below the network admission limit are legitimate operator restrictiveness:
|
|
260
|
+
// the node simply builds smaller blocks and such txs stay in the pool for other proposers. Warn only.
|
|
261
|
+
const networkLimit = computeNetworkTxGasLimits({
|
|
262
|
+
maxBlocksPerCheckpoint,
|
|
263
|
+
manaCheckpointBudget: this.l1Constants.rollupManaLimit,
|
|
264
|
+
});
|
|
265
|
+
if (config.maxDABlockGas !== undefined && config.maxDABlockGas < networkLimit.daGas) {
|
|
266
|
+
this.log.warn(
|
|
267
|
+
`Sequencer maxDABlockGas (${config.maxDABlockGas}) is below the network DA admission limit ` +
|
|
268
|
+
`(${networkLimit.daGas}): txs declaring more DA gas are admitted over RPC/gossip but will be skipped ` +
|
|
269
|
+
`by this proposer's own blocks and left in the pool for other proposers.`,
|
|
270
|
+
{ maxDABlockGas: config.maxDABlockGas, networkDaGas: networkLimit.daGas, maxBlocksPerCheckpoint },
|
|
271
|
+
);
|
|
272
|
+
}
|
|
273
|
+
if (config.maxL2BlockGas !== undefined && config.maxL2BlockGas < networkLimit.l2Gas) {
|
|
274
|
+
this.log.warn(
|
|
275
|
+
`Sequencer maxL2BlockGas (${config.maxL2BlockGas}) is below the network L2 admission limit ` +
|
|
276
|
+
`(${networkLimit.l2Gas}): txs declaring more L2 gas are admitted over RPC/gossip but will be skipped ` +
|
|
277
|
+
`by this proposer's own blocks and left in the pool for other proposers.`,
|
|
278
|
+
{ maxL2BlockGas: config.maxL2BlockGas, networkL2Gas: networkLimit.l2Gas, maxBlocksPerCheckpoint },
|
|
279
|
+
);
|
|
280
|
+
}
|
|
130
281
|
}
|
|
131
282
|
|
|
132
283
|
/** Initializes the sequencer (precomputes tables). Takes about 3s. */
|
|
@@ -134,8 +285,22 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
134
285
|
getKzg();
|
|
135
286
|
}
|
|
136
287
|
|
|
137
|
-
/**
|
|
288
|
+
/**
|
|
289
|
+
* Starts (or restarts) the sequencer and moves it to the IDLE state. Idempotent: a start while already
|
|
290
|
+
* running is a no-op, so it never orphans the previous poll loop. A start while STOPPING throws, since the
|
|
291
|
+
* in-flight stop would mark the fresh loop's state STOPPED and orphan it — silently doing nothing would
|
|
292
|
+
* leave the caller believing the sequencer is running when it is on its way to STOPPED. Safe to call after
|
|
293
|
+
* a previous {@link stop} has resolved; the caller must also restart the publishers (see
|
|
294
|
+
* {@link SequencerClient.start}) so L1 publishing is re-enabled.
|
|
295
|
+
*/
|
|
138
296
|
public start() {
|
|
297
|
+
if (this.state === SequencerState.STOPPING) {
|
|
298
|
+
throw new Error('Cannot start sequencer while it is stopping');
|
|
299
|
+
}
|
|
300
|
+
if (this.runningPromise?.isRunning()) {
|
|
301
|
+
this.log.warn('Attempted to start sequencer that is already running');
|
|
302
|
+
return;
|
|
303
|
+
}
|
|
139
304
|
this.runningPromise = new RunningPromise(
|
|
140
305
|
this.safeWork.bind(this),
|
|
141
306
|
this.log,
|
|
@@ -146,35 +311,68 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
146
311
|
this.log.info('Started sequencer');
|
|
147
312
|
}
|
|
148
313
|
|
|
149
|
-
/**
|
|
314
|
+
/** Triggers an immediate run of the sequencer, bypassing the polling interval. */
|
|
315
|
+
public trigger() {
|
|
316
|
+
return this.runningPromise?.trigger();
|
|
317
|
+
}
|
|
318
|
+
|
|
319
|
+
/**
|
|
320
|
+
* Stops the sequencer from building blocks and moves it to STOPPED state, interrupting the in-flight
|
|
321
|
+
* work() iteration and its pending L1 submissions for a fast shutdown, then draining before returning.
|
|
322
|
+
* Idempotent: a second call while already stopped or stopping is a no-op. Lifecycle calls are expected
|
|
323
|
+
* to be serialized by the caller. For a restartable pause that lets in-flight work finish untouched
|
|
324
|
+
* (e.g. around a test clock warp), use {@link pause} instead.
|
|
325
|
+
*/
|
|
150
326
|
public async stop(): Promise<void> {
|
|
327
|
+
if (this.state === SequencerState.STOPPED || this.state === SequencerState.STOPPING) {
|
|
328
|
+
this.log.debug(`Sequencer already ${this.state.toLowerCase()}, ignoring stop`);
|
|
329
|
+
return;
|
|
330
|
+
}
|
|
151
331
|
this.log.info(`Stopping sequencer`);
|
|
152
332
|
this.setState(SequencerState.STOPPING, undefined, { force: true });
|
|
333
|
+
this.lastCheckpointProposalJob?.interrupt();
|
|
153
334
|
await this.publisherFactory.stopAll();
|
|
335
|
+
// Stop the poll loop and await the in-flight work() iteration. work() registers its fire-and-forget
|
|
336
|
+
// requests synchronously before returning, so once the loop has stopped, pendingRequests holds every
|
|
337
|
+
// request that will ever exist. Interrupting any earlier could miss a request registered by the last
|
|
338
|
+
// in-flight iteration.
|
|
154
339
|
await this.runningPromise?.stop();
|
|
155
|
-
|
|
340
|
+
this.pendingRequests.interruptRequests();
|
|
341
|
+
await this.pendingRequests.awaitRequests();
|
|
156
342
|
this.setState(SequencerState.STOPPED, undefined, { force: true });
|
|
157
343
|
this.log.info('Stopped sequencer');
|
|
158
344
|
}
|
|
159
345
|
|
|
346
|
+
/**
|
|
347
|
+
* Gracefully pauses block production so the sequencer can later be resumed with {@link start}: halts the
|
|
348
|
+
* poll loop and waits for the in-flight work() iteration and every pending L1 submission / fallback send
|
|
349
|
+
* to finish, without interrupting them. No interrupt lands mid-build, so no spurious checkpoint-error is
|
|
350
|
+
* emitted and no enqueued checkpoint is dropped. Deliberately does not stop inner services (validator/HA
|
|
351
|
+
* signer, publishers), so the slashing-protection store stays open and the sequencer stays restartable.
|
|
352
|
+
* Used by tests that pause sequencers around an L1 clock warp. Idempotent.
|
|
353
|
+
*/
|
|
354
|
+
public async pause(): Promise<void> {
|
|
355
|
+
if (!this.runningPromise?.isRunning()) {
|
|
356
|
+
this.log.debug('Sequencer not running, ignoring pause');
|
|
357
|
+
return;
|
|
358
|
+
}
|
|
359
|
+
this.log.info('Pausing sequencer');
|
|
360
|
+
// Halt the poll loop and let the in-flight iteration finish naturally — no interrupt, and no STOPPING
|
|
361
|
+
// state, since entering STOPPING would make the iteration's own setState calls throw and fail it.
|
|
362
|
+
// work() registers its fire-and-forget requests synchronously before returning, so once the loop has
|
|
363
|
+
// stopped pendingRequests holds every request that will ever exist; awaiting it lets them all finish.
|
|
364
|
+
await this.runningPromise.stop();
|
|
365
|
+
await this.pendingRequests.awaitRequests();
|
|
366
|
+
this.log.info('Paused sequencer');
|
|
367
|
+
}
|
|
368
|
+
|
|
160
369
|
/** Main sequencer loop with a try/catch */
|
|
161
370
|
protected async safeWork() {
|
|
162
371
|
try {
|
|
163
372
|
await this.work();
|
|
164
373
|
} catch (err) {
|
|
165
374
|
this.emit('checkpoint-error', { error: err as Error });
|
|
166
|
-
|
|
167
|
-
// Log as warn only if we had to abort halfway through the block proposal
|
|
168
|
-
const logLvl = [SequencerState.INITIALIZING_CHECKPOINT, SequencerState.PROPOSER_CHECK].includes(
|
|
169
|
-
err.proposedState,
|
|
170
|
-
)
|
|
171
|
-
? ('debug' as const)
|
|
172
|
-
: ('warn' as const);
|
|
173
|
-
this.log[logLvl](err.message, { now: this.dateProvider.nowInSeconds() });
|
|
174
|
-
} else {
|
|
175
|
-
// Re-throw other errors
|
|
176
|
-
throw err;
|
|
177
|
-
}
|
|
375
|
+
throw err;
|
|
178
376
|
} finally {
|
|
179
377
|
this.setState(SequencerState.IDLE, undefined);
|
|
180
378
|
}
|
|
@@ -196,8 +394,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
196
394
|
@trackSpan('Sequencer.work')
|
|
197
395
|
protected async work() {
|
|
198
396
|
this.setState(SequencerState.SYNCHRONIZING, undefined);
|
|
199
|
-
const { slot, ts, nowSeconds
|
|
200
|
-
const { slot: targetSlot, epoch: targetEpoch } = this.epochCache.getTargetEpochAndSlotInNextL1Slot();
|
|
397
|
+
const { slot, targetSlot, epoch, targetEpoch, ts, nowSeconds } = this.getSlotContextInNextL1Slot();
|
|
201
398
|
|
|
202
399
|
// Check if we are synced and it's our slot, grab a publisher, check previous block invalidation, etc
|
|
203
400
|
const checkpointProposalJob = await this.prepareCheckpointProposal(
|
|
@@ -235,6 +432,13 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
235
432
|
return checkpoint;
|
|
236
433
|
}
|
|
237
434
|
|
|
435
|
+
/** Returns slot and target slot from a single clock snapshot. */
|
|
436
|
+
protected getSlotContextInNextL1Slot(): SequencerSlotContext {
|
|
437
|
+
const { slot, ts, nowSeconds, epoch } = this.epochCache.getEpochAndSlotInNextL1Slot();
|
|
438
|
+
const targetSlot = SlotNumber(slot + PROPOSER_PIPELINING_SLOT_OFFSET);
|
|
439
|
+
return { slot, targetSlot, epoch, targetEpoch: getEpochAtSlot(targetSlot, this.l1Constants), ts, nowSeconds };
|
|
440
|
+
}
|
|
441
|
+
|
|
238
442
|
/**
|
|
239
443
|
* Prepares the checkpoint proposal by performing all necessary checks and setup.
|
|
240
444
|
* This is the initial step in the main loop.
|
|
@@ -249,14 +453,8 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
249
453
|
ts: bigint,
|
|
250
454
|
nowSeconds: bigint,
|
|
251
455
|
): Promise<CheckpointProposalJob | undefined> {
|
|
252
|
-
// Check we have not already processed this target slot (cheapest check)
|
|
253
|
-
|
|
254
|
-
// running against actual time (eg when we use sandbox-style automining)
|
|
255
|
-
if (
|
|
256
|
-
this.lastSlotForCheckpointProposalJob &&
|
|
257
|
-
this.lastSlotForCheckpointProposalJob >= targetSlot &&
|
|
258
|
-
this.config.enforceTimeTable
|
|
259
|
-
) {
|
|
456
|
+
// Check we have not already processed this target slot (cheapest check).
|
|
457
|
+
if (this.lastSlotForCheckpointProposalJob && this.lastSlotForCheckpointProposalJob >= targetSlot) {
|
|
260
458
|
this.log.trace(`Target slot ${targetSlot} has already been processed`);
|
|
261
459
|
return undefined;
|
|
262
460
|
}
|
|
@@ -269,23 +467,29 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
269
467
|
return undefined;
|
|
270
468
|
}
|
|
271
469
|
|
|
272
|
-
//
|
|
273
|
-
|
|
274
|
-
if (
|
|
275
|
-
|
|
470
|
+
// Test-only: skip proposing for explicitly paused slots. Attestation paths run in the validator
|
|
471
|
+
// client and are not gated by this hook, so paused proposers still attest to others' proposals.
|
|
472
|
+
if (this.config.pauseProposingForSlots?.some(s => s === targetSlot)) {
|
|
473
|
+
this.log.warn(`Skipping proposal for paused slot ${targetSlot} (test-only pauseProposingForSlots hook)`, {
|
|
474
|
+
targetSlot,
|
|
475
|
+
});
|
|
276
476
|
return undefined;
|
|
277
477
|
}
|
|
278
478
|
|
|
479
|
+
// Cheap proposer check first: most nodes are not the proposer for most slots, so gate the
|
|
480
|
+
// expensive multi-subsystem checkSync (and the rest of the build path) behind it. Computed once
|
|
481
|
+
// here and reused for the escape-hatch voting path below. No setState/timing gate on this path:
|
|
482
|
+
// the build-start deadline gate runs only on the proposer build path after a successful checkSync.
|
|
483
|
+
const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
|
|
484
|
+
|
|
279
485
|
// If escape hatch is open for the target epoch, do not start checkpoint proposal work and do not attempt invalidations.
|
|
280
486
|
// Still perform governance/slashing voting (as proposer) once per slot.
|
|
281
487
|
// When pipelining, we check the target epoch (slot+1's epoch) since that's the epoch we're building for.
|
|
282
488
|
const isEscapeHatchOpen = await this.epochCache.isEscapeHatchOpen(targetEpoch);
|
|
283
489
|
|
|
284
490
|
if (isEscapeHatchOpen) {
|
|
285
|
-
this.setState(SequencerState.PROPOSER_CHECK, slot);
|
|
286
|
-
const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
|
|
287
491
|
if (canPropose) {
|
|
288
|
-
await this.tryVoteWhenEscapeHatchOpen({ slot, proposer });
|
|
492
|
+
await this.tryVoteWhenEscapeHatchOpen({ slot, targetSlot, proposer });
|
|
289
493
|
} else {
|
|
290
494
|
this.log.trace(`Escape hatch open but we are not proposer, skipping vote-only actions`, {
|
|
291
495
|
slot,
|
|
@@ -296,19 +500,53 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
296
500
|
return undefined;
|
|
297
501
|
}
|
|
298
502
|
|
|
299
|
-
//
|
|
300
|
-
|
|
503
|
+
// If we are not the proposer, check whether we should invalidate an invalid pending chain (a
|
|
504
|
+
// liveness backstop) and bail before any sync work or build-timing gate. This reads only the
|
|
505
|
+
// archiver's pending-chain validation status, which is authoritative on its own, instead of
|
|
506
|
+
// running the full sync check. Wrapped in try/catch because this leaner path skips the broader
|
|
507
|
+
// proposed-checkpoint/tip coherence screen that checkSync applied, so transient archiver
|
|
508
|
+
// incoherence surfaces as a quiet skip rather than a work-loop error.
|
|
509
|
+
if (!canPropose) {
|
|
510
|
+
try {
|
|
511
|
+
const pendingChainValidationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
|
|
512
|
+
await this.considerInvalidatingCheckpoint(pendingChainValidationStatus, slot);
|
|
513
|
+
} catch (err) {
|
|
514
|
+
this.log.warn(`Failed to consider invalidating checkpoint`, { err, slot, targetSlot });
|
|
515
|
+
}
|
|
516
|
+
return undefined;
|
|
517
|
+
}
|
|
301
518
|
|
|
302
|
-
//
|
|
303
|
-
//
|
|
304
|
-
|
|
305
|
-
|
|
306
|
-
|
|
307
|
-
|
|
308
|
-
|
|
519
|
+
// Explicit build-loop entry gate: if we are past the latest useful block-building start for the
|
|
520
|
+
// target slot, abandon building for this slot. The proposer prioritizes the ideal L1-publish path
|
|
521
|
+
// and does not plan around the late consensus-handoff path. This is the proposer build path's
|
|
522
|
+
// timing gate; it runs only after we know we are the synced proposer, so non-proposer invalidation
|
|
523
|
+
// and escape-hatch voting (which returned above) are never gated by build timing. Vote-only paths
|
|
524
|
+
// still run when block building is abandoned.
|
|
525
|
+
const startDeadline = this.timetable.getBuildStartDeadline(targetSlot);
|
|
526
|
+
const nowForStartGate = this.dateProvider.now() / 1000;
|
|
527
|
+
if (nowForStartGate > startDeadline) {
|
|
528
|
+
this.log.debug(`Past start deadline for slot ${targetSlot}, abandoning block building`, {
|
|
529
|
+
targetSlot,
|
|
530
|
+
nowForStartGate,
|
|
531
|
+
startDeadline,
|
|
532
|
+
});
|
|
533
|
+
// Mark the slot as attempted so a deadline abort is not retried within the same slot. Vote-only actions
|
|
534
|
+
// still need to run because sync can succeed even when it is too late to start building a checkpoint.
|
|
535
|
+
await this.tryVoteAndPruneWhenCannotBuild({ slot, targetSlot });
|
|
536
|
+
this.lastSlotForCheckpointProposalJob = targetSlot;
|
|
309
537
|
return undefined;
|
|
310
538
|
}
|
|
311
539
|
|
|
540
|
+
// We are the proposer, the escape hatch is closed, and we have time before the build start deadline.
|
|
541
|
+
// Now run the full sync check before building.
|
|
542
|
+
const syncedTo = await this.checkSync({ ts, slot });
|
|
543
|
+
if (!syncedTo) {
|
|
544
|
+
return undefined;
|
|
545
|
+
}
|
|
546
|
+
|
|
547
|
+
// Next checkpoint follows from the last synced one
|
|
548
|
+
const checkpointNumber = CheckpointNumber(syncedTo.checkpointNumber + 1);
|
|
549
|
+
|
|
312
550
|
const logCtx = {
|
|
313
551
|
nowSeconds,
|
|
314
552
|
syncedToL2Slot: syncedTo.syncedL2Slot,
|
|
@@ -319,13 +557,16 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
319
557
|
isPendingChainValid: pick(syncedTo.pendingChainValidationStatus, 'valid', 'reason', 'invalidIndex'),
|
|
320
558
|
};
|
|
321
559
|
|
|
322
|
-
//
|
|
323
|
-
this.setState(SequencerState.PROPOSER_CHECK,
|
|
324
|
-
const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
|
|
560
|
+
// We are the synced proposer within the build window; enter the proposer-check state and build.
|
|
561
|
+
this.setState(SequencerState.PROPOSER_CHECK, targetSlot);
|
|
325
562
|
|
|
326
|
-
//
|
|
327
|
-
|
|
328
|
-
|
|
563
|
+
// Guard: don't exceed 1-deep pipeline. Without a proposed checkpoint, we can only build
|
|
564
|
+
// confirmed + 1. With a proposed checkpoint, we can build confirmed + 2.
|
|
565
|
+
const confirmedCkpt = syncedTo.checkpointedCheckpointNumber;
|
|
566
|
+
if (checkpointNumber > confirmedCkpt + 2) {
|
|
567
|
+
this.log.verbose(
|
|
568
|
+
`Skipping slot ${targetSlot}: checkpoint ${checkpointNumber} exceeds max pipeline depth (confirmed=${confirmedCkpt})`,
|
|
569
|
+
);
|
|
329
570
|
return undefined;
|
|
330
571
|
}
|
|
331
572
|
|
|
@@ -347,53 +588,96 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
347
588
|
const { attestorAddress, publisher } = await this.publisherFactory.create(proposerForPublisher);
|
|
348
589
|
this.log.verbose(`Created publisher at address ${publisher.getSenderAddress()} for attestor ${attestorAddress}`);
|
|
349
590
|
|
|
350
|
-
//
|
|
351
|
-
|
|
352
|
-
|
|
353
|
-
|
|
354
|
-
|
|
355
|
-
|
|
356
|
-
|
|
357
|
-
let invalidateCheckpoint = await publisher.simulateInvalidateCheckpoint(syncedTo.pendingChainValidationStatus);
|
|
591
|
+
// Prepare invalidation request if the pending chain is invalid (returns undefined if no need).
|
|
592
|
+
// Only simulate invalidation when there's no proposed parent, since we assume the proposed parent
|
|
593
|
+
// will invalidate the currently invalid checkpoint on L1.
|
|
594
|
+
const invalidateCheckpoint =
|
|
595
|
+
syncedTo.hasProposedCheckpoint || syncedTo.pendingChainValidationStatus.valid
|
|
596
|
+
? undefined
|
|
597
|
+
: await publisher.simulateInvalidateCheckpoint(syncedTo.pendingChainValidationStatus);
|
|
358
598
|
|
|
359
599
|
// Determine the correct archive and L1 state overrides for the canProposeAt check.
|
|
360
600
|
// The L1 contract reads archives[proposedCheckpointNumber] and compares it with the provided archive.
|
|
361
601
|
// When invalidating or pipelining, the local archive may differ from L1's, so we adjust accordingly.
|
|
362
602
|
let archiveForCheck = syncedTo.archive;
|
|
363
|
-
const l1Overrides: {
|
|
364
|
-
forcePendingCheckpointNumber?: CheckpointNumber;
|
|
365
|
-
forceArchive?: { checkpointNumber: CheckpointNumber; archive: Fr };
|
|
366
|
-
} = {};
|
|
367
|
-
|
|
368
|
-
if (this.epochCache.isProposerPipeliningEnabled() && syncedTo.hasProposedCheckpoint) {
|
|
369
|
-
// Parent checkpoint hasn't landed on L1 yet. Override both the proposed checkpoint number
|
|
370
|
-
// and the archive at that checkpoint so L1 simulation sees the correct chain tip.
|
|
371
|
-
const parentCheckpointNumber = CheckpointNumber(checkpointNumber - 1);
|
|
372
|
-
l1Overrides.forcePendingCheckpointNumber = parentCheckpointNumber;
|
|
373
|
-
l1Overrides.forceArchive = { checkpointNumber: parentCheckpointNumber, archive: syncedTo.archive };
|
|
374
|
-
this.metrics.recordPipelineDepth(1);
|
|
375
603
|
|
|
604
|
+
if (syncedTo.hasProposedCheckpoint) {
|
|
605
|
+
this.metrics.recordPipelineDepth(syncedTo.checkpointNumber - syncedTo.checkpointedCheckpointNumber);
|
|
376
606
|
this.log.verbose(
|
|
377
|
-
`Building on top of proposed checkpoint (pending=${syncedTo.proposedCheckpointData?.checkpointNumber})`,
|
|
607
|
+
`Building on top of proposed checkpoint (pending=${syncedTo.proposedCheckpointData?.checkpointNumber}) for target slot ${targetSlot}`,
|
|
608
|
+
{ targetSlot, parentCheckpointNumber: CheckpointNumber(checkpointNumber - 1) },
|
|
378
609
|
);
|
|
379
|
-
//
|
|
380
|
-
|
|
610
|
+
// Match what L1 will see at archives[pending] once the proposed parent lands: the parent's
|
|
611
|
+
// own archive root from the gossiped proposal. `syncedTo.archive` is the world-state-local
|
|
612
|
+
// view and can transiently diverge from the proposed parent (e.g. before the proposed
|
|
613
|
+
// parent's blocks have been applied locally); diverging here would cause the canProposeAt
|
|
614
|
+
// override to set archives[pending] to one value while we present another for comparison.
|
|
615
|
+
archiveForCheck = syncedTo.proposedCheckpointData!.archive.root;
|
|
381
616
|
} else if (invalidateCheckpoint) {
|
|
382
617
|
// After invalidation, L1 will roll back to checkpoint N-1. The archive at N-1 already
|
|
383
618
|
// exists on L1, so we just pass the matching archive (the lastArchive of the invalid checkpoint).
|
|
384
619
|
archiveForCheck = invalidateCheckpoint.lastArchive;
|
|
385
|
-
l1Overrides.forcePendingCheckpointNumber = invalidateCheckpoint.forcePendingCheckpointNumber;
|
|
386
620
|
this.metrics.recordPipelineDepth(0);
|
|
387
621
|
} else {
|
|
388
622
|
this.metrics.recordPipelineDepth(0);
|
|
389
623
|
}
|
|
390
624
|
|
|
391
|
-
|
|
625
|
+
// Build the simulation plan: pending/proven override from pipelining or invalidation (or the
|
|
626
|
+
// current snapshot when neither applies, to short-circuit any pending prune in simulation),
|
|
627
|
+
// plus the parent checkpoint cell and fee header when pipelining.
|
|
628
|
+
const simulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
|
|
629
|
+
checkpointNumber,
|
|
630
|
+
proposedCheckpointData: syncedTo.hasProposedCheckpoint ? syncedTo.proposedCheckpointData : undefined,
|
|
631
|
+
invalidateToPendingCheckpointNumber: invalidateCheckpoint?.forcePendingCheckpointNumber,
|
|
632
|
+
checkpointedCheckpointNumber: syncedTo.checkpointedCheckpointNumber,
|
|
633
|
+
rollup: this.rollupContract,
|
|
634
|
+
signatureContext: this.signatureContext,
|
|
635
|
+
log: this.log,
|
|
636
|
+
});
|
|
637
|
+
|
|
638
|
+
// The plan always pins both pending/proven (to short-circuit `canPruneAtTime` in simulation),
|
|
639
|
+
// so `provenOverride` always reflects the assumed proven checkpoint we are pinning the
|
|
640
|
+
// simulation to. We additionally warn when the pin is load-bearing — i.e. when a prune would
|
|
641
|
+
// actually fire at the target slot without it — so observers can spot "we are building
|
|
642
|
+
// optimistically across a pruning boundary" in the logs.
|
|
643
|
+
const provenOverride = simulationOverridesPlan?.chainTipsOverride?.proven;
|
|
644
|
+
if (provenOverride !== undefined && (await this.l2BlockSource.isPruneDueAtSlot(targetSlot))) {
|
|
645
|
+
this.log.warn(
|
|
646
|
+
`Assuming proof for epoch ending at checkpoint ${provenOverride} lands by target slot ${targetSlot}`,
|
|
647
|
+
{ checkpointNumber, slot, targetSlot, provenOverride },
|
|
648
|
+
);
|
|
649
|
+
}
|
|
650
|
+
|
|
651
|
+
this.emit('preparing-checkpoint', {
|
|
652
|
+
targetSlot,
|
|
653
|
+
checkpointNumber,
|
|
654
|
+
hadProposedParent: syncedTo.hasProposedCheckpoint,
|
|
655
|
+
provenOverride,
|
|
656
|
+
simulatedPending: simulationOverridesPlan?.chainTipsOverride?.pending,
|
|
657
|
+
});
|
|
658
|
+
|
|
659
|
+
const canProposeCheck = await publisher.canProposeAt(
|
|
660
|
+
archiveForCheck,
|
|
661
|
+
proposer ?? EthAddress.ZERO,
|
|
662
|
+
simulationOverridesPlan,
|
|
663
|
+
);
|
|
664
|
+
|
|
665
|
+
const proposeContext = {
|
|
666
|
+
hasProposedCheckpoint: syncedTo.hasProposedCheckpoint,
|
|
667
|
+
proposedCheckpointNumber: syncedTo.proposedCheckpointData?.checkpointNumber,
|
|
668
|
+
checkpointedCheckpointNumber: syncedTo.checkpointedCheckpointNumber,
|
|
669
|
+
isInvalidating: !!invalidateCheckpoint,
|
|
670
|
+
invalidatingCheckpointNumber: invalidateCheckpoint?.checkpointNumber,
|
|
671
|
+
archiveForCheck: archiveForCheck.toString(),
|
|
672
|
+
overridePendingCheckpointNumber: simulationOverridesPlan?.chainTipsOverride?.pending,
|
|
673
|
+
overrideArchive: simulationOverridesPlan?.pendingCheckpointState?.archive,
|
|
674
|
+
overrideFeeHeader: simulationOverridesPlan?.pendingCheckpointState?.feeHeader,
|
|
675
|
+
};
|
|
392
676
|
|
|
393
677
|
if (canProposeCheck === undefined) {
|
|
394
678
|
this.log.warn(
|
|
395
679
|
`Cannot propose checkpoint ${checkpointNumber} at slot ${slot} due to failed rollup contract check`,
|
|
396
|
-
logCtx,
|
|
680
|
+
{ ...logCtx, ...proposeContext },
|
|
397
681
|
);
|
|
398
682
|
this.emit('proposer-rollup-check-failed', { reason: 'Rollup contract check failed', slot });
|
|
399
683
|
this.metrics.recordCheckpointPrecheckFailed('rollup_contract_check_failed');
|
|
@@ -403,7 +687,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
403
687
|
if (canProposeCheck.slot !== targetSlot) {
|
|
404
688
|
this.log.warn(
|
|
405
689
|
`Cannot propose block due to slot mismatch with rollup contract (this can be caused by a clock out of sync). Expected slot ${targetSlot} but got ${canProposeCheck.slot}.`,
|
|
406
|
-
{ ...logCtx, rollup: canProposeCheck, expectedSlot: targetSlot },
|
|
690
|
+
{ ...logCtx, ...proposeContext, rollup: canProposeCheck, expectedSlot: targetSlot },
|
|
407
691
|
);
|
|
408
692
|
this.emit('proposer-rollup-check-failed', { reason: 'Slot mismatch', slot });
|
|
409
693
|
this.metrics.recordCheckpointPrecheckFailed('slot_mismatch');
|
|
@@ -413,7 +697,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
413
697
|
if (canProposeCheck.checkpointNumber !== checkpointNumber) {
|
|
414
698
|
this.log.warn(
|
|
415
699
|
`Cannot propose due to block mismatch with rollup contract (this can be caused by a pending archiver sync). Expected checkpoint ${checkpointNumber} but got ${canProposeCheck.checkpointNumber}.`,
|
|
416
|
-
{ ...logCtx, rollup: canProposeCheck, expectedSlot: slot },
|
|
700
|
+
{ ...logCtx, ...proposeContext, rollup: canProposeCheck, expectedSlot: slot },
|
|
417
701
|
);
|
|
418
702
|
this.emit('proposer-rollup-check-failed', { reason: 'Block mismatch', slot });
|
|
419
703
|
this.metrics.recordCheckpointPrecheckFailed('block_number_mismatch');
|
|
@@ -427,18 +711,18 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
427
711
|
`Preparing checkpoint proposal ${checkpointNumber} for target slot ${targetSlot} during wall-clock slot ${slot}`,
|
|
428
712
|
{
|
|
429
713
|
...logCtx,
|
|
714
|
+
...proposeContext,
|
|
430
715
|
proposer,
|
|
431
|
-
pipeliningEnabled: this.epochCache.isProposerPipeliningEnabled(),
|
|
432
716
|
},
|
|
433
717
|
);
|
|
434
718
|
|
|
435
719
|
// Create and return the checkpoint proposal job
|
|
436
720
|
return this.createCheckpointProposalJob(
|
|
437
|
-
slot,
|
|
438
721
|
targetSlot,
|
|
439
722
|
targetEpoch,
|
|
440
723
|
checkpointNumber,
|
|
441
724
|
syncedTo.blockNumber,
|
|
725
|
+
syncedTo.checkpointedCheckpointNumber,
|
|
442
726
|
proposer,
|
|
443
727
|
publisher,
|
|
444
728
|
attestorAddress,
|
|
@@ -448,11 +732,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
448
732
|
}
|
|
449
733
|
|
|
450
734
|
protected createCheckpointProposalJob(
|
|
451
|
-
slot: SlotNumber,
|
|
452
735
|
targetSlot: SlotNumber,
|
|
453
736
|
targetEpoch: EpochNumber,
|
|
454
737
|
checkpointNumber: CheckpointNumber,
|
|
455
738
|
syncedToBlockNumber: BlockNumber,
|
|
739
|
+
checkpointedCheckpointNumber: CheckpointNumber,
|
|
456
740
|
proposer: EthAddress | undefined,
|
|
457
741
|
publisher: SequencerPublisher,
|
|
458
742
|
attestorAddress: EthAddress,
|
|
@@ -460,11 +744,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
460
744
|
proposedCheckpointData?: ProposedCheckpointData,
|
|
461
745
|
): CheckpointProposalJob {
|
|
462
746
|
return new CheckpointProposalJob(
|
|
463
|
-
slot,
|
|
464
747
|
targetSlot,
|
|
465
748
|
targetEpoch,
|
|
466
749
|
checkpointNumber,
|
|
467
750
|
syncedToBlockNumber,
|
|
751
|
+
checkpointedCheckpointNumber,
|
|
468
752
|
proposer,
|
|
469
753
|
publisher,
|
|
470
754
|
attestorAddress,
|
|
@@ -478,13 +762,16 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
478
762
|
this.checkpointsBuilder,
|
|
479
763
|
this.l2BlockSource,
|
|
480
764
|
this.l1Constants,
|
|
765
|
+
this.signatureContext,
|
|
481
766
|
this.config,
|
|
482
767
|
this.timetable,
|
|
483
768
|
this.slasherClient,
|
|
484
769
|
this.epochCache,
|
|
485
770
|
this.dateProvider,
|
|
486
771
|
this.metrics,
|
|
772
|
+
this.checkpointProposalJobMetrics.createRecorder(),
|
|
487
773
|
this,
|
|
774
|
+
this.pendingRequests,
|
|
488
775
|
this.setState.bind(this),
|
|
489
776
|
this.tracer,
|
|
490
777
|
this.log.getBindings(),
|
|
@@ -500,9 +787,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
500
787
|
}
|
|
501
788
|
|
|
502
789
|
/**
|
|
503
|
-
* Internal helper for setting the sequencer state
|
|
790
|
+
* Internal helper for setting the sequencer state. Pure: sets the state, emits `state-changed`, and
|
|
791
|
+
* records metrics. Timing deadlines are queried explicitly at the relevant call sites, not gated here.
|
|
504
792
|
* @param proposedState - The new state to transition to.
|
|
505
|
-
* @param slotNumber - The
|
|
793
|
+
* @param slotNumber - The target slot being proposed for, emitted as `targetSlot` on the event payload
|
|
794
|
+
* and used to anchor `secondsIntoBuildFrame`. Undefined for lifecycle states with no associated slot.
|
|
506
795
|
* @param force - Whether to force the transition even if the sequencer is stopped.
|
|
507
796
|
*/
|
|
508
797
|
protected setState(
|
|
@@ -518,25 +807,39 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
518
807
|
this.log.warn(`Cannot set sequencer from ${this.state} to ${proposedState} as it is stopped.`);
|
|
519
808
|
return;
|
|
520
809
|
}
|
|
521
|
-
|
|
522
|
-
|
|
523
|
-
|
|
524
|
-
|
|
525
|
-
|
|
810
|
+
const secondsIntoBuildFrame = slotNumber !== undefined ? this.getSecondsIntoBuildFrame(slotNumber) : undefined;
|
|
811
|
+
|
|
812
|
+
const oldState = this.state;
|
|
813
|
+
const oldStateSlotNumber = this.stateSlotNumber;
|
|
814
|
+
const stateChanged = proposedState !== oldState;
|
|
815
|
+
// Wall-clock time spent in the previous state: the delta between consecutive state-changing setState
|
|
816
|
+
// calls, read from the date provider so it tracks simulated time under a test/manual clock.
|
|
817
|
+
const transitionAtMs = this.dateProvider.now();
|
|
818
|
+
const stateDurationMs = transitionAtMs - this.stateEnteredAtMs;
|
|
526
819
|
|
|
527
820
|
const boringStates = [SequencerState.IDLE, SequencerState.SYNCHRONIZING];
|
|
528
821
|
const logLevel =
|
|
529
|
-
boringStates.includes(proposedState) && boringStates.includes(
|
|
530
|
-
|
|
531
|
-
|
|
532
|
-
|
|
822
|
+
boringStates.includes(proposedState) && boringStates.includes(oldState) ? ('trace' as const) : ('debug' as const);
|
|
823
|
+
this.stateLog[logLevel](`Transitioning from ${oldState} to ${proposedState}`, {
|
|
824
|
+
oldState,
|
|
825
|
+
newState: proposedState,
|
|
826
|
+
slotNumber,
|
|
827
|
+
stateSlotNumber: oldStateSlotNumber,
|
|
828
|
+
secondsIntoBuildFrame,
|
|
829
|
+
...(stateChanged && { stateDurationMs: Math.ceil(stateDurationMs) }),
|
|
830
|
+
});
|
|
533
831
|
|
|
534
832
|
this.emit('state-changed', {
|
|
535
|
-
oldState
|
|
833
|
+
oldState,
|
|
536
834
|
newState: proposedState,
|
|
537
|
-
|
|
538
|
-
|
|
835
|
+
secondsIntoBuildFrame,
|
|
836
|
+
targetSlot: slotNumber,
|
|
539
837
|
});
|
|
838
|
+
if (stateChanged) {
|
|
839
|
+
this.metrics.recordStateDuration(stateDurationMs, oldState);
|
|
840
|
+
this.stateEnteredAtMs = transitionAtMs;
|
|
841
|
+
this.stateSlotNumber = slotNumber;
|
|
842
|
+
}
|
|
540
843
|
this.state = proposedState;
|
|
541
844
|
}
|
|
542
845
|
|
|
@@ -563,63 +866,85 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
563
866
|
number: syncSummary.latestBlockNumber,
|
|
564
867
|
hash: syncSummary.latestBlockHash,
|
|
565
868
|
})),
|
|
566
|
-
this.l2BlockSource
|
|
567
|
-
.getL2Tips()
|
|
568
|
-
.then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed, proposedCheckpoint: t.proposedCheckpoint })),
|
|
869
|
+
this.l2BlockSource.getL2Tips().then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed })),
|
|
569
870
|
this.p2pClient.getStatus().then(p2p => p2p.syncedToL2Block),
|
|
570
|
-
this.l1ToL2MessageSource.getL2Tips().then(t => t.proposed),
|
|
871
|
+
this.l1ToL2MessageSource.getL2Tips().then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed })),
|
|
571
872
|
this.l2BlockSource.getPendingChainValidationStatus(),
|
|
572
|
-
this.l2BlockSource.
|
|
873
|
+
this.l2BlockSource.getProposedCheckpointData(),
|
|
573
874
|
] as const);
|
|
574
875
|
|
|
575
|
-
const [worldState, l2Tips, p2p,
|
|
876
|
+
const [worldState, l2Tips, p2p, l1ToL2MessageSourceTips, pendingChainValidationStatus, proposedCheckpointData] =
|
|
576
877
|
syncedBlocks;
|
|
577
878
|
|
|
578
|
-
// Handle zero as a special case, since the block hash won't match across services if we're changing the prefilled data for the genesis block,
|
|
579
|
-
// as the world state can compute the new genesis block hash, but other components use the hardcoded constant.
|
|
580
|
-
// TODO(palla/mbps): Fix the above. All components should be able to handle dynamic genesis block hashes.
|
|
581
879
|
const result =
|
|
582
|
-
|
|
583
|
-
|
|
584
|
-
|
|
585
|
-
|
|
586
|
-
|
|
587
|
-
p2p.hash === l2Tips.proposed.hash &&
|
|
588
|
-
l1ToL2MessageSource.hash === l2Tips.proposed.hash);
|
|
880
|
+
worldState.hash === l2Tips.proposed.hash &&
|
|
881
|
+
p2p.hash === l2Tips.proposed.hash &&
|
|
882
|
+
l1ToL2MessageSourceTips.proposed.hash === l2Tips.proposed.hash &&
|
|
883
|
+
l1ToL2MessageSourceTips.checkpointed.block.hash === l2Tips.checkpointed.block.hash &&
|
|
884
|
+
l1ToL2MessageSourceTips.checkpointed.checkpoint.hash === l2Tips.checkpointed.checkpoint.hash;
|
|
589
885
|
|
|
590
886
|
if (!result) {
|
|
591
887
|
this.log.debug(`Sequencer sync check failed`, {
|
|
592
888
|
worldState,
|
|
593
889
|
l2BlockSource: l2Tips.proposed,
|
|
594
890
|
p2p,
|
|
595
|
-
|
|
891
|
+
l1ToL2MessageSourceTips,
|
|
596
892
|
});
|
|
597
893
|
return undefined;
|
|
598
894
|
}
|
|
599
895
|
|
|
600
|
-
// Special case for genesis state
|
|
601
896
|
const blockNumber = worldState.number;
|
|
602
|
-
|
|
603
|
-
|
|
604
|
-
|
|
605
|
-
|
|
606
|
-
|
|
607
|
-
blockNumber: BlockNumber.ZERO,
|
|
608
|
-
archive,
|
|
609
|
-
hasProposedCheckpoint: false,
|
|
897
|
+
const blockData = await this.l2BlockSource.getBlockData({ number: blockNumber });
|
|
898
|
+
if (!blockData) {
|
|
899
|
+
this.log.warn(`Sequencer sync check failed: failed to get L2 block data ${blockNumber} from the archiver`, {
|
|
900
|
+
blockNumber,
|
|
901
|
+
l2Tips,
|
|
610
902
|
syncedL2Slot,
|
|
611
|
-
|
|
612
|
-
};
|
|
903
|
+
...args,
|
|
904
|
+
});
|
|
905
|
+
return undefined;
|
|
613
906
|
}
|
|
614
907
|
|
|
615
|
-
|
|
616
|
-
|
|
617
|
-
|
|
618
|
-
|
|
908
|
+
// Refuse to build a checkpoint on top of a proposed block whose enclosing checkpoint was never
|
|
909
|
+
// proposed. Under pipelining we may have received and reexecuted such a block locally — advancing
|
|
910
|
+
// our world-state tip past the checkpointed tip — while the proposing node never published the
|
|
911
|
+
// matching proposed checkpoint (e.g. it crashed before assembling it). Building on this orphan block
|
|
912
|
+
// would fork the chain off a tip no other node can follow. The archiver prunes these orphan blocks
|
|
913
|
+
// once their build slot ends; this guard is the correctness barrier during the grace window before.
|
|
914
|
+
// `getProposedCheckpointData()` returns the latest proposed checkpoint payload, which is always
|
|
915
|
+
// the leading one (a proposed entry is only stored beyond the confirmed frontier and is deleted
|
|
916
|
+
// on confirmation). It carries no tip, so there is no tip-vs-payload split read to reconcile.
|
|
917
|
+
if (
|
|
918
|
+
blockData.checkpointNumber > l2Tips.checkpointed.checkpoint.number &&
|
|
919
|
+
proposedCheckpointData?.checkpointNumber !== blockData.checkpointNumber
|
|
920
|
+
) {
|
|
921
|
+
const logCtx = {
|
|
922
|
+
blockCheckpointNumber: blockData.checkpointNumber,
|
|
923
|
+
checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
|
|
924
|
+
proposedCheckpointTipNumber: proposedCheckpointData?.checkpointNumber,
|
|
925
|
+
blockNumber: blockData.header.getBlockNumber(),
|
|
926
|
+
blockSlot: blockData.header.getSlot(),
|
|
927
|
+
syncedL2Slot,
|
|
928
|
+
...args,
|
|
929
|
+
};
|
|
930
|
+
|
|
931
|
+
this.log.debug(`Waiting for proposed checkpoint to catch up with reexecuted block`, logCtx);
|
|
619
932
|
return undefined;
|
|
620
933
|
}
|
|
621
934
|
|
|
622
|
-
const hasProposedCheckpoint =
|
|
935
|
+
const hasProposedCheckpoint = proposedCheckpointData !== undefined;
|
|
936
|
+
|
|
937
|
+
// Check that the proposed checkpoint is indeed the parent of the checkpoint we'll be building
|
|
938
|
+
// The checkpoint number to build is derived as blockData.checkpointNumber + 1
|
|
939
|
+
if (proposedCheckpointData && proposedCheckpointData.checkpointNumber !== blockData.checkpointNumber) {
|
|
940
|
+
this.log.warn(`Sequencer sync check failed: proposed checkpoint number mismatch`, {
|
|
941
|
+
proposedCheckpointNumber: proposedCheckpointData.checkpointNumber,
|
|
942
|
+
blockCheckpointNumber: blockData.checkpointNumber,
|
|
943
|
+
syncedL2Slot,
|
|
944
|
+
...args,
|
|
945
|
+
});
|
|
946
|
+
return undefined;
|
|
947
|
+
}
|
|
623
948
|
|
|
624
949
|
return {
|
|
625
950
|
blockData,
|
|
@@ -684,36 +1009,29 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
684
1009
|
}
|
|
685
1010
|
|
|
686
1011
|
/**
|
|
687
|
-
* Tries to vote on slashing actions and governance
|
|
688
|
-
* This allows the sequencer to participate in governance/slashing votes even when it
|
|
1012
|
+
* Tries to vote on slashing actions and governance and to prune when we cannot build and are past the
|
|
1013
|
+
* block-building window. This allows the sequencer to participate in governance/slashing votes even when it
|
|
1014
|
+
* cannot build blocks, and to prune the pending chain so it can recover from bad data that is blocking sync.
|
|
689
1015
|
*/
|
|
690
|
-
@trackSpan('
|
|
691
|
-
protected async
|
|
1016
|
+
@trackSpan('Sequencer.tryVoteAndPruneWhenCannotBuild', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
|
|
1017
|
+
protected async tryVoteAndPruneWhenCannotBuild(args: { slot: SlotNumber; targetSlot: SlotNumber }): Promise<void> {
|
|
692
1018
|
const { slot, targetSlot } = args;
|
|
693
1019
|
|
|
694
1020
|
// Prevent duplicate attempts in the same slot
|
|
695
|
-
if (this.
|
|
696
|
-
this.log.trace(`Already attempted
|
|
1021
|
+
if (this.lastSlotForFallbackAction === slot) {
|
|
1022
|
+
this.log.trace(`Already attempted fallback actions in slot ${slot} (skipping)`);
|
|
697
1023
|
return;
|
|
698
1024
|
}
|
|
699
1025
|
|
|
700
|
-
//
|
|
701
|
-
|
|
702
|
-
|
|
703
|
-
|
|
704
|
-
|
|
705
|
-
// We use INITIALIZING_PROPOSAL time limit because if we're past that, we can't build a block anyway
|
|
706
|
-
if (maxAllowedTime === undefined || secondsIntoSlot <= maxAllowedTime) {
|
|
707
|
-
this.log.trace(`Not attempting to vote since there is still time for block building`, {
|
|
708
|
-
secondsIntoSlot,
|
|
709
|
-
maxAllowedTime,
|
|
710
|
-
});
|
|
711
|
-
return;
|
|
712
|
-
}
|
|
1026
|
+
// Vote-only actions do not give up the slot: if sync recovers, a later work-loop iteration can still build.
|
|
1027
|
+
// Under proposer pipelining the work loop reasons about the next L1 slot, so waiting for the target slot's
|
|
1028
|
+
// build-start deadline can miss the whole fallback window when the target advances with the clock.
|
|
1029
|
+
const nowSeconds = this.dateProvider.now() / 1000;
|
|
1030
|
+
const startDeadline = this.timetable.getBuildStartDeadline(targetSlot);
|
|
713
1031
|
|
|
714
|
-
this.log.trace(`
|
|
715
|
-
|
|
716
|
-
|
|
1032
|
+
this.log.trace(`Cannot build for slot ${slot}, checking for voting opportunities`, {
|
|
1033
|
+
nowSeconds,
|
|
1034
|
+
startDeadline,
|
|
717
1035
|
});
|
|
718
1036
|
|
|
719
1037
|
// Check if we're a proposer or proposal is open
|
|
@@ -724,7 +1042,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
724
1042
|
}
|
|
725
1043
|
|
|
726
1044
|
// Mark this slot as attempted
|
|
727
|
-
this.
|
|
1045
|
+
this.lastSlotForFallbackAction = slot;
|
|
728
1046
|
|
|
729
1047
|
// Get a publisher for voting
|
|
730
1048
|
const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
|
|
@@ -749,13 +1067,41 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
749
1067
|
const votesPromises = voter.enqueueVotes();
|
|
750
1068
|
const votes = await Promise.all(votesPromises);
|
|
751
1069
|
|
|
752
|
-
if (
|
|
753
|
-
|
|
1070
|
+
// Even if we cannot build, try to prune so a stuck pending chain (e.g. bad data blocking sync) can
|
|
1071
|
+
// recover. prune() is permissionless, so it rides the same fallback multicall as the votes.
|
|
1072
|
+
const pruneEnqueued = await this.tryEnqueuePruneIfPrunable(targetSlot, publisher);
|
|
1073
|
+
|
|
1074
|
+
// Bail if nothing to do
|
|
1075
|
+
if (votes.every(p => !p) && !pruneEnqueued) {
|
|
1076
|
+
this.log.debug(`Nothing to enqueue for slot ${slot} (no votes, not prunable)`);
|
|
754
1077
|
return;
|
|
755
1078
|
}
|
|
756
1079
|
|
|
757
|
-
|
|
758
|
-
|
|
1080
|
+
const [governanceVoteEnqueued, slashingVoteEnqueued] = votes;
|
|
1081
|
+
this.log.info(`Submitting fallback requests in slot ${slot} despite sync failure`, {
|
|
1082
|
+
slot,
|
|
1083
|
+
pruneEnqueued,
|
|
1084
|
+
governanceVoteEnqueued: !!governanceVoteEnqueued,
|
|
1085
|
+
slashingVoteEnqueued: !!slashingVoteEnqueued,
|
|
1086
|
+
});
|
|
1087
|
+
|
|
1088
|
+
// Votes are EIP-712-signed for `targetSlot` (the pipelined slot in which the multicall is
|
|
1089
|
+
// expected to mine). Delay submission to the start of `targetSlot` so the tx mines in the
|
|
1090
|
+
// slot the votes were signed for. We fire-and-forget so we don't block the sequencer's
|
|
1091
|
+
// work loop while waiting for the target slot to start, but track it so stop() can drain it.
|
|
1092
|
+
const send = publisher.sendRequestsAt(targetSlot).catch(err => {
|
|
1093
|
+
this.log.error(`Failed to publish fallback requests despite sync failure for slot ${slot}`, err, { slot });
|
|
1094
|
+
});
|
|
1095
|
+
this.pendingRequests.trackRequest(send, () => publisher.interrupt());
|
|
1096
|
+
}
|
|
1097
|
+
|
|
1098
|
+
private async tryEnqueuePruneIfPrunable(targetSlot: SlotNumber, publisher: SequencerPublisher): Promise<boolean> {
|
|
1099
|
+
try {
|
|
1100
|
+
return await publisher.enqueuePruneIfPrunable(targetSlot);
|
|
1101
|
+
} catch (err) {
|
|
1102
|
+
this.log.error(`Failed to enqueue rollup prune for slot ${targetSlot}`, err, { targetSlot });
|
|
1103
|
+
return false;
|
|
1104
|
+
}
|
|
759
1105
|
}
|
|
760
1106
|
|
|
761
1107
|
/**
|
|
@@ -765,25 +1111,34 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
765
1111
|
@trackSpan('Sequencer.tryVoteWhenEscapeHatchOpen', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
|
|
766
1112
|
protected async tryVoteWhenEscapeHatchOpen(args: {
|
|
767
1113
|
slot: SlotNumber;
|
|
1114
|
+
targetSlot: SlotNumber;
|
|
768
1115
|
proposer: EthAddress | undefined;
|
|
769
1116
|
}): Promise<void> {
|
|
770
|
-
const { slot, proposer } = args;
|
|
1117
|
+
const { slot, targetSlot, proposer } = args;
|
|
771
1118
|
|
|
772
1119
|
// Prevent duplicate attempts in the same slot
|
|
773
|
-
if (this.
|
|
1120
|
+
if (this.lastSlotForFallbackAction === slot) {
|
|
774
1121
|
this.log.trace(`Already attempted to vote in slot ${slot} (escape hatch open, skipping)`);
|
|
775
1122
|
return;
|
|
776
1123
|
}
|
|
777
1124
|
|
|
778
1125
|
// Mark this slot as attempted
|
|
779
|
-
this.
|
|
1126
|
+
this.lastSlotForFallbackAction = slot;
|
|
780
1127
|
|
|
781
1128
|
const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
|
|
782
1129
|
|
|
783
|
-
this.log.debug(`Escape hatch open for slot ${slot}, attempting vote-only actions`, {
|
|
1130
|
+
this.log.debug(`Escape hatch open for slot ${slot}, attempting vote-only actions`, {
|
|
1131
|
+
slot,
|
|
1132
|
+
targetSlot,
|
|
1133
|
+
attestorAddress,
|
|
1134
|
+
});
|
|
784
1135
|
|
|
1136
|
+
// Under proposer pipelining, the multicall is expected to mine in `targetSlot` (slot + 1).
|
|
1137
|
+
// Governance and slashing votes are EIP-712-signed against the slot they will mine in, and the
|
|
1138
|
+
// L1 contract checks `msg.sender == getCurrentProposer()` using the mining slot. So we must
|
|
1139
|
+
// sign for `targetSlot` and delay submission to the start of `targetSlot`.
|
|
785
1140
|
const voter = new CheckpointVoter(
|
|
786
|
-
|
|
1141
|
+
targetSlot,
|
|
787
1142
|
publisher,
|
|
788
1143
|
attestorAddress,
|
|
789
1144
|
this.validatorClient,
|
|
@@ -802,8 +1157,17 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
802
1157
|
return;
|
|
803
1158
|
}
|
|
804
1159
|
|
|
805
|
-
this.log.info(`Voting in slot ${slot} (escape hatch open)`, { slot });
|
|
806
|
-
|
|
1160
|
+
this.log.info(`Voting in slot ${slot} (escape hatch open)`, { slot, targetSlot });
|
|
1161
|
+
// Votes are EIP-712-signed for `targetSlot`. Delay submission to the start of `targetSlot` so
|
|
1162
|
+
// the multicall mines in the slot the votes were signed for; otherwise the L1 contract reads
|
|
1163
|
+
// `signaler = getCurrentProposer()` against the wrong slot and signature verification fails
|
|
1164
|
+
// silently inside Multicall3. Fire-and-forget so we don't block the sequencer's work loop while
|
|
1165
|
+
// waiting for the target slot to start, mirroring tryVoteAndPruneWhenCannotBuild, but tracked so
|
|
1166
|
+
// stop() can drain it.
|
|
1167
|
+
const send = publisher.sendRequestsAt(targetSlot).catch(err => {
|
|
1168
|
+
this.log.error(`Failed to publish escape-hatch votes for slot ${slot}`, err, { slot, targetSlot });
|
|
1169
|
+
});
|
|
1170
|
+
this.pendingRequests.trackRequest(send, () => publisher.interrupt());
|
|
807
1171
|
}
|
|
808
1172
|
|
|
809
1173
|
/**
|
|
@@ -811,17 +1175,34 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
811
1175
|
* has been there without being invalidated and whether the sequencer is in the committee or not. We always
|
|
812
1176
|
* have the proposer try to invalidate, but if they fail, the sequencers in the committee are expected to try,
|
|
813
1177
|
* and if they fail, any sequencer will try as well.
|
|
1178
|
+
* @param pendingChainValidationStatus - The archiver's pending-chain validation status, authoritative on its own.
|
|
1179
|
+
* @param currentSlot - The wall-clock slot, used for committee lookup, the per-(checkpoint, slot) dedup guard, and logging.
|
|
814
1180
|
*/
|
|
815
1181
|
protected async considerInvalidatingCheckpoint(
|
|
816
|
-
|
|
1182
|
+
pendingChainValidationStatus: ValidateCheckpointResult,
|
|
817
1183
|
currentSlot: SlotNumber,
|
|
818
1184
|
): Promise<void> {
|
|
819
|
-
const { pendingChainValidationStatus, syncedL2Slot } = syncedTo;
|
|
820
1185
|
if (pendingChainValidationStatus.valid) {
|
|
821
1186
|
return;
|
|
822
1187
|
}
|
|
823
1188
|
|
|
824
1189
|
const invalidCheckpointNumber = pendingChainValidationStatus.checkpoint.checkpointNumber;
|
|
1190
|
+
|
|
1191
|
+
// Avoid re-running the committee lookup, simulation, and submission on every tick within a slot.
|
|
1192
|
+
// The guard is keyed by (checkpoint, slot) — so a different invalid checkpoint surfacing later in
|
|
1193
|
+
// the same slot is not suppressed — and is set only after a request is successfully simulated below,
|
|
1194
|
+
// so a transient simulation failure (or thresholds not yet met) still retries on the next tick.
|
|
1195
|
+
if (
|
|
1196
|
+
this.lastInvalidationAttempt?.slot === currentSlot &&
|
|
1197
|
+
this.lastInvalidationAttempt.checkpointNumber === invalidCheckpointNumber
|
|
1198
|
+
) {
|
|
1199
|
+
this.log.trace(`Already attempted to invalidate checkpoint ${invalidCheckpointNumber} in slot ${currentSlot}`, {
|
|
1200
|
+
currentSlot,
|
|
1201
|
+
invalidCheckpointNumber,
|
|
1202
|
+
});
|
|
1203
|
+
return;
|
|
1204
|
+
}
|
|
1205
|
+
|
|
825
1206
|
const invalidCheckpointTimestamp = pendingChainValidationStatus.checkpoint.timestamp;
|
|
826
1207
|
const timeSinceChainInvalid = this.dateProvider.nowInSeconds() - Number(invalidCheckpointTimestamp);
|
|
827
1208
|
const ourValidatorAddresses = this.validatorClient.getValidatorAddresses();
|
|
@@ -831,7 +1212,6 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
831
1212
|
|
|
832
1213
|
const logData = {
|
|
833
1214
|
invalidL1Timestamp: invalidCheckpointTimestamp,
|
|
834
|
-
syncedL2Slot,
|
|
835
1215
|
invalidCheckpoint: pendingChainValidationStatus.checkpoint,
|
|
836
1216
|
secondsBeforeInvalidatingBlockAsCommitteeMember,
|
|
837
1217
|
secondsBeforeInvalidatingBlockAsNonCommitteeMember,
|
|
@@ -884,6 +1264,10 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
884
1264
|
return;
|
|
885
1265
|
}
|
|
886
1266
|
|
|
1267
|
+
// We produced a valid invalidation request; record it so further ticks within this slot skip the
|
|
1268
|
+
// committee lookup, simulation, and submission above for this same invalid checkpoint.
|
|
1269
|
+
this.lastInvalidationAttempt = { slot: currentSlot, checkpointNumber: invalidCheckpointNumber };
|
|
1270
|
+
|
|
887
1271
|
this.log.info(
|
|
888
1272
|
invalidateAsCommitteeMember
|
|
889
1273
|
? `Invalidating checkpoint ${invalidCheckpointNumber} as committee member`
|
|
@@ -930,13 +1314,13 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
930
1314
|
});
|
|
931
1315
|
}
|
|
932
1316
|
|
|
933
|
-
|
|
934
|
-
|
|
935
|
-
|
|
936
|
-
|
|
937
|
-
private
|
|
938
|
-
const
|
|
939
|
-
return Number((this.dateProvider.now() / 1000 -
|
|
1317
|
+
/**
|
|
1318
|
+
* Wall-clock seconds elapsed since the build-frame start of the given target slot
|
|
1319
|
+
* (`now − getBuildFrameStart(targetSlot)`). May be negative if called before the build frame opens.
|
|
1320
|
+
*/
|
|
1321
|
+
private getSecondsIntoBuildFrame(targetSlot: SlotNumber): number {
|
|
1322
|
+
const buildFrameStart = this.timetable.getBuildFrameStart(targetSlot);
|
|
1323
|
+
return Number((this.dateProvider.now() / 1000 - buildFrameStart).toFixed(3));
|
|
940
1324
|
}
|
|
941
1325
|
|
|
942
1326
|
public get aztecSlotDuration() {
|
|
@@ -967,10 +1351,6 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
|
|
|
967
1351
|
public getConfig() {
|
|
968
1352
|
return this.config;
|
|
969
1353
|
}
|
|
970
|
-
|
|
971
|
-
private get l1PublishingTime(): number {
|
|
972
|
-
return this.config.l1PublishingTime ?? this.l1Constants.ethereumSlotDuration;
|
|
973
|
-
}
|
|
974
1354
|
}
|
|
975
1355
|
|
|
976
1356
|
type SequencerSyncCheckResult = {
|