@aztec/sequencer-client 5.0.0-private.20260318 → 5.0.0-rc.1
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 -15
- package/dest/client/sequencer-client.d.ts.map +1 -1
- package/dest/client/sequencer-client.js +35 -96
- package/dest/config.d.ts +8 -2
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +48 -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 +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 +67 -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 -51
- 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/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 +317 -532
- 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 +184 -0
- package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
- package/dest/sequencer/automine/automine_sequencer.js +677 -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/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 +77 -17
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_proposal_job.js +768 -209
- 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/errors.d.ts +1 -8
- package/dest/sequencer/errors.d.ts.map +1 -1
- package/dest/sequencer/errors.js +0 -9
- package/dest/sequencer/events.d.ts +62 -4
- package/dest/sequencer/events.d.ts.map +1 -1
- 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 +115 -30
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +506 -174
- 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/mock_checkpoint_builder.d.ts +4 -8
- package/dest/test/mock_checkpoint_builder.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 +23 -6
- package/package.json +28 -27
- package/src/client/sequencer-client.ts +53 -123
- package/src/config.ts +57 -23
- 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 +80 -0
- package/src/global_variable_builder/global_builder.ts +25 -63
- package/src/global_variable_builder/index.ts +3 -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 +376 -577
- package/src/sequencer/automine/README.md +60 -0
- package/src/sequencer/automine/automine_factory.ts +152 -0
- package/src/sequencer/automine/automine_sequencer.ts +783 -0
- package/src/sequencer/automine/index.ts +6 -0
- package/src/sequencer/chain_state_overrides.ts +169 -0
- package/src/sequencer/checkpoint_proposal_job.ts +917 -241
- package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
- package/src/sequencer/checkpoint_voter.ts +1 -12
- package/src/sequencer/errors.ts +0 -15
- package/src/sequencer/events.ts +65 -4
- package/src/sequencer/metrics.ts +57 -24
- package/src/sequencer/sequencer.ts +604 -195
- package/src/sequencer/types.ts +1 -1
- package/src/test/index.ts +2 -2
- package/src/test/mock_checkpoint_builder.ts +3 -3
- package/src/test/utils.ts +59 -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/README.md +0 -531
- package/src/sequencer/timetable.ts +0 -283
|
@@ -372,29 +372,30 @@ function _apply_decs_2203_r(targetClass, memberDecs, classDecs, parentClass) {
|
|
|
372
372
|
}
|
|
373
373
|
var _dec, _dec1, _dec2, _dec3, _initProto;
|
|
374
374
|
import { getKzg } from '@aztec/blob-lib';
|
|
375
|
-
import {
|
|
375
|
+
import { PROPOSER_PIPELINING_SLOT_OFFSET } from '@aztec/epoch-cache';
|
|
376
376
|
import { NoCommitteeError } from '@aztec/ethereum/contracts';
|
|
377
|
-
import {
|
|
377
|
+
import { CheckpointNumber, SlotNumber } from '@aztec/foundation/branded-types';
|
|
378
378
|
import { merge, omit, pick } from '@aztec/foundation/collection';
|
|
379
|
-
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
380
379
|
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
381
380
|
import { createLogger } from '@aztec/foundation/log';
|
|
382
381
|
import { RunningPromise } from '@aztec/foundation/running-promise';
|
|
383
|
-
import {
|
|
382
|
+
import { getEpochAtSlot } from '@aztec/stdlib/epoch-helpers';
|
|
383
|
+
import { MIN_PER_BLOCK_ALLOCATION_MULTIPLIER, MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER, computeNetworkTxGasLimits } from '@aztec/stdlib/gas';
|
|
384
384
|
import { SequencerConfigSchema } from '@aztec/stdlib/interfaces/server';
|
|
385
385
|
import { pickFromSchema } from '@aztec/stdlib/schemas';
|
|
386
|
-
import {
|
|
386
|
+
import { buildProposerTimetable } from '@aztec/stdlib/timetable';
|
|
387
387
|
import { Attributes, getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
|
|
388
388
|
import EventEmitter from 'node:events';
|
|
389
389
|
import { DefaultSequencerConfig } from '../config.js';
|
|
390
|
+
import { buildCheckpointSimulationOverridesPlan } from './chain_state_overrides.js';
|
|
390
391
|
import { CheckpointProposalJob } from './checkpoint_proposal_job.js';
|
|
392
|
+
import { CheckpointProposalJobMetrics } from './checkpoint_proposal_job_metrics.js';
|
|
391
393
|
import { CheckpointVoter } from './checkpoint_voter.js';
|
|
392
|
-
import { SequencerInterruptedError
|
|
394
|
+
import { SequencerInterruptedError } from './errors.js';
|
|
393
395
|
import { SequencerMetrics } from './metrics.js';
|
|
394
|
-
import { SequencerTimetable } from './timetable.js';
|
|
395
396
|
import { SequencerState } from './utils.js';
|
|
396
397
|
export { SequencerState };
|
|
397
|
-
_dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpointProposal'), _dec2 = trackSpan('
|
|
398
|
+
_dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpointProposal'), _dec2 = trackSpan('Sequencer.tryVoteWhenCannotBuild', ({ slot })=>({
|
|
398
399
|
[Attributes.SLOT_NUMBER]: slot
|
|
399
400
|
})), _dec3 = trackSpan('Sequencer.tryVoteWhenEscapeHatchOpen', ({ slot })=>({
|
|
400
401
|
[Attributes.SLOT_NUMBER]: slot
|
|
@@ -437,7 +438,7 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
437
438
|
[
|
|
438
439
|
_dec2,
|
|
439
440
|
2,
|
|
440
|
-
"
|
|
441
|
+
"tryVoteWhenCannotBuild"
|
|
441
442
|
],
|
|
442
443
|
[
|
|
443
444
|
_dec3,
|
|
@@ -448,35 +449,133 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
448
449
|
}
|
|
449
450
|
runningPromise;
|
|
450
451
|
state;
|
|
452
|
+
stateSlotNumber;
|
|
453
|
+
/** Wall-clock time (ms, via the date provider) at which the current state was entered. */ stateEnteredAtMs;
|
|
451
454
|
metrics;
|
|
455
|
+
checkpointProposalJobMetrics;
|
|
456
|
+
stateLog;
|
|
452
457
|
/** The last slot for which we attempted to perform our voting duties with degraded block production */ lastSlotForFallbackVote;
|
|
458
|
+
/** The (checkpoint, slot) of the last invalidation request we successfully simulated, to prevent
|
|
459
|
+
* re-simulating and re-submitting the same invalidation across the many ticks within a single slot. */ lastInvalidationAttempt;
|
|
453
460
|
/** The last slot for which we logged "no committee" warning, to avoid spam */ lastSlotForNoCommitteeWarning;
|
|
454
461
|
/** The last slot for which we triggered a checkpoint proposal job, to prevent duplicate attempts. */ lastSlotForCheckpointProposalJob;
|
|
455
462
|
/** Last successful checkpoint proposed */ lastCheckpointProposed;
|
|
456
463
|
/** The last epoch for which we logged strategy comparison in fisherman mode. */ lastEpochForStrategyComparison;
|
|
457
|
-
/** The
|
|
464
|
+
/** The last checkpoint proposal job, tracked so we can await its pending L1 submission during shutdown. */ lastCheckpointProposalJob;
|
|
465
|
+
/** Proposer schedule and block sub-slot timetable for the sequencer, rebuilt on every config update. */ timetable;
|
|
458
466
|
/** Config for the sequencer */ config;
|
|
467
|
+
signatureContext;
|
|
459
468
|
constructor(publisherFactory, validatorClient, globalsBuilder, p2pClient, worldState, slasherClient, l2BlockSource, l1ToL2MessageSource, checkpointsBuilder, l1Constants, dateProvider, epochCache, rollupContract, config, telemetry = getTelemetryClient(), log = createLogger('sequencer')){
|
|
460
469
|
super(), this.publisherFactory = publisherFactory, this.validatorClient = validatorClient, this.globalsBuilder = globalsBuilder, this.p2pClient = p2pClient, this.worldState = worldState, this.slasherClient = slasherClient, this.l2BlockSource = l2BlockSource, this.l1ToL2MessageSource = l1ToL2MessageSource, this.checkpointsBuilder = checkpointsBuilder, this.l1Constants = l1Constants, this.dateProvider = dateProvider, this.epochCache = epochCache, this.rollupContract = rollupContract, this.telemetry = telemetry, this.log = log, this.state = (_initProto(this), SequencerState.STOPPED), this.config = DefaultSequencerConfig;
|
|
470
|
+
this.stateLog = log.createChild('state');
|
|
471
|
+
this.stateEnteredAtMs = this.dateProvider.now();
|
|
461
472
|
// Add [FISHERMAN] prefix to logger if in fisherman mode
|
|
462
473
|
if (config.fishermanMode) {
|
|
463
474
|
this.log = log.createChild('[FISHERMAN]');
|
|
464
475
|
}
|
|
476
|
+
this.signatureContext = {
|
|
477
|
+
chainId: config.l1ChainId,
|
|
478
|
+
rollupAddress: config.rollupAddress
|
|
479
|
+
};
|
|
465
480
|
this.metrics = new SequencerMetrics(telemetry, this.rollupContract, 'Sequencer');
|
|
481
|
+
this.checkpointProposalJobMetrics = new CheckpointProposalJobMetrics(telemetry);
|
|
466
482
|
this.updateConfig(config);
|
|
467
483
|
}
|
|
468
|
-
/**
|
|
484
|
+
/**
|
|
485
|
+
* Updates sequencer config by the defined values and rebuilds the timetable.
|
|
486
|
+
*
|
|
487
|
+
* The merged config is validated against a candidate before being committed: {@link buildTimetable} may
|
|
488
|
+
* reject the candidate (invalid timing geometry, or per-block allocation multipliers below the network
|
|
489
|
+
* minimums). On rejection we leave `this.config` and `this.timetable` untouched and rethrow, so a bad update
|
|
490
|
+
* never leaves the sequencer running with a rejected config and a stale timetable.
|
|
491
|
+
*/ updateConfig(config) {
|
|
469
492
|
const filteredConfig = pickFromSchema(config, SequencerConfigSchema);
|
|
493
|
+
const candidate = merge(this.config, filteredConfig);
|
|
494
|
+
let timetable;
|
|
495
|
+
try {
|
|
496
|
+
timetable = this.buildTimetable(candidate);
|
|
497
|
+
} catch (err) {
|
|
498
|
+
this.log.warn(`Rejecting sequencer config update: ${err.message}`, {
|
|
499
|
+
rejectedConfig: omit(filteredConfig, 'txPublicSetupAllowListExtend')
|
|
500
|
+
});
|
|
501
|
+
throw err;
|
|
502
|
+
}
|
|
503
|
+
this.config = candidate;
|
|
504
|
+
this.timetable = timetable;
|
|
470
505
|
this.log.info(`Updated sequencer config`, omit(filteredConfig, 'txPublicSetupAllowListExtend'));
|
|
471
|
-
|
|
472
|
-
|
|
473
|
-
|
|
474
|
-
|
|
475
|
-
|
|
476
|
-
|
|
477
|
-
|
|
478
|
-
|
|
479
|
-
|
|
506
|
+
}
|
|
507
|
+
/**
|
|
508
|
+
* Builds the proposer timetable from the given config and L1 constants via the shared
|
|
509
|
+
* {@link buildProposerTimetable} helper, so the sequencer derives the same blocks-per-checkpoint as the p2p
|
|
510
|
+
* layer and `getNodeInfo`. The fast local/e2e profile and budget clamping happen inside
|
|
511
|
+
* {@link ProposerTimetable}.
|
|
512
|
+
*
|
|
513
|
+
* Throws if the timing geometry is invalid or the per-block allocation multipliers are below the network
|
|
514
|
+
* minimums; callers must treat a throw as a rejected config and not commit it.
|
|
515
|
+
*/ buildTimetable(config) {
|
|
516
|
+
const timetable = buildProposerTimetable(config, this.l1Constants);
|
|
517
|
+
const maxNumberOfBlocks = timetable.getMaxBlocksPerCheckpoint();
|
|
518
|
+
this.log.info(`Sequencer timetable initialized with ${maxNumberOfBlocks} blocks per slot`, {
|
|
519
|
+
aztecSlotDuration: timetable.aztecSlotDuration,
|
|
520
|
+
ethereumSlotDuration: timetable.ethereumSlotDuration,
|
|
521
|
+
blockDuration: timetable.blockDuration,
|
|
522
|
+
minBlockDuration: timetable.minBlockDuration,
|
|
523
|
+
p2pPropagationTime: timetable.p2pPropagationTime,
|
|
524
|
+
checkpointProposalPrepareTime: timetable.checkpointProposalPrepareTime,
|
|
525
|
+
maxNumberOfBlocks
|
|
526
|
+
});
|
|
527
|
+
this.assertConfigMeetsNetworkTxLimits(config, maxNumberOfBlocks);
|
|
528
|
+
return timetable;
|
|
529
|
+
}
|
|
530
|
+
/**
|
|
531
|
+
* Checks this node's configured per-block allocation against the network admission limit. A node
|
|
532
|
+
* advertises and admits txs up to the limit derived from the network-minimum multipliers (see
|
|
533
|
+
* {@link computeNetworkTxGasLimits}).
|
|
534
|
+
*
|
|
535
|
+
* Fails startup (and runtime config updates) only when the configured per-block allocation *multipliers*
|
|
536
|
+
* (`perBlockAllocationMultiplier` / `perBlockDAAllocationMultiplier`) are below the network minimums: such
|
|
537
|
+
* a node would accept txs over RPC/gossip that its builder can never pack into a block regardless of block
|
|
538
|
+
* size. Operators may configure a higher (more generous) multiplier, but not a lower one.
|
|
539
|
+
*
|
|
540
|
+
* When the multipliers meet the floor but an absolute per-block gas cap (`maxDABlockGas` / `maxL2BlockGas`)
|
|
541
|
+
* shrinks the builder's effective grant below the network limit, this is legitimate operator
|
|
542
|
+
* restrictiveness — the node simply builds smaller blocks and such txs stay in the pool for other
|
|
543
|
+
* proposers — so we only log a warning rather than failing startup. Restrictive tx-count caps
|
|
544
|
+
* (`maxTxsPerBlock` / `maxTxsPerCheckpoint`) can likewise make the builder skip admitted txs; they are
|
|
545
|
+
* intentionally not modeled here for the same reason.
|
|
546
|
+
*/ assertConfigMeetsNetworkTxLimits(config, maxBlocksPerCheckpoint) {
|
|
547
|
+
// Mirror CheckpointBuilder.capLimitsByCheckpointBudgets: DA falls back to the general multiplier.
|
|
548
|
+
const l2Multiplier = config.perBlockAllocationMultiplier;
|
|
549
|
+
const daMultiplier = config.perBlockDAAllocationMultiplier ?? l2Multiplier;
|
|
550
|
+
// The allocation is monotonic in the multiplier, so a multiplier at or above the network minimum
|
|
551
|
+
// guarantees the builder grants at least the network admission limit. Checking the multipliers directly
|
|
552
|
+
// is sufficient (and strictly more conservative than modeling the resulting gas grant).
|
|
553
|
+
if (daMultiplier < MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER) {
|
|
554
|
+
throw new Error(`perBlockDAAllocationMultiplier (${daMultiplier}) is below the network minimum ` + `${MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER}; the node would admit txs its own builder can never include.`);
|
|
555
|
+
}
|
|
556
|
+
if (l2Multiplier < MIN_PER_BLOCK_ALLOCATION_MULTIPLIER) {
|
|
557
|
+
throw new Error(`perBlockAllocationMultiplier (${l2Multiplier}) is below the network minimum ` + `${MIN_PER_BLOCK_ALLOCATION_MULTIPLIER}; the node would admit txs its own builder can never include.`);
|
|
558
|
+
}
|
|
559
|
+
// Absolute per-block gas caps below the network admission limit are legitimate operator restrictiveness:
|
|
560
|
+
// the node simply builds smaller blocks and such txs stay in the pool for other proposers. Warn only.
|
|
561
|
+
const networkLimit = computeNetworkTxGasLimits({
|
|
562
|
+
maxBlocksPerCheckpoint,
|
|
563
|
+
manaCheckpointBudget: this.l1Constants.rollupManaLimit
|
|
564
|
+
});
|
|
565
|
+
if (config.maxDABlockGas !== undefined && config.maxDABlockGas < networkLimit.daGas) {
|
|
566
|
+
this.log.warn(`Sequencer maxDABlockGas (${config.maxDABlockGas}) is below the network DA admission limit ` + `(${networkLimit.daGas}): txs declaring more DA gas are admitted over RPC/gossip but will be skipped ` + `by this proposer's own blocks and left in the pool for other proposers.`, {
|
|
567
|
+
maxDABlockGas: config.maxDABlockGas,
|
|
568
|
+
networkDaGas: networkLimit.daGas,
|
|
569
|
+
maxBlocksPerCheckpoint
|
|
570
|
+
});
|
|
571
|
+
}
|
|
572
|
+
if (config.maxL2BlockGas !== undefined && config.maxL2BlockGas < networkLimit.l2Gas) {
|
|
573
|
+
this.log.warn(`Sequencer maxL2BlockGas (${config.maxL2BlockGas}) is below the network L2 admission limit ` + `(${networkLimit.l2Gas}): txs declaring more L2 gas are admitted over RPC/gossip but will be skipped ` + `by this proposer's own blocks and left in the pool for other proposers.`, {
|
|
574
|
+
maxL2BlockGas: config.maxL2BlockGas,
|
|
575
|
+
networkL2Gas: networkLimit.l2Gas,
|
|
576
|
+
maxBlocksPerCheckpoint
|
|
577
|
+
});
|
|
578
|
+
}
|
|
480
579
|
}
|
|
481
580
|
/** Initializes the sequencer (precomputes tables). Takes about 3s. */ init() {
|
|
482
581
|
getKzg();
|
|
@@ -489,13 +588,18 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
489
588
|
this.runningPromise.start();
|
|
490
589
|
this.log.info('Started sequencer');
|
|
491
590
|
}
|
|
591
|
+
/** Triggers an immediate run of the sequencer, bypassing the polling interval. */ trigger() {
|
|
592
|
+
return this.runningPromise?.trigger();
|
|
593
|
+
}
|
|
492
594
|
/** Stops the sequencer from building blocks and moves to STOPPED state. */ async stop() {
|
|
493
595
|
this.log.info(`Stopping sequencer`);
|
|
494
596
|
this.setState(SequencerState.STOPPING, undefined, {
|
|
495
597
|
force: true
|
|
496
598
|
});
|
|
497
|
-
this.
|
|
599
|
+
this.lastCheckpointProposalJob?.interrupt();
|
|
600
|
+
await this.publisherFactory.stopAll();
|
|
498
601
|
await this.runningPromise?.stop();
|
|
602
|
+
await this.lastCheckpointProposalJob?.awaitPendingSubmission();
|
|
499
603
|
this.setState(SequencerState.STOPPED, undefined, {
|
|
500
604
|
force: true
|
|
501
605
|
});
|
|
@@ -508,19 +612,7 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
508
612
|
this.emit('checkpoint-error', {
|
|
509
613
|
error: err
|
|
510
614
|
});
|
|
511
|
-
|
|
512
|
-
// Log as warn only if we had to abort halfway through the block proposal
|
|
513
|
-
const logLvl = [
|
|
514
|
-
SequencerState.INITIALIZING_CHECKPOINT,
|
|
515
|
-
SequencerState.PROPOSER_CHECK
|
|
516
|
-
].includes(err.proposedState) ? 'debug' : 'warn';
|
|
517
|
-
this.log[logLvl](err.message, {
|
|
518
|
-
now: this.dateProvider.nowInSeconds()
|
|
519
|
-
});
|
|
520
|
-
} else {
|
|
521
|
-
// Re-throw other errors
|
|
522
|
-
throw err;
|
|
523
|
-
}
|
|
615
|
+
throw err;
|
|
524
616
|
} finally{
|
|
525
617
|
this.setState(SequencerState.IDLE, undefined);
|
|
526
618
|
}
|
|
@@ -539,63 +631,76 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
539
631
|
* - Submit checkpoint
|
|
540
632
|
*/ async work() {
|
|
541
633
|
this.setState(SequencerState.SYNCHRONIZING, undefined);
|
|
542
|
-
const { slot,
|
|
634
|
+
const { slot, targetSlot, epoch, targetEpoch, ts, nowSeconds } = this.getSlotContextInNextL1Slot();
|
|
543
635
|
// Check if we are synced and it's our slot, grab a publisher, check previous block invalidation, etc
|
|
544
|
-
const checkpointProposalJob = await this.prepareCheckpointProposal(epoch,
|
|
636
|
+
const checkpointProposalJob = await this.prepareCheckpointProposal(slot, targetSlot, epoch, targetEpoch, ts, nowSeconds);
|
|
545
637
|
if (!checkpointProposalJob) {
|
|
546
638
|
return;
|
|
547
639
|
}
|
|
640
|
+
// Track the job so we can await its pending L1 submission during shutdown
|
|
641
|
+
this.lastCheckpointProposalJob = checkpointProposalJob;
|
|
548
642
|
// Execute the checkpoint proposal job
|
|
549
643
|
const checkpoint = await checkpointProposalJob.execute();
|
|
550
644
|
// Update last checkpoint proposed (currently unused)
|
|
551
645
|
if (checkpoint) {
|
|
552
646
|
this.lastCheckpointProposed = checkpoint;
|
|
553
647
|
}
|
|
554
|
-
// Log fee strategy comparison if on fisherman
|
|
555
|
-
if (this.config.fishermanMode && (this.lastEpochForStrategyComparison === undefined ||
|
|
556
|
-
this.logStrategyComparison(
|
|
557
|
-
this.lastEpochForStrategyComparison =
|
|
648
|
+
// Log fee strategy comparison if on fisherman (uses target epoch since we mirror the proposer's perspective)
|
|
649
|
+
if (this.config.fishermanMode && (this.lastEpochForStrategyComparison === undefined || targetEpoch > this.lastEpochForStrategyComparison)) {
|
|
650
|
+
this.logStrategyComparison(targetEpoch, checkpointProposalJob.getPublisher());
|
|
651
|
+
this.lastEpochForStrategyComparison = targetEpoch;
|
|
558
652
|
}
|
|
559
653
|
return checkpoint;
|
|
560
654
|
}
|
|
655
|
+
/** Returns slot and target slot from a single clock snapshot. */ getSlotContextInNextL1Slot() {
|
|
656
|
+
const { slot, ts, nowSeconds, epoch } = this.epochCache.getEpochAndSlotInNextL1Slot();
|
|
657
|
+
const targetSlot = SlotNumber(slot + PROPOSER_PIPELINING_SLOT_OFFSET);
|
|
658
|
+
return {
|
|
659
|
+
slot,
|
|
660
|
+
targetSlot,
|
|
661
|
+
epoch,
|
|
662
|
+
targetEpoch: getEpochAtSlot(targetSlot, this.l1Constants),
|
|
663
|
+
ts,
|
|
664
|
+
nowSeconds
|
|
665
|
+
};
|
|
666
|
+
}
|
|
561
667
|
/**
|
|
562
668
|
* Prepares the checkpoint proposal by performing all necessary checks and setup.
|
|
563
669
|
* This is the initial step in the main loop.
|
|
564
670
|
* @returns CheckpointProposalJob if successful, undefined if we are not yet synced or are not the proposer.
|
|
565
|
-
*/ async prepareCheckpointProposal(epoch,
|
|
566
|
-
// Check we have not already processed this slot (cheapest check)
|
|
567
|
-
|
|
568
|
-
|
|
569
|
-
if (this.lastSlotForCheckpointProposalJob && this.lastSlotForCheckpointProposalJob >= slot && this.config.enforceTimeTable) {
|
|
570
|
-
this.log.trace(`Slot ${slot} has already been processed`);
|
|
671
|
+
*/ async prepareCheckpointProposal(slot, targetSlot, epoch, targetEpoch, ts, nowSeconds) {
|
|
672
|
+
// Check we have not already processed this target slot (cheapest check).
|
|
673
|
+
if (this.lastSlotForCheckpointProposalJob && this.lastSlotForCheckpointProposalJob >= targetSlot) {
|
|
674
|
+
this.log.trace(`Target slot ${targetSlot} has already been processed`);
|
|
571
675
|
return undefined;
|
|
572
676
|
}
|
|
573
|
-
// But if we have already proposed for this slot,
|
|
574
|
-
if (this.lastCheckpointProposed && this.lastCheckpointProposed.header.slotNumber >=
|
|
575
|
-
this.log.trace(`Slot ${
|
|
677
|
+
// But if we have already proposed for this slot, then we definitely have to skip it, automining or not
|
|
678
|
+
if (this.lastCheckpointProposed && this.lastCheckpointProposed.header.slotNumber >= targetSlot) {
|
|
679
|
+
this.log.trace(`Slot ${targetSlot} has already been published as checkpoint ${this.lastCheckpointProposed.number}`);
|
|
576
680
|
return undefined;
|
|
577
681
|
}
|
|
578
|
-
//
|
|
579
|
-
|
|
580
|
-
|
|
581
|
-
slot
|
|
582
|
-
|
|
583
|
-
if (!syncedTo) {
|
|
584
|
-
await this.tryVoteWhenSyncFails({
|
|
585
|
-
slot,
|
|
586
|
-
ts
|
|
682
|
+
// Test-only: skip proposing for explicitly paused slots. Attestation paths run in the validator
|
|
683
|
+
// client and are not gated by this hook, so paused proposers still attest to others' proposals.
|
|
684
|
+
if (this.config.pauseProposingForSlots?.some((s)=>s === targetSlot)) {
|
|
685
|
+
this.log.warn(`Skipping proposal for paused slot ${targetSlot} (test-only pauseProposingForSlots hook)`, {
|
|
686
|
+
targetSlot
|
|
587
687
|
});
|
|
588
688
|
return undefined;
|
|
589
689
|
}
|
|
590
|
-
//
|
|
690
|
+
// Cheap proposer check first: most nodes are not the proposer for most slots, so gate the
|
|
691
|
+
// expensive multi-subsystem checkSync (and the rest of the build path) behind it. Computed once
|
|
692
|
+
// here and reused for the escape-hatch voting path below. No setState/timing gate on this path:
|
|
693
|
+
// the build-start deadline gate runs only on the proposer build path after a successful checkSync.
|
|
694
|
+
const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
|
|
695
|
+
// If escape hatch is open for the target epoch, do not start checkpoint proposal work and do not attempt invalidations.
|
|
591
696
|
// Still perform governance/slashing voting (as proposer) once per slot.
|
|
592
|
-
|
|
697
|
+
// When pipelining, we check the target epoch (slot+1's epoch) since that's the epoch we're building for.
|
|
698
|
+
const isEscapeHatchOpen = await this.epochCache.isEscapeHatchOpen(targetEpoch);
|
|
593
699
|
if (isEscapeHatchOpen) {
|
|
594
|
-
this.setState(SequencerState.PROPOSER_CHECK, slot);
|
|
595
|
-
const [canPropose, proposer] = await this.checkCanPropose(slot);
|
|
596
700
|
if (canPropose) {
|
|
597
701
|
await this.tryVoteWhenEscapeHatchOpen({
|
|
598
702
|
slot,
|
|
703
|
+
targetSlot,
|
|
599
704
|
proposer
|
|
600
705
|
});
|
|
601
706
|
} else {
|
|
@@ -607,27 +712,83 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
607
712
|
}
|
|
608
713
|
return undefined;
|
|
609
714
|
}
|
|
715
|
+
// If we are not the proposer, check whether we should invalidate an invalid pending chain (a
|
|
716
|
+
// liveness backstop) and bail before any sync work or build-timing gate. This reads only the
|
|
717
|
+
// archiver's pending-chain validation status, which is authoritative on its own, instead of
|
|
718
|
+
// running the full sync check. Wrapped in try/catch because this leaner path skips the broader
|
|
719
|
+
// proposed-checkpoint/tip coherence screen that checkSync applied, so transient archiver
|
|
720
|
+
// incoherence surfaces as a quiet skip rather than a work-loop error.
|
|
721
|
+
if (!canPropose) {
|
|
722
|
+
try {
|
|
723
|
+
const pendingChainValidationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
|
|
724
|
+
await this.considerInvalidatingCheckpoint(pendingChainValidationStatus, slot);
|
|
725
|
+
} catch (err) {
|
|
726
|
+
this.log.warn(`Failed to consider invalidating checkpoint`, {
|
|
727
|
+
err,
|
|
728
|
+
slot,
|
|
729
|
+
targetSlot
|
|
730
|
+
});
|
|
731
|
+
}
|
|
732
|
+
return undefined;
|
|
733
|
+
}
|
|
734
|
+
// We are the proposer and the escape hatch is closed: now run the full sync check before building.
|
|
735
|
+
const syncedTo = await this.checkSync({
|
|
736
|
+
ts,
|
|
737
|
+
slot
|
|
738
|
+
});
|
|
739
|
+
if (!syncedTo) {
|
|
740
|
+
await this.tryVoteWhenCannotBuild({
|
|
741
|
+
slot,
|
|
742
|
+
targetSlot
|
|
743
|
+
});
|
|
744
|
+
return undefined;
|
|
745
|
+
}
|
|
746
|
+
// Explicit build-loop entry gate: if we are past the latest useful block-building start for the
|
|
747
|
+
// target slot, abandon building for this slot. The proposer prioritizes the ideal L1-publish path
|
|
748
|
+
// and does not plan around the late consensus-handoff path. This is the proposer build path's
|
|
749
|
+
// timing gate; it runs only after we know we are the synced proposer, so non-proposer invalidation
|
|
750
|
+
// and escape-hatch voting (which returned above) are never gated by build timing. Vote-only paths
|
|
751
|
+
// still run when block building is abandoned.
|
|
752
|
+
const startDeadline = this.timetable.getBuildStartDeadline(targetSlot);
|
|
753
|
+
const nowForStartGate = this.dateProvider.now() / 1000;
|
|
754
|
+
if (nowForStartGate > startDeadline) {
|
|
755
|
+
this.log.debug(`Past start deadline for slot ${targetSlot}, abandoning block building`, {
|
|
756
|
+
targetSlot,
|
|
757
|
+
nowForStartGate,
|
|
758
|
+
startDeadline
|
|
759
|
+
});
|
|
760
|
+
// Mark the slot as attempted so a deadline abort is not retried within the same slot. Vote-only actions
|
|
761
|
+
// still need to run because sync can succeed even when it is too late to start building a checkpoint.
|
|
762
|
+
await this.tryVoteWhenCannotBuild({
|
|
763
|
+
slot,
|
|
764
|
+
targetSlot
|
|
765
|
+
});
|
|
766
|
+
this.lastSlotForCheckpointProposalJob = targetSlot;
|
|
767
|
+
return undefined;
|
|
768
|
+
}
|
|
610
769
|
// Next checkpoint follows from the last synced one
|
|
611
770
|
const checkpointNumber = CheckpointNumber(syncedTo.checkpointNumber + 1);
|
|
612
771
|
const logCtx = {
|
|
613
|
-
|
|
772
|
+
nowSeconds,
|
|
614
773
|
syncedToL2Slot: syncedTo.syncedL2Slot,
|
|
615
774
|
slot,
|
|
775
|
+
targetSlot,
|
|
616
776
|
slotTs: ts,
|
|
617
777
|
checkpointNumber,
|
|
618
778
|
isPendingChainValid: pick(syncedTo.pendingChainValidationStatus, 'valid', 'reason', 'invalidIndex')
|
|
619
779
|
};
|
|
620
|
-
//
|
|
621
|
-
this.setState(SequencerState.PROPOSER_CHECK,
|
|
622
|
-
|
|
623
|
-
//
|
|
624
|
-
|
|
625
|
-
|
|
780
|
+
// We are the synced proposer within the build window; enter the proposer-check state and build.
|
|
781
|
+
this.setState(SequencerState.PROPOSER_CHECK, targetSlot);
|
|
782
|
+
// Guard: don't exceed 1-deep pipeline. Without a proposed checkpoint, we can only build
|
|
783
|
+
// confirmed + 1. With a proposed checkpoint, we can build confirmed + 2.
|
|
784
|
+
const confirmedCkpt = syncedTo.checkpointedCheckpointNumber;
|
|
785
|
+
if (checkpointNumber > confirmedCkpt + 2) {
|
|
786
|
+
this.log.verbose(`Skipping slot ${targetSlot}: checkpoint ${checkpointNumber} exceeds max pipeline depth (confirmed=${confirmedCkpt})`);
|
|
626
787
|
return undefined;
|
|
627
788
|
}
|
|
628
|
-
// Check that the slot is not taken by a block already (should never happen, since only us can propose for this slot)
|
|
629
|
-
if (syncedTo.blockData && syncedTo.blockData.header.getSlot() >=
|
|
630
|
-
this.log.warn(`Cannot propose block at
|
|
789
|
+
// Check that the target slot is not taken by a block already (should never happen, since only us can propose for this slot)
|
|
790
|
+
if (syncedTo.blockData && syncedTo.blockData.header.getSlot() >= targetSlot) {
|
|
791
|
+
this.log.warn(`Cannot propose block at target slot ${targetSlot} since that slot was taken by block ${syncedTo.blockNumber}`, {
|
|
631
792
|
...logCtx,
|
|
632
793
|
block: syncedTo.blockData.header.toInspect()
|
|
633
794
|
});
|
|
@@ -641,18 +802,84 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
641
802
|
const proposerForPublisher = this.config.fishermanMode ? undefined : proposer;
|
|
642
803
|
const { attestorAddress, publisher } = await this.publisherFactory.create(proposerForPublisher);
|
|
643
804
|
this.log.verbose(`Created publisher at address ${publisher.getSenderAddress()} for attestor ${attestorAddress}`);
|
|
644
|
-
//
|
|
645
|
-
|
|
646
|
-
|
|
647
|
-
|
|
648
|
-
|
|
649
|
-
//
|
|
650
|
-
|
|
651
|
-
|
|
652
|
-
|
|
653
|
-
|
|
805
|
+
// Prepare invalidation request if the pending chain is invalid (returns undefined if no need).
|
|
806
|
+
// Only simulate invalidation when there's no proposed parent, since we assume the proposed parent
|
|
807
|
+
// will invalidate the currently invalid checkpoint on L1.
|
|
808
|
+
const invalidateCheckpoint = syncedTo.hasProposedCheckpoint || syncedTo.pendingChainValidationStatus.valid ? undefined : await publisher.simulateInvalidateCheckpoint(syncedTo.pendingChainValidationStatus);
|
|
809
|
+
// Determine the correct archive and L1 state overrides for the canProposeAt check.
|
|
810
|
+
// The L1 contract reads archives[proposedCheckpointNumber] and compares it with the provided archive.
|
|
811
|
+
// When invalidating or pipelining, the local archive may differ from L1's, so we adjust accordingly.
|
|
812
|
+
let archiveForCheck = syncedTo.archive;
|
|
813
|
+
if (syncedTo.hasProposedCheckpoint) {
|
|
814
|
+
this.metrics.recordPipelineDepth(syncedTo.checkpointNumber - syncedTo.checkpointedCheckpointNumber);
|
|
815
|
+
this.log.verbose(`Building on top of proposed checkpoint (pending=${syncedTo.proposedCheckpointData?.checkpointNumber}) for target slot ${targetSlot}`, {
|
|
816
|
+
targetSlot,
|
|
817
|
+
parentCheckpointNumber: CheckpointNumber(checkpointNumber - 1)
|
|
818
|
+
});
|
|
819
|
+
// Match what L1 will see at archives[pending] once the proposed parent lands: the parent's
|
|
820
|
+
// own archive root from the gossiped proposal. `syncedTo.archive` is the world-state-local
|
|
821
|
+
// view and can transiently diverge from the proposed parent (e.g. before the proposed
|
|
822
|
+
// parent's blocks have been applied locally); diverging here would cause the canProposeAt
|
|
823
|
+
// override to set archives[pending] to one value while we present another for comparison.
|
|
824
|
+
archiveForCheck = syncedTo.proposedCheckpointData.archive.root;
|
|
825
|
+
} else if (invalidateCheckpoint) {
|
|
826
|
+
// After invalidation, L1 will roll back to checkpoint N-1. The archive at N-1 already
|
|
827
|
+
// exists on L1, so we just pass the matching archive (the lastArchive of the invalid checkpoint).
|
|
828
|
+
archiveForCheck = invalidateCheckpoint.lastArchive;
|
|
829
|
+
this.metrics.recordPipelineDepth(0);
|
|
830
|
+
} else {
|
|
831
|
+
this.metrics.recordPipelineDepth(0);
|
|
832
|
+
}
|
|
833
|
+
// Build the simulation plan: pending/proven override from pipelining or invalidation (or the
|
|
834
|
+
// current snapshot when neither applies, to short-circuit any pending prune in simulation),
|
|
835
|
+
// plus the parent checkpoint cell and fee header when pipelining.
|
|
836
|
+
const simulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
|
|
837
|
+
checkpointNumber,
|
|
838
|
+
proposedCheckpointData: syncedTo.hasProposedCheckpoint ? syncedTo.proposedCheckpointData : undefined,
|
|
839
|
+
invalidateToPendingCheckpointNumber: invalidateCheckpoint?.forcePendingCheckpointNumber,
|
|
840
|
+
checkpointedCheckpointNumber: syncedTo.checkpointedCheckpointNumber,
|
|
841
|
+
rollup: this.rollupContract,
|
|
842
|
+
signatureContext: this.signatureContext,
|
|
843
|
+
log: this.log
|
|
844
|
+
});
|
|
845
|
+
// The plan always pins both pending/proven (to short-circuit `canPruneAtTime` in simulation),
|
|
846
|
+
// so `provenOverride` always reflects the assumed proven checkpoint we are pinning the
|
|
847
|
+
// simulation to. We additionally warn when the pin is load-bearing — i.e. when a prune would
|
|
848
|
+
// actually fire at the target slot without it — so observers can spot "we are building
|
|
849
|
+
// optimistically across a pruning boundary" in the logs.
|
|
850
|
+
const provenOverride = simulationOverridesPlan?.chainTipsOverride?.proven;
|
|
851
|
+
if (provenOverride !== undefined && await this.l2BlockSource.isPruneDueAtSlot(targetSlot)) {
|
|
852
|
+
this.log.warn(`Assuming proof for epoch ending at checkpoint ${provenOverride} lands by target slot ${targetSlot}`, {
|
|
853
|
+
checkpointNumber,
|
|
854
|
+
slot,
|
|
855
|
+
targetSlot,
|
|
856
|
+
provenOverride
|
|
857
|
+
});
|
|
858
|
+
}
|
|
859
|
+
this.emit('preparing-checkpoint', {
|
|
860
|
+
targetSlot,
|
|
861
|
+
checkpointNumber,
|
|
862
|
+
hadProposedParent: syncedTo.hasProposedCheckpoint,
|
|
863
|
+
provenOverride,
|
|
864
|
+
simulatedPending: simulationOverridesPlan?.chainTipsOverride?.pending
|
|
865
|
+
});
|
|
866
|
+
const canProposeCheck = await publisher.canProposeAt(archiveForCheck, proposer ?? EthAddress.ZERO, simulationOverridesPlan);
|
|
867
|
+
const proposeContext = {
|
|
868
|
+
hasProposedCheckpoint: syncedTo.hasProposedCheckpoint,
|
|
869
|
+
proposedCheckpointNumber: syncedTo.proposedCheckpointData?.checkpointNumber,
|
|
870
|
+
checkpointedCheckpointNumber: syncedTo.checkpointedCheckpointNumber,
|
|
871
|
+
isInvalidating: !!invalidateCheckpoint,
|
|
872
|
+
invalidatingCheckpointNumber: invalidateCheckpoint?.checkpointNumber,
|
|
873
|
+
archiveForCheck: archiveForCheck.toString(),
|
|
874
|
+
overridePendingCheckpointNumber: simulationOverridesPlan?.chainTipsOverride?.pending,
|
|
875
|
+
overrideArchive: simulationOverridesPlan?.pendingCheckpointState?.archive,
|
|
876
|
+
overrideFeeHeader: simulationOverridesPlan?.pendingCheckpointState?.feeHeader
|
|
877
|
+
};
|
|
654
878
|
if (canProposeCheck === undefined) {
|
|
655
|
-
this.log.warn(`Cannot propose checkpoint ${checkpointNumber} at slot ${slot} due to failed rollup contract check`,
|
|
879
|
+
this.log.warn(`Cannot propose checkpoint ${checkpointNumber} at slot ${slot} due to failed rollup contract check`, {
|
|
880
|
+
...logCtx,
|
|
881
|
+
...proposeContext
|
|
882
|
+
});
|
|
656
883
|
this.emit('proposer-rollup-check-failed', {
|
|
657
884
|
reason: 'Rollup contract check failed',
|
|
658
885
|
slot
|
|
@@ -660,11 +887,12 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
660
887
|
this.metrics.recordCheckpointPrecheckFailed('rollup_contract_check_failed');
|
|
661
888
|
return undefined;
|
|
662
889
|
}
|
|
663
|
-
if (canProposeCheck.slot !==
|
|
664
|
-
this.log.warn(`Cannot propose block due to slot mismatch with rollup contract (this can be caused by a clock out of sync). Expected slot ${
|
|
890
|
+
if (canProposeCheck.slot !== targetSlot) {
|
|
891
|
+
this.log.warn(`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}.`, {
|
|
665
892
|
...logCtx,
|
|
893
|
+
...proposeContext,
|
|
666
894
|
rollup: canProposeCheck,
|
|
667
|
-
expectedSlot:
|
|
895
|
+
expectedSlot: targetSlot
|
|
668
896
|
});
|
|
669
897
|
this.emit('proposer-rollup-check-failed', {
|
|
670
898
|
reason: 'Slot mismatch',
|
|
@@ -676,6 +904,7 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
676
904
|
if (canProposeCheck.checkpointNumber !== checkpointNumber) {
|
|
677
905
|
this.log.warn(`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}.`, {
|
|
678
906
|
...logCtx,
|
|
907
|
+
...proposeContext,
|
|
679
908
|
rollup: canProposeCheck,
|
|
680
909
|
expectedSlot: slot
|
|
681
910
|
});
|
|
@@ -686,17 +915,18 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
686
915
|
this.metrics.recordCheckpointPrecheckFailed('block_number_mismatch');
|
|
687
916
|
return undefined;
|
|
688
917
|
}
|
|
689
|
-
this.lastSlotForCheckpointProposalJob =
|
|
690
|
-
await this.p2pClient.prepareForSlot(
|
|
691
|
-
this.log.info(`Preparing checkpoint proposal ${checkpointNumber}
|
|
918
|
+
this.lastSlotForCheckpointProposalJob = targetSlot;
|
|
919
|
+
await this.p2pClient.prepareForSlot(targetSlot);
|
|
920
|
+
this.log.info(`Preparing checkpoint proposal ${checkpointNumber} for target slot ${targetSlot} during wall-clock slot ${slot}`, {
|
|
692
921
|
...logCtx,
|
|
922
|
+
...proposeContext,
|
|
693
923
|
proposer
|
|
694
924
|
});
|
|
695
925
|
// Create and return the checkpoint proposal job
|
|
696
|
-
return this.createCheckpointProposalJob(
|
|
926
|
+
return this.createCheckpointProposalJob(targetSlot, targetEpoch, checkpointNumber, syncedTo.blockNumber, syncedTo.checkpointedCheckpointNumber, proposer, publisher, attestorAddress, invalidateCheckpoint, syncedTo.proposedCheckpointData);
|
|
697
927
|
}
|
|
698
|
-
createCheckpointProposalJob(
|
|
699
|
-
return new CheckpointProposalJob(
|
|
928
|
+
createCheckpointProposalJob(targetSlot, targetEpoch, checkpointNumber, syncedToBlockNumber, checkpointedCheckpointNumber, proposer, publisher, attestorAddress, invalidateCheckpoint, proposedCheckpointData) {
|
|
929
|
+
return new CheckpointProposalJob(targetSlot, targetEpoch, checkpointNumber, syncedToBlockNumber, checkpointedCheckpointNumber, proposer, publisher, attestorAddress, invalidateCheckpoint, this.validatorClient, this.globalsBuilder, this.p2pClient, this.worldState, this.l1ToL2MessageSource, this.l2BlockSource, this.checkpointsBuilder, this.l2BlockSource, this.l1Constants, this.signatureContext, this.config, this.timetable, this.slasherClient, this.epochCache, this.dateProvider, this.metrics, this.checkpointProposalJobMetrics.createRecorder(), this, this.setState.bind(this), this.tracer, this.log.getBindings(), proposedCheckpointData);
|
|
700
930
|
}
|
|
701
931
|
/**
|
|
702
932
|
* Returns the current sequencer state.
|
|
@@ -704,9 +934,11 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
704
934
|
return this.state;
|
|
705
935
|
}
|
|
706
936
|
/**
|
|
707
|
-
* Internal helper for setting the sequencer state
|
|
937
|
+
* Internal helper for setting the sequencer state. Pure: sets the state, emits `state-changed`, and
|
|
938
|
+
* records metrics. Timing deadlines are queried explicitly at the relevant call sites, not gated here.
|
|
708
939
|
* @param proposedState - The new state to transition to.
|
|
709
|
-
* @param slotNumber - The
|
|
940
|
+
* @param slotNumber - The target slot being proposed for, emitted as `targetSlot` on the event payload
|
|
941
|
+
* and used to anchor `secondsIntoBuildFrame`. Undefined for lifecycle states with no associated slot.
|
|
710
942
|
* @param force - Whether to force the transition even if the sequencer is stopped.
|
|
711
943
|
*/ setState(proposedState, slotNumber, opts = {}) {
|
|
712
944
|
if (this.state === SequencerState.STOPPING && proposedState !== SequencerState.STOPPED && !opts.force) {
|
|
@@ -717,26 +949,40 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
717
949
|
this.log.warn(`Cannot set sequencer from ${this.state} to ${proposedState} as it is stopped.`);
|
|
718
950
|
return;
|
|
719
951
|
}
|
|
720
|
-
|
|
721
|
-
|
|
722
|
-
|
|
723
|
-
|
|
724
|
-
|
|
952
|
+
const secondsIntoBuildFrame = slotNumber !== undefined ? this.getSecondsIntoBuildFrame(slotNumber) : undefined;
|
|
953
|
+
const oldState = this.state;
|
|
954
|
+
const oldStateSlotNumber = this.stateSlotNumber;
|
|
955
|
+
const stateChanged = proposedState !== oldState;
|
|
956
|
+
// Wall-clock time spent in the previous state: the delta between consecutive state-changing setState
|
|
957
|
+
// calls, read from the date provider so it tracks simulated time under a test/manual clock.
|
|
958
|
+
const transitionAtMs = this.dateProvider.now();
|
|
959
|
+
const stateDurationMs = transitionAtMs - this.stateEnteredAtMs;
|
|
725
960
|
const boringStates = [
|
|
726
961
|
SequencerState.IDLE,
|
|
727
962
|
SequencerState.SYNCHRONIZING
|
|
728
963
|
];
|
|
729
|
-
const logLevel = boringStates.includes(proposedState) && boringStates.includes(
|
|
730
|
-
this.
|
|
964
|
+
const logLevel = boringStates.includes(proposedState) && boringStates.includes(oldState) ? 'trace' : 'debug';
|
|
965
|
+
this.stateLog[logLevel](`Transitioning from ${oldState} to ${proposedState}`, {
|
|
966
|
+
oldState,
|
|
967
|
+
newState: proposedState,
|
|
731
968
|
slotNumber,
|
|
732
|
-
|
|
969
|
+
stateSlotNumber: oldStateSlotNumber,
|
|
970
|
+
secondsIntoBuildFrame,
|
|
971
|
+
...stateChanged && {
|
|
972
|
+
stateDurationMs: Math.ceil(stateDurationMs)
|
|
973
|
+
}
|
|
733
974
|
});
|
|
734
975
|
this.emit('state-changed', {
|
|
735
|
-
oldState
|
|
976
|
+
oldState,
|
|
736
977
|
newState: proposedState,
|
|
737
|
-
|
|
738
|
-
|
|
978
|
+
secondsIntoBuildFrame,
|
|
979
|
+
targetSlot: slotNumber
|
|
739
980
|
});
|
|
981
|
+
if (stateChanged) {
|
|
982
|
+
this.metrics.recordStateDuration(stateDurationMs, oldState);
|
|
983
|
+
this.stateEnteredAtMs = transitionAtMs;
|
|
984
|
+
this.stateSlotNumber = slotNumber;
|
|
985
|
+
}
|
|
740
986
|
this.state = proposedState;
|
|
741
987
|
}
|
|
742
988
|
/**
|
|
@@ -744,8 +990,8 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
744
990
|
* We don't check against the previous block submitted since it may have been reorg'd out.
|
|
745
991
|
*/ async checkSync(args) {
|
|
746
992
|
// Check that the archiver has fully synced the L2 slot before the one we want to propose in.
|
|
747
|
-
//
|
|
748
|
-
//
|
|
993
|
+
// The archiver reports sync progress via L1 block timestamps and synced checkpoint slots.
|
|
994
|
+
// See getSyncedL2SlotNumber for how missed L1 blocks are handled.
|
|
749
995
|
const syncedL2Slot = await this.l2BlockSource.getSyncedL2SlotNumber();
|
|
750
996
|
const { slot } = args;
|
|
751
997
|
if (syncedL2Slot === undefined || syncedL2Slot + 1 < slot) {
|
|
@@ -760,48 +1006,98 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
760
1006
|
number: syncSummary.latestBlockNumber,
|
|
761
1007
|
hash: syncSummary.latestBlockHash
|
|
762
1008
|
})),
|
|
763
|
-
this.l2BlockSource.getL2Tips().then((t)=>
|
|
1009
|
+
this.l2BlockSource.getL2Tips().then((t)=>({
|
|
1010
|
+
proposed: t.proposed,
|
|
1011
|
+
checkpointed: t.checkpointed,
|
|
1012
|
+
proposedCheckpoint: t.proposedCheckpoint
|
|
1013
|
+
})),
|
|
764
1014
|
this.p2pClient.getStatus().then((p2p)=>p2p.syncedToL2Block),
|
|
765
|
-
this.l1ToL2MessageSource.getL2Tips().then((t)=>
|
|
766
|
-
|
|
1015
|
+
this.l1ToL2MessageSource.getL2Tips().then((t)=>({
|
|
1016
|
+
proposed: t.proposed,
|
|
1017
|
+
checkpointed: t.checkpointed
|
|
1018
|
+
})),
|
|
1019
|
+
this.l2BlockSource.getPendingChainValidationStatus(),
|
|
1020
|
+
this.l2BlockSource.getProposedCheckpointData()
|
|
767
1021
|
]);
|
|
768
|
-
const [worldState,
|
|
769
|
-
|
|
770
|
-
// as the world state can compute the new genesis block hash, but other components use the hardcoded constant.
|
|
771
|
-
// TODO(palla/mbps): Fix the above. All components should be able to handle dynamic genesis block hashes.
|
|
772
|
-
const result = l2BlockSource.number === 0 && worldState.number === 0 && p2p.number === 0 && l1ToL2MessageSource.number === 0 || worldState.hash === l2BlockSource.hash && p2p.hash === l2BlockSource.hash && l1ToL2MessageSource.hash === l2BlockSource.hash;
|
|
1022
|
+
const [worldState, l2Tips, p2p, l1ToL2MessageSourceTips, pendingChainValidationStatus, proposedCheckpointData] = syncedBlocks;
|
|
1023
|
+
const result = worldState.hash === l2Tips.proposed.hash && p2p.hash === l2Tips.proposed.hash && l1ToL2MessageSourceTips.proposed.hash === l2Tips.proposed.hash && l1ToL2MessageSourceTips.checkpointed.block.hash === l2Tips.checkpointed.block.hash && l1ToL2MessageSourceTips.checkpointed.checkpoint.hash === l2Tips.checkpointed.checkpoint.hash;
|
|
773
1024
|
if (!result) {
|
|
774
1025
|
this.log.debug(`Sequencer sync check failed`, {
|
|
775
1026
|
worldState,
|
|
776
|
-
l2BlockSource,
|
|
1027
|
+
l2BlockSource: l2Tips.proposed,
|
|
777
1028
|
p2p,
|
|
778
|
-
|
|
1029
|
+
l1ToL2MessageSourceTips
|
|
779
1030
|
});
|
|
780
1031
|
return undefined;
|
|
781
1032
|
}
|
|
782
|
-
// Special case for genesis state
|
|
783
1033
|
const blockNumber = worldState.number;
|
|
784
|
-
|
|
785
|
-
|
|
786
|
-
|
|
787
|
-
|
|
788
|
-
|
|
789
|
-
|
|
1034
|
+
const blockData = await this.l2BlockSource.getBlockData({
|
|
1035
|
+
number: blockNumber
|
|
1036
|
+
});
|
|
1037
|
+
if (!blockData) {
|
|
1038
|
+
this.log.warn(`Sequencer sync check failed: failed to get L2 block data ${blockNumber} from the archiver`, {
|
|
1039
|
+
blockNumber,
|
|
1040
|
+
l2Tips,
|
|
790
1041
|
syncedL2Slot,
|
|
791
|
-
|
|
1042
|
+
...args
|
|
1043
|
+
});
|
|
1044
|
+
return undefined;
|
|
1045
|
+
}
|
|
1046
|
+
// Refuse to build a checkpoint on top of a proposed block whose enclosing checkpoint was never
|
|
1047
|
+
// proposed. Under pipelining we may have received and reexecuted such a block locally — advancing
|
|
1048
|
+
// our world-state tip past the checkpointed tip — while the proposing node never published the
|
|
1049
|
+
// matching proposed checkpoint (e.g. it crashed before assembling it). Building on this orphan block
|
|
1050
|
+
// would fork the chain off a tip no other node can follow. The archiver prunes these orphan blocks
|
|
1051
|
+
// once their build slot ends; this guard is the correctness barrier during the grace window before.
|
|
1052
|
+
if (blockData.checkpointNumber > l2Tips.checkpointed.checkpoint.number && (l2Tips.proposedCheckpoint.checkpoint.number !== blockData.checkpointNumber || proposedCheckpointData?.checkpointNumber !== blockData.checkpointNumber)) {
|
|
1053
|
+
const logCtx = {
|
|
1054
|
+
blockCheckpointNumber: blockData.checkpointNumber,
|
|
1055
|
+
checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
|
|
1056
|
+
proposedCheckpointTipNumber: l2Tips.proposedCheckpoint.checkpoint.number,
|
|
1057
|
+
proposedCheckpointDataNumber: proposedCheckpointData?.checkpointNumber,
|
|
1058
|
+
blockNumber: blockData.header.getBlockNumber(),
|
|
1059
|
+
blockSlot: blockData.header.getSlot(),
|
|
1060
|
+
syncedL2Slot,
|
|
1061
|
+
...args
|
|
792
1062
|
};
|
|
1063
|
+
this.log.debug(`Waiting for proposed checkpoint to catch up with reexecuted block`, logCtx);
|
|
1064
|
+
return undefined;
|
|
793
1065
|
}
|
|
794
|
-
const
|
|
795
|
-
|
|
796
|
-
|
|
797
|
-
|
|
1066
|
+
const hasProposedCheckpoint = l2Tips.proposedCheckpoint.checkpoint.number > l2Tips.checkpointed.checkpoint.number;
|
|
1067
|
+
// The l2Tips and proposedCheckpointData reads above come from independent archiver snapshots
|
|
1068
|
+
// (a JS-side tips cache vs. a direct store read on `#proposedCheckpoints`). A concurrent archiver
|
|
1069
|
+
// write that mutates both can be observed split, leaving us with `hasProposedCheckpoint=true` but
|
|
1070
|
+
// no proposedCheckpointData (or one whose number doesn't match the tip). Refuse to proceed in that
|
|
1071
|
+
// window — the next checkSync tick will see a coherent snapshot.
|
|
1072
|
+
if (hasProposedCheckpoint && (!proposedCheckpointData || proposedCheckpointData.checkpointNumber !== l2Tips.proposedCheckpoint.checkpoint.number)) {
|
|
1073
|
+
this.log.warn(`Sequencer sync check failed: inconsistent proposed-checkpoint state`, {
|
|
1074
|
+
proposedCheckpointTipNumber: l2Tips.proposedCheckpoint.checkpoint.number,
|
|
1075
|
+
checkpointedTipNumber: l2Tips.checkpointed.checkpoint.number,
|
|
1076
|
+
proposedCheckpointDataNumber: proposedCheckpointData?.checkpointNumber,
|
|
1077
|
+
syncedL2Slot,
|
|
1078
|
+
...args
|
|
1079
|
+
});
|
|
1080
|
+
return undefined;
|
|
1081
|
+
}
|
|
1082
|
+
// Check that the proposed checkpoint is indeed the parent of the checkpoint we'll be building
|
|
1083
|
+
// The checkpoint number to build is derived as blockData.checkpointNumber + 1
|
|
1084
|
+
if (proposedCheckpointData && proposedCheckpointData.checkpointNumber !== blockData.checkpointNumber) {
|
|
1085
|
+
this.log.warn(`Sequencer sync check failed: proposed checkpoint number mismatch`, {
|
|
1086
|
+
proposedCheckpointNumber: proposedCheckpointData.checkpointNumber,
|
|
1087
|
+
blockCheckpointNumber: blockData.checkpointNumber,
|
|
1088
|
+
syncedL2Slot,
|
|
1089
|
+
...args
|
|
1090
|
+
});
|
|
798
1091
|
return undefined;
|
|
799
1092
|
}
|
|
800
1093
|
return {
|
|
801
1094
|
blockData,
|
|
802
1095
|
blockNumber: blockData.header.getBlockNumber(),
|
|
803
1096
|
checkpointNumber: blockData.checkpointNumber,
|
|
1097
|
+
checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
|
|
804
1098
|
archive: blockData.archive.root,
|
|
1099
|
+
hasProposedCheckpoint,
|
|
1100
|
+
proposedCheckpointData,
|
|
805
1101
|
syncedL2Slot,
|
|
806
1102
|
pendingChainValidationStatus
|
|
807
1103
|
};
|
|
@@ -809,22 +1105,22 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
809
1105
|
/**
|
|
810
1106
|
* Checks if we are the proposer for the next slot.
|
|
811
1107
|
* @returns True if we can propose, and the proposer address (undefined if anyone can propose)
|
|
812
|
-
*/ async checkCanPropose(
|
|
1108
|
+
*/ async checkCanPropose(targetSlot) {
|
|
813
1109
|
let proposer;
|
|
814
1110
|
try {
|
|
815
|
-
proposer = await this.epochCache.getProposerAttesterAddressInSlot(
|
|
1111
|
+
proposer = await this.epochCache.getProposerAttesterAddressInSlot(targetSlot);
|
|
816
1112
|
} catch (e) {
|
|
817
1113
|
if (e instanceof NoCommitteeError) {
|
|
818
|
-
if (this.lastSlotForNoCommitteeWarning !==
|
|
819
|
-
this.lastSlotForNoCommitteeWarning =
|
|
820
|
-
this.log.warn(`Cannot propose at
|
|
1114
|
+
if (this.lastSlotForNoCommitteeWarning !== targetSlot) {
|
|
1115
|
+
this.lastSlotForNoCommitteeWarning = targetSlot;
|
|
1116
|
+
this.log.warn(`Cannot propose at target slot ${targetSlot} since the committee does not exist on L1`);
|
|
821
1117
|
}
|
|
822
1118
|
return [
|
|
823
1119
|
false,
|
|
824
1120
|
undefined
|
|
825
1121
|
];
|
|
826
1122
|
}
|
|
827
|
-
this.log.error(`Error getting proposer for slot ${
|
|
1123
|
+
this.log.error(`Error getting proposer for target slot ${targetSlot}`, e);
|
|
828
1124
|
return [
|
|
829
1125
|
false,
|
|
830
1126
|
undefined
|
|
@@ -847,7 +1143,8 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
847
1143
|
const validatorAddresses = this.validatorClient.getValidatorAddresses();
|
|
848
1144
|
const weAreProposer = validatorAddresses.some((addr)=>addr.equals(proposer));
|
|
849
1145
|
if (!weAreProposer) {
|
|
850
|
-
this.log.debug(`Cannot propose at slot ${
|
|
1146
|
+
this.log.debug(`Cannot propose at target slot ${targetSlot} since we are not a proposer`, {
|
|
1147
|
+
targetSlot,
|
|
851
1148
|
validatorAddresses,
|
|
852
1149
|
proposer
|
|
853
1150
|
});
|
|
@@ -856,39 +1153,36 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
856
1153
|
proposer
|
|
857
1154
|
];
|
|
858
1155
|
}
|
|
1156
|
+
this.log.info(`We are the proposer for pipeline slot ${targetSlot}`, {
|
|
1157
|
+
targetSlot,
|
|
1158
|
+
proposer
|
|
1159
|
+
});
|
|
859
1160
|
return [
|
|
860
1161
|
true,
|
|
861
1162
|
proposer
|
|
862
1163
|
];
|
|
863
1164
|
}
|
|
864
1165
|
/**
|
|
865
|
-
* Tries to vote on slashing actions and governance when
|
|
1166
|
+
* Tries to vote on slashing actions and governance when we cannot build and are past the block-building window.
|
|
866
1167
|
* This allows the sequencer to participate in governance/slashing votes even when it cannot build blocks.
|
|
867
|
-
*/ async
|
|
868
|
-
const { slot } = args;
|
|
1168
|
+
*/ async tryVoteWhenCannotBuild(args) {
|
|
1169
|
+
const { slot, targetSlot } = args;
|
|
869
1170
|
// Prevent duplicate attempts in the same slot
|
|
870
1171
|
if (this.lastSlotForFallbackVote === slot) {
|
|
871
1172
|
this.log.trace(`Already attempted to vote in slot ${slot} (skipping)`);
|
|
872
1173
|
return;
|
|
873
1174
|
}
|
|
874
|
-
//
|
|
875
|
-
|
|
876
|
-
|
|
877
|
-
|
|
878
|
-
|
|
879
|
-
|
|
880
|
-
|
|
881
|
-
|
|
882
|
-
maxAllowedTime
|
|
883
|
-
});
|
|
884
|
-
return;
|
|
885
|
-
}
|
|
886
|
-
this.log.trace(`Sync for slot ${slot} failed, checking for voting opportunities`, {
|
|
887
|
-
secondsIntoSlot,
|
|
888
|
-
maxAllowedTime
|
|
1175
|
+
// Vote-only actions do not give up the slot: if sync recovers, a later work-loop iteration can still build.
|
|
1176
|
+
// Under proposer pipelining the work loop reasons about the next L1 slot, so waiting for the target slot's
|
|
1177
|
+
// build-start deadline can miss the whole fallback window when the target advances with the clock.
|
|
1178
|
+
const nowSeconds = this.dateProvider.now() / 1000;
|
|
1179
|
+
const startDeadline = this.timetable.getBuildStartDeadline(targetSlot);
|
|
1180
|
+
this.log.trace(`Cannot build for slot ${slot}, checking for voting opportunities`, {
|
|
1181
|
+
nowSeconds,
|
|
1182
|
+
startDeadline
|
|
889
1183
|
});
|
|
890
1184
|
// Check if we're a proposer or proposal is open
|
|
891
|
-
const [canPropose, proposer] = await this.checkCanPropose(
|
|
1185
|
+
const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
|
|
892
1186
|
if (!canPropose) {
|
|
893
1187
|
this.log.trace(`Cannot vote in slot ${slot} since we are not a proposer`, {
|
|
894
1188
|
slot,
|
|
@@ -904,8 +1198,8 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
904
1198
|
attestorAddress,
|
|
905
1199
|
slot
|
|
906
1200
|
});
|
|
907
|
-
// Enqueue governance and slashing votes
|
|
908
|
-
const voter = new CheckpointVoter(
|
|
1201
|
+
// Enqueue governance and slashing votes (voter uses the target slot for L1 submission)
|
|
1202
|
+
const voter = new CheckpointVoter(targetSlot, publisher, attestorAddress, this.validatorClient, this.slasherClient, this.l1Constants, this.config, this.metrics, this.log);
|
|
909
1203
|
const votesPromises = voter.enqueueVotes();
|
|
910
1204
|
const votes = await Promise.all(votesPromises);
|
|
911
1205
|
if (votes.every((p)=>!p)) {
|
|
@@ -915,13 +1209,21 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
915
1209
|
this.log.info(`Voting in slot ${slot} despite sync failure`, {
|
|
916
1210
|
slot
|
|
917
1211
|
});
|
|
918
|
-
|
|
1212
|
+
// Votes are EIP-712-signed for `targetSlot` (the pipelined slot in which the multicall is
|
|
1213
|
+
// expected to mine). Delay submission to the start of `targetSlot` so the tx mines in the
|
|
1214
|
+
// slot the votes were signed for. We fire-and-forget so we don't block the sequencer's
|
|
1215
|
+
// work loop while waiting for the target slot to start.
|
|
1216
|
+
void publisher.sendRequestsAt(targetSlot).catch((err)=>{
|
|
1217
|
+
this.log.error(`Failed to publish votes despite sync failure for slot ${slot}`, err, {
|
|
1218
|
+
slot
|
|
1219
|
+
});
|
|
1220
|
+
});
|
|
919
1221
|
}
|
|
920
1222
|
/**
|
|
921
1223
|
* Tries to vote on slashing actions and governance proposals when escape hatch is open.
|
|
922
1224
|
* This allows the sequencer to participate in voting without performing checkpoint proposal work.
|
|
923
1225
|
*/ async tryVoteWhenEscapeHatchOpen(args) {
|
|
924
|
-
const { slot, proposer } = args;
|
|
1226
|
+
const { slot, targetSlot, proposer } = args;
|
|
925
1227
|
// Prevent duplicate attempts in the same slot
|
|
926
1228
|
if (this.lastSlotForFallbackVote === slot) {
|
|
927
1229
|
this.log.trace(`Already attempted to vote in slot ${slot} (escape hatch open, skipping)`);
|
|
@@ -932,9 +1234,14 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
932
1234
|
const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
|
|
933
1235
|
this.log.debug(`Escape hatch open for slot ${slot}, attempting vote-only actions`, {
|
|
934
1236
|
slot,
|
|
1237
|
+
targetSlot,
|
|
935
1238
|
attestorAddress
|
|
936
1239
|
});
|
|
937
|
-
|
|
1240
|
+
// Under proposer pipelining, the multicall is expected to mine in `targetSlot` (slot + 1).
|
|
1241
|
+
// Governance and slashing votes are EIP-712-signed against the slot they will mine in, and the
|
|
1242
|
+
// L1 contract checks `msg.sender == getCurrentProposer()` using the mining slot. So we must
|
|
1243
|
+
// sign for `targetSlot` and delay submission to the start of `targetSlot`.
|
|
1244
|
+
const voter = new CheckpointVoter(targetSlot, publisher, attestorAddress, this.validatorClient, this.slasherClient, this.l1Constants, this.config, this.metrics, this.log);
|
|
938
1245
|
const votesPromises = voter.enqueueVotes();
|
|
939
1246
|
const votes = await Promise.all(votesPromises);
|
|
940
1247
|
if (votes.every((p)=>!p)) {
|
|
@@ -942,28 +1249,50 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
942
1249
|
return;
|
|
943
1250
|
}
|
|
944
1251
|
this.log.info(`Voting in slot ${slot} (escape hatch open)`, {
|
|
945
|
-
slot
|
|
1252
|
+
slot,
|
|
1253
|
+
targetSlot
|
|
1254
|
+
});
|
|
1255
|
+
// Votes are EIP-712-signed for `targetSlot`. Delay submission to the start of `targetSlot` so
|
|
1256
|
+
// the multicall mines in the slot the votes were signed for; otherwise the L1 contract reads
|
|
1257
|
+
// `signaler = getCurrentProposer()` against the wrong slot and signature verification fails
|
|
1258
|
+
// silently inside Multicall3. Fire-and-forget so we don't block the sequencer's work loop while
|
|
1259
|
+
// waiting for the target slot to start, mirroring tryVoteWhenCannotBuild.
|
|
1260
|
+
void publisher.sendRequestsAt(targetSlot).catch((err)=>{
|
|
1261
|
+
this.log.error(`Failed to publish escape-hatch votes for slot ${slot}`, err, {
|
|
1262
|
+
slot,
|
|
1263
|
+
targetSlot
|
|
1264
|
+
});
|
|
946
1265
|
});
|
|
947
|
-
await publisher.sendRequests();
|
|
948
1266
|
}
|
|
949
1267
|
/**
|
|
950
1268
|
* Considers invalidating a block if the pending chain is invalid. Depends on how long the invalid block
|
|
951
1269
|
* has been there without being invalidated and whether the sequencer is in the committee or not. We always
|
|
952
1270
|
* have the proposer try to invalidate, but if they fail, the sequencers in the committee are expected to try,
|
|
953
1271
|
* and if they fail, any sequencer will try as well.
|
|
954
|
-
|
|
955
|
-
|
|
1272
|
+
* @param pendingChainValidationStatus - The archiver's pending-chain validation status, authoritative on its own.
|
|
1273
|
+
* @param currentSlot - The wall-clock slot, used for committee lookup, the per-(checkpoint, slot) dedup guard, and logging.
|
|
1274
|
+
*/ async considerInvalidatingCheckpoint(pendingChainValidationStatus, currentSlot) {
|
|
956
1275
|
if (pendingChainValidationStatus.valid) {
|
|
957
1276
|
return;
|
|
958
1277
|
}
|
|
959
1278
|
const invalidCheckpointNumber = pendingChainValidationStatus.checkpoint.checkpointNumber;
|
|
1279
|
+
// Avoid re-running the committee lookup, simulation, and submission on every tick within a slot.
|
|
1280
|
+
// The guard is keyed by (checkpoint, slot) — so a different invalid checkpoint surfacing later in
|
|
1281
|
+
// the same slot is not suppressed — and is set only after a request is successfully simulated below,
|
|
1282
|
+
// so a transient simulation failure (or thresholds not yet met) still retries on the next tick.
|
|
1283
|
+
if (this.lastInvalidationAttempt?.slot === currentSlot && this.lastInvalidationAttempt.checkpointNumber === invalidCheckpointNumber) {
|
|
1284
|
+
this.log.trace(`Already attempted to invalidate checkpoint ${invalidCheckpointNumber} in slot ${currentSlot}`, {
|
|
1285
|
+
currentSlot,
|
|
1286
|
+
invalidCheckpointNumber
|
|
1287
|
+
});
|
|
1288
|
+
return;
|
|
1289
|
+
}
|
|
960
1290
|
const invalidCheckpointTimestamp = pendingChainValidationStatus.checkpoint.timestamp;
|
|
961
1291
|
const timeSinceChainInvalid = this.dateProvider.nowInSeconds() - Number(invalidCheckpointTimestamp);
|
|
962
1292
|
const ourValidatorAddresses = this.validatorClient.getValidatorAddresses();
|
|
963
1293
|
const { secondsBeforeInvalidatingBlockAsCommitteeMember, secondsBeforeInvalidatingBlockAsNonCommitteeMember } = this.config;
|
|
964
1294
|
const logData = {
|
|
965
1295
|
invalidL1Timestamp: invalidCheckpointTimestamp,
|
|
966
|
-
syncedL2Slot,
|
|
967
1296
|
invalidCheckpoint: pendingChainValidationStatus.checkpoint,
|
|
968
1297
|
secondsBeforeInvalidatingBlockAsCommitteeMember,
|
|
969
1298
|
secondsBeforeInvalidatingBlockAsNonCommitteeMember,
|
|
@@ -997,6 +1326,12 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
997
1326
|
this.log.warn(`Failed to simulate invalidate checkpoint`, logData);
|
|
998
1327
|
return;
|
|
999
1328
|
}
|
|
1329
|
+
// We produced a valid invalidation request; record it so further ticks within this slot skip the
|
|
1330
|
+
// committee lookup, simulation, and submission above for this same invalid checkpoint.
|
|
1331
|
+
this.lastInvalidationAttempt = {
|
|
1332
|
+
slot: currentSlot,
|
|
1333
|
+
checkpointNumber: invalidCheckpointNumber
|
|
1334
|
+
};
|
|
1000
1335
|
this.log.info(invalidateAsCommitteeMember ? `Invalidating checkpoint ${invalidCheckpointNumber} as committee member` : `Invalidating checkpoint ${invalidCheckpointNumber} as non-committee member`, logData);
|
|
1001
1336
|
publisher.enqueueInvalidateCheckpoint(invalidateCheckpoint);
|
|
1002
1337
|
if (!this.config.fishermanMode) {
|
|
@@ -1032,12 +1367,12 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
1032
1367
|
}))
|
|
1033
1368
|
});
|
|
1034
1369
|
}
|
|
1035
|
-
|
|
1036
|
-
|
|
1037
|
-
|
|
1038
|
-
|
|
1039
|
-
const
|
|
1040
|
-
return Number((this.dateProvider.now() / 1000 -
|
|
1370
|
+
/**
|
|
1371
|
+
* Wall-clock seconds elapsed since the build-frame start of the given target slot
|
|
1372
|
+
* (`now − getBuildFrameStart(targetSlot)`). May be negative if called before the build frame opens.
|
|
1373
|
+
*/ getSecondsIntoBuildFrame(targetSlot) {
|
|
1374
|
+
const buildFrameStart = this.timetable.getBuildFrameStart(targetSlot);
|
|
1375
|
+
return Number((this.dateProvider.now() / 1000 - buildFrameStart).toFixed(3));
|
|
1041
1376
|
}
|
|
1042
1377
|
get aztecSlotDuration() {
|
|
1043
1378
|
return this.l1Constants.slotDuration;
|
|
@@ -1060,7 +1395,4 @@ _dec = trackSpan('Sequencer.work'), _dec1 = trackSpan('Sequencer.prepareCheckpoi
|
|
|
1060
1395
|
getConfig() {
|
|
1061
1396
|
return this.config;
|
|
1062
1397
|
}
|
|
1063
|
-
get l1PublishingTime() {
|
|
1064
|
-
return this.config.l1PublishingTime ?? this.l1Constants.ethereumSlotDuration;
|
|
1065
|
-
}
|
|
1066
1398
|
}
|