@aztec/sequencer-client 0.0.1-commit.c2eed6949 → 0.0.1-commit.c52d6e7
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 +12 -4
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +61 -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 +6 -25
- package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
- package/dest/global_variable_builder/global_builder.js +7 -65
- 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 +101 -69
- package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher.js +635 -538
- 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 +696 -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 +71 -21
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_proposal_job.js +733 -213
- package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
- package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
- package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
- package/dest/sequencer/checkpoint_voter.d.ts +1 -2
- package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_voter.js +2 -5
- package/dest/sequencer/events.d.ts +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/missing_committee.d.ts +72 -0
- package/dest/sequencer/missing_committee.d.ts.map +1 -0
- package/dest/sequencer/missing_committee.js +139 -0
- 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 +158 -35
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +556 -162
- 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 +29 -28
- package/src/client/sequencer-client.ts +37 -27
- package/src/config.ts +71 -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 +12 -80
- 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 +735 -612
- 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 +800 -0
- package/src/sequencer/automine/index.ts +6 -0
- package/src/sequencer/checkpoint_proposal_job.ts +849 -242
- package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
- package/src/sequencer/checkpoint_voter.ts +1 -12
- package/src/sequencer/events.ts +66 -5
- package/src/sequencer/metrics.ts +43 -24
- package/src/sequencer/missing_committee.ts +192 -0
- package/src/sequencer/requests_tracker.ts +43 -0
- package/src/sequencer/sequencer.ts +639 -180
- 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
|
@@ -435,18 +435,20 @@ function applyDecs2203RFactory() {
|
|
|
435
435
|
function _apply_decs_2203_r(targetClass, memberDecs, classDecs, parentClass) {
|
|
436
436
|
return (_apply_decs_2203_r = applyDecs2203RFactory())(targetClass, memberDecs, classDecs, parentClass);
|
|
437
437
|
}
|
|
438
|
-
var _dec, _dec1, _dec2, _dec3, _dec4, _dec5, _dec6,
|
|
439
|
-
import {
|
|
438
|
+
var _dec, _dec1, _dec2, _dec3, _dec4, _dec5, _dec6, _initProto;
|
|
439
|
+
import { PROPOSER_PIPELINING_SLOT_OFFSET } from '@aztec/epoch-cache';
|
|
440
|
+
import { BlockNumber, CheckpointNumber, IndexWithinCheckpoint, SlotNumber } from '@aztec/foundation/branded-types';
|
|
440
441
|
import { randomInt } from '@aztec/foundation/crypto/random';
|
|
441
442
|
import { flipSignature, generateRecoverableSignature, generateUnrecoverableSignature } from '@aztec/foundation/crypto/secp256k1-signer';
|
|
443
|
+
import { InterruptError, TimeoutError } from '@aztec/foundation/error';
|
|
442
444
|
import { filter } from '@aztec/foundation/iterator';
|
|
443
445
|
import { createLogger } from '@aztec/foundation/log';
|
|
444
|
-
import {
|
|
446
|
+
import { InterruptibleSleep } from '@aztec/foundation/sleep';
|
|
445
447
|
import { Timer } from '@aztec/foundation/timer';
|
|
446
448
|
import { isErrorClass, unfreeze } from '@aztec/foundation/types';
|
|
447
|
-
import { CommitteeAttestationsAndSigners, MaliciousCommitteeAttestationsAndSigners } from '@aztec/stdlib/block';
|
|
448
|
-
import { validateCheckpoint } from '@aztec/stdlib/checkpoint';
|
|
449
|
-
import {
|
|
449
|
+
import { CommitteeAttestationsAndSigners, MaliciousCommitteeAttestationsAndSigners, MaliciousYParityCommitteeAttestationsAndSigners } from '@aztec/stdlib/block';
|
|
450
|
+
import { buildCheckpointSimulationOverridesPlan, getPreviousCheckpointOutHashes, validateCheckpoint } from '@aztec/stdlib/checkpoint';
|
|
451
|
+
import { computeQuorum, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
|
|
450
452
|
import { Gas } from '@aztec/stdlib/gas';
|
|
451
453
|
import { InsufficientValidTxsError } from '@aztec/stdlib/interfaces/server';
|
|
452
454
|
import { computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
|
|
@@ -458,25 +460,25 @@ import { CheckpointVoter } from './checkpoint_voter.js';
|
|
|
458
460
|
import { SequencerInterruptedError } from './errors.js';
|
|
459
461
|
import { SequencerState } from './utils.js';
|
|
460
462
|
/** How much time to sleep while waiting for min transactions to accumulate for a block */ const TXS_POLLING_MS = 500;
|
|
463
|
+
const ARCHIVER_SYNC_POLLING_MS = 200;
|
|
461
464
|
_dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('CheckpointProposalJob.proposeCheckpoint', function() {
|
|
462
465
|
return {
|
|
463
466
|
// nullish operator needed for tests
|
|
464
467
|
[Attributes.COINBASE]: this.validatorClient.getCoinbaseForAttestor(this.attestorAddress)?.toString(),
|
|
465
468
|
[Attributes.SLOT_NUMBER]: this.targetSlot
|
|
466
469
|
};
|
|
467
|
-
}), _dec2 = trackSpan('CheckpointProposalJob.buildBlocksForCheckpoint'), _dec3 = trackSpan('CheckpointProposalJob.waitUntilNextSubslot'), _dec4 = trackSpan('CheckpointProposalJob.buildSingleBlock'), _dec5 = trackSpan('CheckpointProposalJob.waitForMinTxs'), _dec6 = trackSpan('CheckpointProposalJob.waitForAttestations')
|
|
470
|
+
}), _dec2 = trackSpan('CheckpointProposalJob.buildBlocksForCheckpoint'), _dec3 = trackSpan('CheckpointProposalJob.waitUntilNextSubslot'), _dec4 = trackSpan('CheckpointProposalJob.buildSingleBlock'), _dec5 = trackSpan('CheckpointProposalJob.waitForMinTxs'), _dec6 = trackSpan('CheckpointProposalJob.waitForAttestations');
|
|
468
471
|
/**
|
|
469
472
|
* Handles the execution of a checkpoint proposal after the initial preparation phase.
|
|
470
473
|
* This includes building blocks, collecting attestations, and publishing the checkpoint to L1,
|
|
471
474
|
* as well as enqueueing votes for slashing and governance proposals. This class is created from
|
|
472
475
|
* the Sequencer once the check for being the proposer for the slot has succeeded.
|
|
473
476
|
*/ export class CheckpointProposalJob {
|
|
474
|
-
slotNow;
|
|
475
477
|
targetSlot;
|
|
476
|
-
epochNow;
|
|
477
478
|
targetEpoch;
|
|
478
479
|
checkpointNumber;
|
|
479
480
|
syncedToBlockNumber;
|
|
481
|
+
checkpointedCheckpointNumber;
|
|
480
482
|
proposer;
|
|
481
483
|
publisher;
|
|
482
484
|
attestorAddress;
|
|
@@ -490,15 +492,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
490
492
|
checkpointsBuilder;
|
|
491
493
|
blockSink;
|
|
492
494
|
l1Constants;
|
|
495
|
+
signatureContext;
|
|
493
496
|
config;
|
|
494
497
|
timetable;
|
|
495
498
|
slasherClient;
|
|
496
499
|
epochCache;
|
|
497
500
|
dateProvider;
|
|
498
501
|
metrics;
|
|
502
|
+
checkpointMetrics;
|
|
499
503
|
eventEmitter;
|
|
504
|
+
pendingRequests;
|
|
500
505
|
setStateFn;
|
|
501
506
|
tracer;
|
|
507
|
+
proposedCheckpointData;
|
|
502
508
|
static{
|
|
503
509
|
({ e: [_initProto] } = _apply_decs_2203_r(this, [
|
|
504
510
|
[
|
|
@@ -535,23 +541,31 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
535
541
|
_dec6,
|
|
536
542
|
2,
|
|
537
543
|
"waitForAttestations"
|
|
538
|
-
],
|
|
539
|
-
[
|
|
540
|
-
_dec7,
|
|
541
|
-
2,
|
|
542
|
-
"waitUntilTimeInSlot"
|
|
543
544
|
]
|
|
544
545
|
], []));
|
|
545
546
|
}
|
|
546
547
|
log;
|
|
547
|
-
|
|
548
|
-
|
|
549
|
-
|
|
548
|
+
checkpointEventLog;
|
|
549
|
+
interruptibleSleep;
|
|
550
|
+
interrupted;
|
|
551
|
+
/**
|
|
552
|
+
* Chain state overrides built once per slot in proposeCheckpoint after the checkpoint is
|
|
553
|
+
* complete. Carries the pending parent override (archive + slot + fee header) for pipelining,
|
|
554
|
+
* or the invalidation pending override when rolling back. Consumed by
|
|
555
|
+
* publisher.validateBlockHeader before broadcast.
|
|
556
|
+
*/ checkpointSimulationOverridesPlan;
|
|
557
|
+
getSignatureContext() {
|
|
558
|
+
return this.signatureContext;
|
|
559
|
+
}
|
|
560
|
+
constructor(targetSlot, targetEpoch, checkpointNumber, syncedToBlockNumber, checkpointedCheckpointNumber, // TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
|
|
561
|
+
proposer, publisher, attestorAddress, invalidateCheckpoint, validatorClient, globalsBuilder, p2pClient, worldState, l1ToL2MessageSource, l2BlockSource, checkpointsBuilder, blockSink, l1Constants, signatureContext, config, timetable, slasherClient, epochCache, dateProvider, metrics, checkpointMetrics, eventEmitter, // Shared with the owning sequencer, which drains it during shutdown; the fire-and-forget L1
|
|
562
|
+
// submission this job backgrounds is tracked here rather than in a job-local tracker.
|
|
563
|
+
pendingRequests, setStateFn, tracer, bindings, proposedCheckpointData){
|
|
550
564
|
this.targetSlot = targetSlot;
|
|
551
|
-
this.epochNow = epochNow;
|
|
552
565
|
this.targetEpoch = targetEpoch;
|
|
553
566
|
this.checkpointNumber = checkpointNumber;
|
|
554
567
|
this.syncedToBlockNumber = syncedToBlockNumber;
|
|
568
|
+
this.checkpointedCheckpointNumber = checkpointedCheckpointNumber;
|
|
555
569
|
this.proposer = proposer;
|
|
556
570
|
this.publisher = publisher;
|
|
557
571
|
this.attestorAddress = attestorAddress;
|
|
@@ -565,82 +579,351 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
565
579
|
this.checkpointsBuilder = checkpointsBuilder;
|
|
566
580
|
this.blockSink = blockSink;
|
|
567
581
|
this.l1Constants = l1Constants;
|
|
582
|
+
this.signatureContext = signatureContext;
|
|
568
583
|
this.config = config;
|
|
569
584
|
this.timetable = timetable;
|
|
570
585
|
this.slasherClient = slasherClient;
|
|
571
586
|
this.epochCache = epochCache;
|
|
572
587
|
this.dateProvider = dateProvider;
|
|
573
588
|
this.metrics = metrics;
|
|
589
|
+
this.checkpointMetrics = checkpointMetrics;
|
|
574
590
|
this.eventEmitter = eventEmitter;
|
|
591
|
+
this.pendingRequests = pendingRequests;
|
|
575
592
|
this.setStateFn = setStateFn;
|
|
576
593
|
this.tracer = tracer;
|
|
577
|
-
|
|
594
|
+
this.proposedCheckpointData = proposedCheckpointData;
|
|
595
|
+
this.interruptibleSleep = (_initProto(this), new InterruptibleSleep());
|
|
596
|
+
this.interrupted = false;
|
|
578
597
|
this.log = createLogger('sequencer:checkpoint-proposal', {
|
|
579
598
|
...bindings,
|
|
580
|
-
instanceId: `slot-${this.
|
|
599
|
+
instanceId: `slot-${this.getBuildSlot()}`
|
|
600
|
+
});
|
|
601
|
+
this.checkpointEventLog = createLogger('sequencer:checkpoint-events', {
|
|
602
|
+
...bindings,
|
|
603
|
+
instanceId: `slot-${this.getBuildSlot()}`
|
|
581
604
|
});
|
|
582
605
|
}
|
|
583
|
-
/**
|
|
584
|
-
|
|
606
|
+
/**
|
|
607
|
+
* The wall-clock slot during which this job builds, i.e. the slot one before {@link targetSlot} under
|
|
608
|
+
* proposer pipelining. Also the slot of the parent checkpoint this job builds on top of.
|
|
609
|
+
*/ getBuildSlot() {
|
|
610
|
+
return SlotNumber(this.targetSlot - PROPOSER_PIPELINING_SLOT_OFFSET);
|
|
611
|
+
}
|
|
612
|
+
/**
|
|
613
|
+
* Sets the sequencer state for this job, reporting the target slot the checkpoint is being proposed for
|
|
614
|
+
* (not the wall-clock build slot). The slot is informational on the event payload/metrics; the job knows
|
|
615
|
+
* its own target slot, so callers only pass the state.
|
|
616
|
+
*/ setState(state) {
|
|
617
|
+
this.setStateFn(state, this.targetSlot);
|
|
585
618
|
}
|
|
586
|
-
/**
|
|
587
|
-
|
|
619
|
+
/** Interrupts job-owned waits, including the publisher's send-at-slot sleep, so shutdown can finish. */ interrupt() {
|
|
620
|
+
this.interrupted = true;
|
|
621
|
+
this.interruptibleSleep.interrupt(true);
|
|
622
|
+
this.publisher.interrupt();
|
|
623
|
+
}
|
|
624
|
+
async awaitInterruptibleSleep(ms) {
|
|
625
|
+
if (this.interrupted) {
|
|
626
|
+
throw new SequencerInterruptedError();
|
|
627
|
+
}
|
|
628
|
+
if (ms <= 0) {
|
|
629
|
+
return;
|
|
630
|
+
}
|
|
631
|
+
try {
|
|
632
|
+
await this.interruptibleSleep.sleep(ms);
|
|
633
|
+
} catch (err) {
|
|
634
|
+
if (err instanceof InterruptError) {
|
|
635
|
+
throw new SequencerInterruptedError();
|
|
636
|
+
}
|
|
637
|
+
throw err;
|
|
638
|
+
}
|
|
639
|
+
}
|
|
640
|
+
logCheckpointEvent(eventName, message, fields) {
|
|
641
|
+
this.checkpointEventLog.debug(message, {
|
|
642
|
+
eventName: `sequencer-checkpoint-${eventName}`,
|
|
643
|
+
...fields
|
|
644
|
+
});
|
|
588
645
|
}
|
|
589
646
|
/**
|
|
590
647
|
* Executes the checkpoint proposal job.
|
|
591
|
-
*
|
|
648
|
+
* Builds blocks, assembles checkpoint, and broadcasts the proposal (blocking).
|
|
649
|
+
* Attestation collection, signing, and L1 submission are backgrounded so the
|
|
650
|
+
* work loop can return to IDLE immediately for consecutive slot proposals.
|
|
651
|
+
* Returns the built checkpoint if successful, undefined otherwise.
|
|
592
652
|
*/ async execute() {
|
|
593
653
|
// Enqueue governance and slashing votes (returns promises that will be awaited later)
|
|
594
654
|
// In fisherman mode, we simulate slashing but don't actually publish to L1
|
|
595
655
|
// These are constant for the whole slot, so we only enqueue them once
|
|
596
656
|
const votesPromises = new CheckpointVoter(this.targetSlot, this.publisher, this.attestorAddress, this.validatorClient, this.slasherClient, this.l1Constants, this.config, this.metrics, this.log).enqueueVotes();
|
|
597
|
-
// Build
|
|
598
|
-
|
|
599
|
-
|
|
600
|
-
|
|
601
|
-
|
|
602
|
-
|
|
657
|
+
// Build blocks, assemble checkpoint, and broadcast proposal (BLOCKING).
|
|
658
|
+
// Returns after broadcast — attestation collection is deferred.
|
|
659
|
+
const broadcast = await this.proposeCheckpoint();
|
|
660
|
+
if (!broadcast) {
|
|
661
|
+
await Promise.all(votesPromises);
|
|
662
|
+
// Still submit votes even without a checkpoint.
|
|
663
|
+
// Under proposer pipelining, vote-offenses signatures are EIP-712-bound to `targetSlot`
|
|
664
|
+
// (the pipelined slot in which the multicall is expected to mine). Submitting at the
|
|
665
|
+
// wall-clock time would let the multicall mine in a different L2 slot, causing
|
|
666
|
+
// signature verification to fail silently inside Multicall3. Delay submission to the
|
|
667
|
+
// start of `targetSlot` so the tx mines in the slot the vote was signed for.
|
|
668
|
+
if (!this.config.fishermanMode) {
|
|
669
|
+
this.pendingRequests.trackRequest(this.publisher.sendRequestsAt(this.targetSlot), ()=>this.interrupt());
|
|
670
|
+
}
|
|
671
|
+
return undefined;
|
|
603
672
|
}
|
|
673
|
+
const { checkpoint } = broadcast;
|
|
674
|
+
this.metrics.recordCheckpointProposalSuccess();
|
|
604
675
|
// Do not post anything to L1 if we are fishermen, but do perform L1 fee analysis
|
|
605
676
|
if (this.config.fishermanMode) {
|
|
606
677
|
await this.handleCheckpointEndAsFisherman(checkpoint);
|
|
607
|
-
return;
|
|
678
|
+
return checkpoint;
|
|
608
679
|
}
|
|
609
|
-
//
|
|
610
|
-
|
|
611
|
-
|
|
612
|
-
|
|
613
|
-
|
|
614
|
-
|
|
615
|
-
|
|
680
|
+
// Background the attestation → signing → L1 pipeline so the work loop is unblocked
|
|
681
|
+
this.pendingRequests.trackRequest(this.waitForAttestationsAndEnqueueSubmissionAsync(broadcast, votesPromises), ()=>this.interrupt());
|
|
682
|
+
// Return the built checkpoint immediately — the work loop is now unblocked
|
|
683
|
+
return checkpoint;
|
|
684
|
+
}
|
|
685
|
+
/**
|
|
686
|
+
* Background pipeline: collects attestations, signs them, enqueues the checkpoint, and submits to L1.
|
|
687
|
+
* Runs as a fire-and-forget task tracked in the sequencer's shared tracker so the work loop is unblocked.
|
|
688
|
+
*/ async waitForAttestationsAndEnqueueSubmissionAsync(broadcast, votesPromises) {
|
|
689
|
+
const { checkpoint } = broadcast;
|
|
690
|
+
try {
|
|
691
|
+
// Wait for all votes actions, enqueued at the beginning, to resolve
|
|
692
|
+
await Promise.all(votesPromises);
|
|
693
|
+
// Try to collect attestations from the committee
|
|
694
|
+
const signedAttestations = await this.getSignedCommitteeAttestations(broadcast);
|
|
695
|
+
// Wait for the previous checkpoint to land on L1 before submitting, so we can check it
|
|
696
|
+
// matches the proposed checkpoint we used as parent, and has valid attestations.
|
|
697
|
+
if (signedAttestations && await this.waitForValidParentCheckpointOnL1()) {
|
|
698
|
+
await this.enqueueCheckpointForSubmission({
|
|
699
|
+
checkpoint,
|
|
700
|
+
...signedAttestations
|
|
701
|
+
});
|
|
702
|
+
}
|
|
703
|
+
// If we failed to collect attestations, at least check if we need to issue an invalidation
|
|
704
|
+
if (!signedAttestations && await this.waitForSyncedL2SlotNumber(this.getBuildSlot())) {
|
|
705
|
+
const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
|
|
706
|
+
if (!validationStatus.valid) {
|
|
707
|
+
this.log.warn(`Checkpoint ${validationStatus.checkpoint.checkpointNumber} has invalid attestations, enqueuing invalidation in spite of attestation collection failure`, {
|
|
708
|
+
checkpoint: validationStatus.checkpoint,
|
|
709
|
+
reason: validationStatus.reason
|
|
710
|
+
});
|
|
711
|
+
await this.enqueueInvalidation(validationStatus);
|
|
712
|
+
}
|
|
713
|
+
}
|
|
714
|
+
// Send whatever was enqueued: votes + (propose | invalidation | nothing).
|
|
715
|
+
const l1Response = await this.publisher.sendRequestsAt(this.targetSlot);
|
|
716
|
+
const proposedAction = l1Response?.successfulActions.find((a)=>a === 'propose');
|
|
717
|
+
if (proposedAction) {
|
|
718
|
+
this.logCheckpointEvent('published', `Checkpoint published for slot ${this.targetSlot}`, {
|
|
719
|
+
slot: this.targetSlot,
|
|
720
|
+
checkpointNumber: this.checkpointNumber,
|
|
721
|
+
successfulActions: l1Response?.successfulActions,
|
|
722
|
+
sentActions: l1Response?.sentActions
|
|
723
|
+
});
|
|
724
|
+
this.eventEmitter.emit('checkpoint-published', {
|
|
725
|
+
checkpoint: this.checkpointNumber,
|
|
726
|
+
slot: this.targetSlot
|
|
727
|
+
});
|
|
728
|
+
const coinbase = checkpoint.header.coinbase;
|
|
729
|
+
await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
|
|
730
|
+
} else {
|
|
731
|
+
this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
|
|
732
|
+
slot: this.targetSlot,
|
|
733
|
+
checkpointNumber: this.checkpointNumber,
|
|
734
|
+
successfulActions: l1Response?.successfulActions,
|
|
735
|
+
failedActions: l1Response?.failedActions,
|
|
736
|
+
sentActions: l1Response?.sentActions,
|
|
737
|
+
expiredActions: l1Response?.expiredActions,
|
|
738
|
+
reason: 'propose_action_not_successful'
|
|
739
|
+
});
|
|
740
|
+
this.log.warn(`Checkpoint publish failed for slot ${this.targetSlot}`, {
|
|
741
|
+
slot: this.targetSlot,
|
|
742
|
+
checkpointNumber: this.checkpointNumber,
|
|
743
|
+
successfulActions: l1Response?.successfulActions,
|
|
744
|
+
failedActions: l1Response?.failedActions,
|
|
745
|
+
sentActions: l1Response?.sentActions,
|
|
746
|
+
expiredActions: l1Response?.expiredActions,
|
|
747
|
+
reason: 'propose_action_not_successful'
|
|
748
|
+
});
|
|
749
|
+
this.eventEmitter.emit('checkpoint-publish-failed', {
|
|
750
|
+
...l1Response,
|
|
751
|
+
slot: this.targetSlot
|
|
752
|
+
});
|
|
753
|
+
this.metrics.recordPipelineDiscard();
|
|
754
|
+
}
|
|
755
|
+
} catch (err) {
|
|
756
|
+
if (err instanceof SequencerInterruptedError) {
|
|
757
|
+
return;
|
|
758
|
+
}
|
|
759
|
+
this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
|
|
760
|
+
slot: this.targetSlot,
|
|
761
|
+
checkpointNumber: this.checkpointNumber,
|
|
762
|
+
reason: err instanceof Error ? err.message : String(err)
|
|
616
763
|
});
|
|
617
|
-
|
|
618
|
-
|
|
619
|
-
|
|
620
|
-
|
|
621
|
-
|
|
622
|
-
|
|
623
|
-
|
|
624
|
-
|
|
764
|
+
this.log.error(`Background attestation/L1 pipeline failed for slot ${this.targetSlot}`, err, {
|
|
765
|
+
slot: this.targetSlot,
|
|
766
|
+
checkpointNumber: this.checkpointNumber,
|
|
767
|
+
reason: err instanceof Error ? err.message : String(err)
|
|
768
|
+
});
|
|
769
|
+
this.eventEmitter.emit('checkpoint-publish-failed', {
|
|
770
|
+
slot: this.targetSlot
|
|
771
|
+
});
|
|
772
|
+
this.metrics.recordPipelineDiscard();
|
|
773
|
+
}
|
|
774
|
+
}
|
|
775
|
+
/** Enqueues the checkpoint for L1 submission. Called after pipeline sleep in execute(). */ async enqueueCheckpointForSubmission(result) {
|
|
776
|
+
const { checkpoint, attestations, attestationsSignature } = result;
|
|
777
|
+
this.setState(SequencerState.PUBLISHING_CHECKPOINT);
|
|
778
|
+
// Latest L1 block the propose can still land in for the target slot: the last Ethereum block inside
|
|
779
|
+
// the target slot (`target_slot_start + S - E`). This is one ethereum slot later than
|
|
780
|
+
// `attestation_deadline` (= last_ethereum_block_in_target_slot - E), which bounds when validators must
|
|
781
|
+
// have signed, not when the proposer must have sent. Using the attestation deadline here is too tight:
|
|
782
|
+
// attestations are collected up to (and, when not enforcing, past) it, so the propose tx would be
|
|
783
|
+
// enqueued already expired and time out before it can mine.
|
|
784
|
+
const lastL1BlockInTargetSlot = Number(getTimestampForSlot(this.targetSlot, this.l1Constants)) + this.l1Constants.slotDuration - this.l1Constants.ethereumSlotDuration;
|
|
785
|
+
const txTimeoutAt = new Date(lastL1BlockInTargetSlot * 1000);
|
|
786
|
+
// If we have been configured to potentially skip publishing checkpoint then roll the dice here
|
|
787
|
+
if (this.config.skipPublishingCheckpointsPercent !== undefined && this.config.skipPublishingCheckpointsPercent > 0) {
|
|
788
|
+
const roll = Math.max(0, randomInt(100));
|
|
789
|
+
if (roll < this.config.skipPublishingCheckpointsPercent) {
|
|
790
|
+
this.log.warn(`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${roll}`);
|
|
791
|
+
return;
|
|
625
792
|
}
|
|
626
793
|
}
|
|
627
|
-
|
|
628
|
-
|
|
629
|
-
|
|
630
|
-
|
|
631
|
-
|
|
632
|
-
|
|
633
|
-
|
|
794
|
+
await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
|
|
795
|
+
txTimeoutAt
|
|
796
|
+
});
|
|
797
|
+
}
|
|
798
|
+
/**
|
|
799
|
+
* Wait until the archiver syncs past the given L2 slot number.
|
|
800
|
+
* The deadline is the end of `this.targetSlot`, beyond which any pipelined work would miss its
|
|
801
|
+
* L1 submission window and is no longer useful.
|
|
802
|
+
*/ async waitForSyncedL2SlotNumber(waitForSlot) {
|
|
803
|
+
const targetSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
|
|
804
|
+
const targetSlotEndMs = (targetSlotStart + this.l1Constants.slotDuration) * 1000;
|
|
805
|
+
const syncDelayTolerance = this.l1Constants.ethereumSlotDuration * 2 * 1000;
|
|
806
|
+
const timeoutSeconds = Math.max(0.1, (targetSlotEndMs + syncDelayTolerance - this.dateProvider.now()) / 1000);
|
|
807
|
+
try {
|
|
808
|
+
const timer = new Timer();
|
|
809
|
+
while(true){
|
|
810
|
+
const syncedSlot = await this.l2BlockSource.getSyncedL2SlotNumber();
|
|
811
|
+
if (syncedSlot !== undefined && syncedSlot >= waitForSlot) {
|
|
812
|
+
return true;
|
|
813
|
+
}
|
|
814
|
+
if (timeoutSeconds && timer.s() > timeoutSeconds) {
|
|
815
|
+
throw new TimeoutError(`Timeout awaiting archiver sync past slot ${waitForSlot}`);
|
|
816
|
+
}
|
|
817
|
+
await this.awaitInterruptibleSleep(ARCHIVER_SYNC_POLLING_MS);
|
|
818
|
+
}
|
|
819
|
+
} catch (err) {
|
|
820
|
+
if (err instanceof SequencerInterruptedError) {
|
|
821
|
+
throw err;
|
|
822
|
+
}
|
|
823
|
+
this.log.warn(`Archiver did not sync L1 past slot ${waitForSlot} before slot ${this.targetSlot} expired, discarding pipelined work`, {
|
|
824
|
+
checkpointNumber: this.checkpointNumber
|
|
634
825
|
});
|
|
635
|
-
|
|
636
|
-
|
|
637
|
-
|
|
638
|
-
|
|
639
|
-
|
|
640
|
-
|
|
641
|
-
|
|
826
|
+
this.emitPipelinedCheckpointDiscarded('archiver-sync-timeout');
|
|
827
|
+
return false;
|
|
828
|
+
}
|
|
829
|
+
}
|
|
830
|
+
/**
|
|
831
|
+
* Waits for the parent checkpoint to land on L1 before submitting a pipelined checkpoint.
|
|
832
|
+
* Polls until the archiver has synced L1 past the parent's slot, then verifies:
|
|
833
|
+
* - If we built on a proposed parent: it must have landed on L1 with matching hash and valid attestations.
|
|
834
|
+
* - If we built without a proposed parent: no new checkpoint must have appeared for that slot.
|
|
835
|
+
* If the parent has invalid attestations, enqueues an invalidation. Returns whether to proceed with the proposal.
|
|
836
|
+
*/ async waitForValidParentCheckpointOnL1() {
|
|
837
|
+
if (this.config.skipWaitForValidParentCheckpointOnL1) {
|
|
838
|
+
this.log.warn(`Skipping waitForValidParentCheckpointOnL1 due to test configuration`, {
|
|
839
|
+
checkpointNumber: this.checkpointNumber
|
|
840
|
+
});
|
|
841
|
+
return true;
|
|
842
|
+
}
|
|
843
|
+
const parentCheckpointNumber = CheckpointNumber(this.checkpointNumber - 1);
|
|
844
|
+
// Wait until archiver has synced L1 past the parent's slot (the build slot, one before targetSlot)
|
|
845
|
+
if (!await this.waitForSyncedL2SlotNumber(this.getBuildSlot())) {
|
|
846
|
+
return false;
|
|
847
|
+
}
|
|
848
|
+
const tips = await this.l2BlockSource.getL2Tips();
|
|
849
|
+
const checkpointedNumber = tips.checkpointed.checkpoint.number;
|
|
850
|
+
// We built on top of a proposed checkpoint. Verify it landed on L1 as expected.
|
|
851
|
+
if (this.proposedCheckpointData) {
|
|
852
|
+
// After syncing from L1 we see the chain tip has invalid attestations. This means the parent checkpoint was posted
|
|
853
|
+
// with invalid attestations, or it built on top of something with invalid attestations and didnt invalidate them.
|
|
854
|
+
// Either way, we thought our parent would be valid, so we have to throw away our work. But at least we'll try and
|
|
855
|
+
// invalidate on L1 so we clean up the chain for the next proposer. And we'll slash them, but that's handled elsewhere.
|
|
856
|
+
const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
|
|
857
|
+
if (!validationStatus.valid) {
|
|
858
|
+
this.log.warn(`Parent checkpoint ${parentCheckpointNumber} has invalid attestations, discarding pipelined work`, {
|
|
859
|
+
checkpointNumber: this.checkpointNumber,
|
|
860
|
+
reason: validationStatus.reason
|
|
861
|
+
});
|
|
862
|
+
this.emitPipelinedCheckpointDiscarded('parent-invalid-attestations');
|
|
863
|
+
await this.enqueueInvalidation(validationStatus);
|
|
864
|
+
return false;
|
|
865
|
+
}
|
|
866
|
+
// The pending chain is valid. But did the parent checkpoint land on L1 at all?
|
|
867
|
+
if (checkpointedNumber < parentCheckpointNumber) {
|
|
868
|
+
this.log.warn(`Parent checkpoint ${parentCheckpointNumber} did not land on L1, discarding pipelined work`, {
|
|
869
|
+
checkpointNumber: this.checkpointNumber,
|
|
870
|
+
checkpointedNumber
|
|
871
|
+
});
|
|
872
|
+
this.emitPipelinedCheckpointDiscarded('parent-not-on-l1');
|
|
873
|
+
return false;
|
|
874
|
+
}
|
|
875
|
+
// It landed. But is it the one we were expecting?
|
|
876
|
+
const expectedHash = this.proposedCheckpointData.header.hash().toString();
|
|
877
|
+
if (tips.checkpointed.checkpoint.hash !== expectedHash) {
|
|
878
|
+
this.log.warn(`Parent checkpoint ${parentCheckpointNumber} hash mismatch on L1, discarding pipelined work`, {
|
|
879
|
+
checkpointNumber: this.checkpointNumber,
|
|
880
|
+
expectedHash,
|
|
881
|
+
actualHash: tips.checkpointed.checkpoint.hash
|
|
882
|
+
});
|
|
883
|
+
this.emitPipelinedCheckpointDiscarded('parent-hash-mismatch');
|
|
884
|
+
return false;
|
|
885
|
+
}
|
|
886
|
+
return true;
|
|
887
|
+
} else {
|
|
888
|
+
// We didn't see a proposed checkpoint at build time, so we built on checkpointed parent from two slots ago.
|
|
889
|
+
// But if a new checkpoint for the previous slot appeared on L1 in the meantime, our checkpoint assumed the wrong parent,
|
|
890
|
+
// so we have to discard our work. This can happen if we're somehow cut off from p2p and fail to see the checkpoint
|
|
891
|
+
// proposal for the previous slot.
|
|
892
|
+
if (checkpointedNumber > parentCheckpointNumber) {
|
|
893
|
+
this.log.warn(`Unexpected checkpoint ${checkpointedNumber} landed on L1 after we built on top of parent ${parentCheckpointNumber}, discarding pipelined work`, {
|
|
894
|
+
checkpointNumber: this.checkpointNumber,
|
|
895
|
+
checkpointedNumber
|
|
896
|
+
});
|
|
897
|
+
this.emitPipelinedCheckpointDiscarded('unexpected-parent-appeared');
|
|
898
|
+
return false;
|
|
899
|
+
}
|
|
900
|
+
return true;
|
|
901
|
+
}
|
|
902
|
+
}
|
|
903
|
+
/** Emits the pipelined-checkpoint-discarded event and records the metric. */ emitPipelinedCheckpointDiscarded(reason) {
|
|
904
|
+
this.metrics.recordPipelineParentCheckpointMismatch(reason);
|
|
905
|
+
this.eventEmitter.emit('pipelined-checkpoint-discarded', {
|
|
906
|
+
slot: this.targetSlot,
|
|
907
|
+
checkpointNumber: this.checkpointNumber,
|
|
908
|
+
reason
|
|
909
|
+
});
|
|
910
|
+
}
|
|
911
|
+
/** Simulates and enqueues an invalidation request for the invalid parent checkpoint. */ async enqueueInvalidation(validationStatus) {
|
|
912
|
+
if (this.config.skipInvalidateBlockAsProposer) {
|
|
913
|
+
this.log.warn(`Skipping checkpoint invalidation as proposer due to test configuration`);
|
|
914
|
+
return;
|
|
915
|
+
}
|
|
916
|
+
const invalidateRequest = await this.publisher.simulateInvalidateCheckpoint(validationStatus);
|
|
917
|
+
if (invalidateRequest) {
|
|
918
|
+
const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
|
|
919
|
+
const txTimeoutAt = new Date((submissionSlotStart + this.l1Constants.slotDuration) * 1000);
|
|
920
|
+
this.publisher.enqueueInvalidateCheckpoint(invalidateRequest, {
|
|
921
|
+
txTimeoutAt
|
|
922
|
+
});
|
|
923
|
+
} else {
|
|
924
|
+
this.log.info(`Invalidation simulation returned undefined, checkpoint may have been removed already`, {
|
|
925
|
+
checkpointNumber: this.checkpointNumber
|
|
642
926
|
});
|
|
643
|
-
return undefined;
|
|
644
927
|
}
|
|
645
928
|
}
|
|
646
929
|
async proposeCheckpoint() {
|
|
@@ -651,16 +934,28 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
651
934
|
hasError: false
|
|
652
935
|
};
|
|
653
936
|
try {
|
|
937
|
+
const now = this.dateProvider.now();
|
|
938
|
+
if (this.proposedCheckpointData) {
|
|
939
|
+
// Measure against the wall-clock slot whose build window we are currently using.
|
|
940
|
+
// In pipelining mode `targetSlot` is intentionally one slot ahead, which makes the
|
|
941
|
+
// target-slot boundary a full slot away from the actual build start time.
|
|
942
|
+
const slotBoundaryMs = Number(getTimestampForSlot(this.getBuildSlot(), this.l1Constants)) * 1000;
|
|
943
|
+
this.checkpointMetrics.recordPipelinedCheckpointBuildStartOffsetFromSlotBoundary(now - slotBoundaryMs);
|
|
944
|
+
}
|
|
945
|
+
this.checkpointMetrics.startCheckpointTiming(now);
|
|
654
946
|
// Get operator configured coinbase and fee recipient for this attestor
|
|
655
947
|
const coinbase = this.validatorClient.getCoinbaseForAttestor(this.attestorAddress);
|
|
656
948
|
const feeRecipient = this.validatorClient.getFeeRecipientForAttestor(this.attestorAddress);
|
|
657
949
|
// Start the checkpoint
|
|
658
|
-
this.
|
|
659
|
-
this.
|
|
660
|
-
buildSlot: this.
|
|
950
|
+
this.setState(SequencerState.INITIALIZING_CHECKPOINT);
|
|
951
|
+
this.logCheckpointEvent('slot-started', `Starting checkpoint proposal for slot ${this.targetSlot}`, {
|
|
952
|
+
buildSlot: this.getBuildSlot(),
|
|
661
953
|
submissionSlot: this.targetSlot,
|
|
662
|
-
|
|
954
|
+
slot: this.targetSlot,
|
|
955
|
+
checkpointNumber: this.checkpointNumber,
|
|
663
956
|
proposer: this.proposer?.toString(),
|
|
957
|
+
attestorAddress: this.attestorAddress.toString(),
|
|
958
|
+
publisherAddress: this.publisher.getSenderAddress().toString(),
|
|
664
959
|
coinbase: coinbase.toString()
|
|
665
960
|
});
|
|
666
961
|
this.metrics.incOpenSlot(this.targetSlot, this.proposer?.toString() ?? 'unknown');
|
|
@@ -668,15 +963,41 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
668
963
|
if (this.invalidateCheckpoint && !this.config.skipInvalidateBlockAsProposer) {
|
|
669
964
|
this.publisher.enqueueInvalidateCheckpoint(this.invalidateCheckpoint);
|
|
670
965
|
}
|
|
671
|
-
//
|
|
672
|
-
|
|
966
|
+
// Build the simulation plan for this slot. When pipelining, this overrides L1's view of
|
|
967
|
+
// pending/archive/fee-header to "as if the proposed parent had landed", so both the
|
|
968
|
+
// mana-min-fee simulation (in the globals builder) and the pre-broadcast
|
|
969
|
+
// validateBlockHeader see the chain tip the eventual L1 send will see.
|
|
970
|
+
this.checkpointSimulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
|
|
971
|
+
checkpointNumber: this.checkpointNumber,
|
|
972
|
+
proposedCheckpointData: this.proposedCheckpointData,
|
|
973
|
+
invalidateToPendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
|
|
974
|
+
checkpointedCheckpointNumber: this.checkpointedCheckpointNumber,
|
|
975
|
+
rollup: this.publisher.rollupContract,
|
|
976
|
+
signatureContext: this.signatureContext,
|
|
977
|
+
log: this.log
|
|
978
|
+
});
|
|
979
|
+
const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(coinbase, feeRecipient, this.targetSlot, this.checkpointSimulationOverridesPlan);
|
|
673
980
|
// Collect L1 to L2 messages for the checkpoint and compute their hash
|
|
674
981
|
const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(this.checkpointNumber);
|
|
675
982
|
const inHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
|
|
676
|
-
// Collect the out hashes of all the checkpoints before this one in the same epoch
|
|
677
|
-
|
|
678
|
-
//
|
|
679
|
-
|
|
983
|
+
// Collect the out hashes of all the checkpoints before this one in the same epoch.
|
|
984
|
+
// Under pipelining the parent checkpoint may not be on L1 yet at build time, so the helper
|
|
985
|
+
// splices in the parent's checkpointOutHash from the locally-known proposed checkpoint so
|
|
986
|
+
// the resulting `epochOutHash` matches what validators (and L1) compute once the parent
|
|
987
|
+
// lands on L1.
|
|
988
|
+
const previousCheckpointOutHashes = await getPreviousCheckpointOutHashes({
|
|
989
|
+
blockSource: this.l2BlockSource,
|
|
990
|
+
epoch: this.targetEpoch,
|
|
991
|
+
checkpointNumber: this.checkpointNumber,
|
|
992
|
+
l1Constants: this.epochCache.getL1Constants(),
|
|
993
|
+
pipeliningEnabled: true,
|
|
994
|
+
proposedCheckpointData: this.proposedCheckpointData,
|
|
995
|
+
log: this.log
|
|
996
|
+
});
|
|
997
|
+
// Anchor the modifier to the predicted parent fee header: L1 will apply it against
|
|
998
|
+
// that, not against the latest published checkpoint (which lags by one under pipelining).
|
|
999
|
+
const predictedParentEthPerFeeAssetE12 = this.checkpointSimulationOverridesPlan?.pendingCheckpointState?.feeHeader?.ethPerFeeAsset;
|
|
1000
|
+
const feeAssetPriceModifier = await this.publisher.getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12);
|
|
680
1001
|
const fork = _ts_add_disposable_resource(env, await this.worldState.fork(this.syncedToBlockNumber, {
|
|
681
1002
|
closeDelayMs: 12_000
|
|
682
1003
|
}), true);
|
|
@@ -689,7 +1010,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
689
1010
|
};
|
|
690
1011
|
const checkpointProposalOptions = {
|
|
691
1012
|
publishFullTxs: !!this.config.publishTxsWithProposals,
|
|
692
|
-
broadcastInvalidCheckpointProposal: this.config.broadcastInvalidBlockProposal
|
|
1013
|
+
broadcastInvalidCheckpointProposal: this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal
|
|
693
1014
|
};
|
|
694
1015
|
let blocksInCheckpoint = [];
|
|
695
1016
|
let blockPendingBroadcast = undefined;
|
|
@@ -709,8 +1030,15 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
709
1030
|
throw err;
|
|
710
1031
|
}
|
|
711
1032
|
if (blocksInCheckpoint.length === 0) {
|
|
1033
|
+
this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
|
|
1034
|
+
slot: this.targetSlot,
|
|
1035
|
+
checkpointNumber: this.checkpointNumber,
|
|
1036
|
+
reason: 'no_blocks_built'
|
|
1037
|
+
});
|
|
712
1038
|
this.log.warn(`No blocks were built for slot ${this.targetSlot}`, {
|
|
713
|
-
slot: this.targetSlot
|
|
1039
|
+
slot: this.targetSlot,
|
|
1040
|
+
checkpointNumber: this.checkpointNumber,
|
|
1041
|
+
reason: 'no_blocks_built'
|
|
714
1042
|
});
|
|
715
1043
|
this.eventEmitter.emit('checkpoint-empty', {
|
|
716
1044
|
slot: this.targetSlot
|
|
@@ -719,16 +1047,25 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
719
1047
|
}
|
|
720
1048
|
const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
|
|
721
1049
|
if (minBlocksForCheckpoint !== undefined && blocksInCheckpoint.length < minBlocksForCheckpoint) {
|
|
1050
|
+
this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
|
|
1051
|
+
slot: this.targetSlot,
|
|
1052
|
+
checkpointNumber: this.checkpointNumber,
|
|
1053
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
1054
|
+
minBlocksForCheckpoint,
|
|
1055
|
+
reason: 'min_blocks_not_met'
|
|
1056
|
+
});
|
|
722
1057
|
this.log.warn(`Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`, {
|
|
723
1058
|
slot: this.targetSlot,
|
|
1059
|
+
checkpointNumber: this.checkpointNumber,
|
|
724
1060
|
blocksBuilt: blocksInCheckpoint.length,
|
|
725
|
-
minBlocksForCheckpoint
|
|
1061
|
+
minBlocksForCheckpoint,
|
|
1062
|
+
reason: 'min_blocks_not_met'
|
|
726
1063
|
});
|
|
727
1064
|
return undefined;
|
|
728
1065
|
}
|
|
729
1066
|
// Assemble and broadcast the checkpoint proposal, including the last block that was not
|
|
730
1067
|
// broadcasted yet, and wait to collect the committee attestations.
|
|
731
|
-
this.
|
|
1068
|
+
this.setState(SequencerState.ASSEMBLING_CHECKPOINT);
|
|
732
1069
|
const checkpoint = await checkpointBuilder.completeCheckpoint();
|
|
733
1070
|
// Final validation: per-block limits are only checked if the operator set them explicitly.
|
|
734
1071
|
// Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
|
|
@@ -741,14 +1078,36 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
741
1078
|
maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint
|
|
742
1079
|
});
|
|
743
1080
|
} catch (err) {
|
|
744
|
-
this.
|
|
1081
|
+
this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
|
|
1082
|
+
slot: this.targetSlot,
|
|
1083
|
+
checkpointNumber: this.checkpointNumber,
|
|
1084
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
1085
|
+
reason: 'invalid_checkpoint',
|
|
1086
|
+
checkpoint: checkpoint.header.toInspect()
|
|
1087
|
+
});
|
|
1088
|
+
this.log.error(`Built an invalid checkpoint at slot ${this.targetSlot} (skipping proposal)`, err, {
|
|
1089
|
+
slot: this.targetSlot,
|
|
1090
|
+
checkpointNumber: this.checkpointNumber,
|
|
1091
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
1092
|
+
reason: 'invalid_checkpoint',
|
|
745
1093
|
checkpoint: checkpoint.header.toInspect()
|
|
746
1094
|
});
|
|
747
1095
|
return undefined;
|
|
748
1096
|
}
|
|
749
1097
|
// Record checkpoint-level build metrics
|
|
750
|
-
this.
|
|
751
|
-
|
|
1098
|
+
this.checkpointMetrics.recordCheckpointBuild(checkpointBuildTimer.ms(), blocksInCheckpoint.length, checkpoint.getStats().txCount, Number(checkpoint.header.totalManaUsed.toBigInt()));
|
|
1099
|
+
this.logCheckpointEvent('built', `Checkpoint built for slot ${this.targetSlot}`, {
|
|
1100
|
+
slot: this.targetSlot,
|
|
1101
|
+
buildSlot: this.getBuildSlot(),
|
|
1102
|
+
checkpointNumber: this.checkpointNumber,
|
|
1103
|
+
proposer: this.proposer?.toString(),
|
|
1104
|
+
attestorAddress: this.attestorAddress.toString(),
|
|
1105
|
+
publisherAddress: this.publisher.getSenderAddress().toString(),
|
|
1106
|
+
blocksBuilt: blocksInCheckpoint.length,
|
|
1107
|
+
txCount: checkpoint.getStats().txCount,
|
|
1108
|
+
totalMana: Number(checkpoint.header.totalManaUsed.toBigInt())
|
|
1109
|
+
});
|
|
1110
|
+
// In fisherman mode, return the checkpoint without broadcasting or collecting attestations
|
|
752
1111
|
if (this.config.fishermanMode) {
|
|
753
1112
|
this.log.info(`Built checkpoint for slot ${this.targetSlot} with ${blocksInCheckpoint.length} blocks. ` + `Skipping proposal in fisherman mode.`, {
|
|
754
1113
|
slot: this.targetSlot,
|
|
@@ -756,53 +1115,58 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
756
1115
|
blocksBuilt: blocksInCheckpoint.length
|
|
757
1116
|
});
|
|
758
1117
|
this.metrics.recordCheckpointSuccess();
|
|
759
|
-
|
|
1118
|
+
// Return a broadcast result with a dummy proposal — fisherman mode skips attestation collection
|
|
1119
|
+
return {
|
|
1120
|
+
checkpoint,
|
|
1121
|
+
proposal: undefined,
|
|
1122
|
+
blockProposedAt: this.dateProvider.now()
|
|
1123
|
+
};
|
|
760
1124
|
}
|
|
761
|
-
//
|
|
762
|
-
|
|
763
|
-
|
|
764
|
-
indexWithinCheckpoint: blockPendingBroadcast.block.indexWithinCheckpoint,
|
|
765
|
-
txs: blockPendingBroadcast.txs
|
|
766
|
-
};
|
|
767
|
-
// Create the checkpoint proposal and broadcast it
|
|
768
|
-
const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, feeAssetPriceModifier, lastBlock, this.proposer, checkpointProposalOptions);
|
|
769
|
-
const blockProposedAt = this.dateProvider.now();
|
|
770
|
-
await this.p2pClient.broadcastCheckpointProposal(proposal);
|
|
771
|
-
this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.targetSlot);
|
|
772
|
-
const attestations = await this.waitForAttestations(proposal);
|
|
773
|
-
const blockAttestedAt = this.dateProvider.now();
|
|
774
|
-
this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
|
|
775
|
-
// Proposer must sign over the attestations before pushing them to L1
|
|
776
|
-
const signer = this.proposer ?? this.publisher.getSenderAddress();
|
|
777
|
-
let attestationsSignature;
|
|
1125
|
+
// Validate the header against L1 state before broadcasting.
|
|
1126
|
+
// If this fails the slot is aborted before any gossip work; state drift between here
|
|
1127
|
+
// and the eventual L1 send is caught by the bundle simulate at send time.
|
|
778
1128
|
try {
|
|
779
|
-
|
|
1129
|
+
await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
|
|
780
1130
|
} catch (err) {
|
|
781
|
-
|
|
782
|
-
|
|
783
|
-
|
|
784
|
-
|
|
785
|
-
|
|
786
|
-
|
|
1131
|
+
this.log.error(`Pre-broadcast header validation failed for slot ${this.targetSlot}; aborting`, err, {
|
|
1132
|
+
slot: this.targetSlot,
|
|
1133
|
+
checkpointNumber: this.checkpointNumber
|
|
1134
|
+
});
|
|
1135
|
+
this.metrics.recordCheckpointProposalFailed('header_validation_failed');
|
|
1136
|
+
this.eventEmitter.emit('header-validation-failed', {
|
|
1137
|
+
slot: this.targetSlot,
|
|
1138
|
+
checkpointNumber: this.checkpointNumber,
|
|
1139
|
+
reason: err instanceof Error ? err.message : String(err)
|
|
1140
|
+
});
|
|
1141
|
+
return undefined;
|
|
787
1142
|
}
|
|
788
|
-
//
|
|
789
|
-
this.
|
|
790
|
-
|
|
791
|
-
|
|
792
|
-
|
|
793
|
-
//
|
|
794
|
-
if
|
|
795
|
-
|
|
796
|
-
|
|
797
|
-
|
|
798
|
-
|
|
1143
|
+
// Create the checkpoint proposal and broadcast it
|
|
1144
|
+
const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, this.checkpointNumber, feeAssetPriceModifier, blockPendingBroadcast, this.proposer, checkpointProposalOptions);
|
|
1145
|
+
// Advance our own optimistic proposed-checkpoint tip locally before gossiping. Gossipsub
|
|
1146
|
+
// doesn't echo our own messages back, so this is how the proposer makes its own proposed
|
|
1147
|
+
// checkpoint visible for pipelining the next slot. Built from local checkpoint data — never
|
|
1148
|
+
// from the broadcast proposal archive, which may be deliberately corrupted under test flags.
|
|
1149
|
+
// Fail closed: if this throws, the outer catch aborts the slot before gossiping.
|
|
1150
|
+
await this.syncProposedCheckpointToArchiver(checkpoint, blocksInCheckpoint.length, feeAssetPriceModifier);
|
|
1151
|
+
const blockProposedAt = this.dateProvider.now();
|
|
1152
|
+
if (this.config.skipBroadcastCheckpointProposal) {
|
|
1153
|
+
// Test-only: suppress the CheckpointProposal so peers never see a proposed checkpoint for
|
|
1154
|
+
// this slot, but still broadcast the held last block standalone so peers' archivers ingest
|
|
1155
|
+
// it as a proposed-but-uncheckpointed tip — the exact orphan-block state that
|
|
1156
|
+
// pruneOrphanProposedBlocks / checkSync must handle.
|
|
1157
|
+
if (blockPendingBroadcast && !this.config.skipBroadcastProposals) {
|
|
1158
|
+
await this.p2pClient.broadcastProposal(blockPendingBroadcast);
|
|
799
1159
|
}
|
|
1160
|
+
} else if (!this.config.skipBroadcastProposals) {
|
|
1161
|
+
await this.p2pClient.broadcastCheckpointProposal(proposal);
|
|
1162
|
+
this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
|
|
800
1163
|
}
|
|
801
|
-
|
|
802
|
-
|
|
803
|
-
|
|
804
|
-
|
|
805
|
-
|
|
1164
|
+
// Return immediately after broadcast — attestation collection happens in the background
|
|
1165
|
+
return {
|
|
1166
|
+
checkpoint,
|
|
1167
|
+
proposal,
|
|
1168
|
+
blockProposedAt
|
|
1169
|
+
};
|
|
806
1170
|
} catch (e) {
|
|
807
1171
|
env.error = e;
|
|
808
1172
|
env.hasError = true;
|
|
@@ -815,7 +1179,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
815
1179
|
// swallow this error. It's already been logged by a function deeper in the stack
|
|
816
1180
|
return undefined;
|
|
817
1181
|
}
|
|
818
|
-
this.log.error(`Error building checkpoint at slot ${this.
|
|
1182
|
+
this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
|
|
819
1183
|
return undefined;
|
|
820
1184
|
}
|
|
821
1185
|
}
|
|
@@ -831,13 +1195,21 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
831
1195
|
const blocksBuilt = blocksInCheckpoint.length;
|
|
832
1196
|
const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
|
|
833
1197
|
const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
|
|
834
|
-
|
|
835
|
-
|
|
1198
|
+
if (blocksBuilt >= this.config.maxBlocksPerCheckpoint) {
|
|
1199
|
+
this.log.debug(`Reached max blocks per checkpoint`, {
|
|
1200
|
+
slot: this.targetSlot,
|
|
1201
|
+
blocksBuilt,
|
|
1202
|
+
maxBlocksPerCheckpoint: this.config.maxBlocksPerCheckpoint
|
|
1203
|
+
});
|
|
1204
|
+
break;
|
|
1205
|
+
}
|
|
1206
|
+
const nowSeconds = this.dateProvider.now() / 1000;
|
|
1207
|
+
const timingInfo = this.timetable.selectNextSubslot(this.targetSlot, nowSeconds);
|
|
836
1208
|
if (!timingInfo.canStart) {
|
|
837
1209
|
this.log.debug(`Not enough time left in slot to start another block`, {
|
|
838
1210
|
slot: this.targetSlot,
|
|
839
1211
|
blocksBuilt,
|
|
840
|
-
|
|
1212
|
+
nowSeconds
|
|
841
1213
|
});
|
|
842
1214
|
break;
|
|
843
1215
|
}
|
|
@@ -846,23 +1218,24 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
846
1218
|
blockTimestamp: timestamp,
|
|
847
1219
|
// Create an empty block if we haven't already and this is the last one
|
|
848
1220
|
forceCreate: timingInfo.isLastBlock && blocksBuilt === 0 && this.config.buildCheckpointIfEmpty,
|
|
849
|
-
|
|
850
|
-
buildDeadline: timingInfo.deadline ? new Date((this.getSlotStartBuildTimestamp() + timingInfo.deadline) * 1000) : undefined,
|
|
1221
|
+
buildDeadline: new Date(timingInfo.deadline * 1000),
|
|
851
1222
|
blockNumber,
|
|
852
1223
|
indexWithinCheckpoint,
|
|
853
1224
|
txHashesAlreadyIncluded
|
|
854
1225
|
});
|
|
855
|
-
//
|
|
856
|
-
if (
|
|
857
|
-
|
|
858
|
-
|
|
859
|
-
|
|
1226
|
+
// If we failed to build the block due to insufficient txs, we try again if there is still time left in the slot
|
|
1227
|
+
if ('failure' in buildResult) {
|
|
1228
|
+
// If this was the last subslot, we're done.
|
|
1229
|
+
if (timingInfo.isLastBlock) {
|
|
1230
|
+
break;
|
|
1231
|
+
}
|
|
1232
|
+
// Otherwise, if there is still time for more blocks, we wait until the next subslot and try again
|
|
860
1233
|
await this.waitUntilNextSubslot(timingInfo.deadline);
|
|
861
1234
|
continue;
|
|
862
|
-
}
|
|
863
|
-
|
|
864
|
-
|
|
865
|
-
|
|
1235
|
+
}
|
|
1236
|
+
// If there was an error building the block, we just exit the loop and give up the rest of the slot.
|
|
1237
|
+
// We don't want to risk building more blocks if something went wrong.
|
|
1238
|
+
if ('error' in buildResult) {
|
|
866
1239
|
if (!(buildResult.error instanceof SequencerInterruptedError)) {
|
|
867
1240
|
this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
|
|
868
1241
|
slot: this.targetSlot,
|
|
@@ -873,34 +1246,36 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
873
1246
|
break;
|
|
874
1247
|
}
|
|
875
1248
|
const { block, usedTxs } = buildResult;
|
|
1249
|
+
this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
|
|
1250
|
+
isFirstBlock: blocksBuilt === 0,
|
|
1251
|
+
isLastBlock: timingInfo.isLastBlock
|
|
1252
|
+
});
|
|
876
1253
|
blocksInCheckpoint.push(block);
|
|
877
1254
|
usedTxs.forEach((tx)=>txHashesAlreadyIncluded.add(tx.txHash.toString()));
|
|
878
|
-
//
|
|
879
|
-
|
|
1255
|
+
// Sign the block proposal. This will throw if HA signing fails.
|
|
1256
|
+
const proposal = await this.createBlockProposal(block, inHash, usedTxs, {
|
|
1257
|
+
...blockProposalOptions,
|
|
1258
|
+
broadcastInvalidBlockProposal: blockProposalOptions.broadcastInvalidBlockProposal || block.indexWithinCheckpoint === this.config.invalidBlockProposalIndexWithinCheckpoint
|
|
1259
|
+
});
|
|
1260
|
+
// Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal,
|
|
1261
|
+
// so we avoid polluting our archive with a block that would fail.
|
|
1262
|
+
// We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
|
|
1263
|
+
// If this throws, we abort the entire checkpoint.
|
|
1264
|
+
await this.syncProposedBlockToArchiver(block);
|
|
1265
|
+
// If this is the last block, do not broadcast it, since it will be included in the checkpoint proposal.
|
|
880
1266
|
if (timingInfo.isLastBlock) {
|
|
881
|
-
await this.syncProposedBlockToArchiver(block);
|
|
882
1267
|
this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
|
|
883
1268
|
slot: this.targetSlot,
|
|
884
1269
|
blockNumber,
|
|
885
1270
|
blocksBuilt
|
|
886
1271
|
});
|
|
887
|
-
blockPendingBroadcast =
|
|
888
|
-
block,
|
|
889
|
-
txs: usedTxs
|
|
890
|
-
};
|
|
1272
|
+
blockPendingBroadcast = proposal;
|
|
891
1273
|
break;
|
|
892
1274
|
}
|
|
893
|
-
// Broadcast the block proposal (unless we're in fisherman mode) unless the block is the last one,
|
|
894
|
-
// in which case we'll broadcast it along with the checkpoint at the end of the loop.
|
|
895
|
-
// Note that we only send the block to the archiver if we manage to create the proposal, so if there's
|
|
896
|
-
// a HA error we don't pollute our archiver with a block that won't make it to the chain.
|
|
897
|
-
const proposal = await this.createBlockProposal(block, inHash, usedTxs, blockProposalOptions);
|
|
898
|
-
// Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal.
|
|
899
|
-
// We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
|
|
900
|
-
// If this throws, we abort the entire checkpoint.
|
|
901
|
-
await this.syncProposedBlockToArchiver(block);
|
|
902
1275
|
// Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
|
|
903
|
-
proposal &&
|
|
1276
|
+
if (proposal && !this.config.skipBroadcastProposals) {
|
|
1277
|
+
await this.p2pClient.broadcastProposal(proposal);
|
|
1278
|
+
}
|
|
904
1279
|
// Wait until the next block's start time
|
|
905
1280
|
await this.waitUntilNextSubslot(timingInfo.deadline);
|
|
906
1281
|
}
|
|
@@ -918,14 +1293,17 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
918
1293
|
this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
|
|
919
1294
|
return Promise.resolve(undefined);
|
|
920
1295
|
}
|
|
921
|
-
return this.validatorClient.createBlockProposal(block.header, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
|
|
1296
|
+
return this.validatorClient.createBlockProposal(block.header, this.checkpointNumber, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
|
|
922
1297
|
}
|
|
923
|
-
/**
|
|
924
|
-
|
|
925
|
-
|
|
1298
|
+
/**
|
|
1299
|
+
* Sleeps until it is time to produce the next block in the slot.
|
|
1300
|
+
* @param nextSubslotStart - Absolute wall-clock timestamp in seconds of the previous sub-slot deadline.
|
|
1301
|
+
*/ async waitUntilNextSubslot(nextSubslotStart) {
|
|
1302
|
+
this.setState(SequencerState.WAITING_UNTIL_NEXT_BLOCK);
|
|
1303
|
+
this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s`, {
|
|
926
1304
|
slot: this.targetSlot
|
|
927
1305
|
});
|
|
928
|
-
await this.
|
|
1306
|
+
await this.awaitInterruptibleSleep(Math.max(0, nextSubslotStart * 1000 - this.dateProvider.now()));
|
|
929
1307
|
}
|
|
930
1308
|
/** Builds a single block. Called from the main block building loop. */ async buildSingleBlock(checkpointBuilder, opts) {
|
|
931
1309
|
const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } = opts;
|
|
@@ -935,30 +1313,46 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
935
1313
|
});
|
|
936
1314
|
try {
|
|
937
1315
|
// Wait until we have enough txs to build the block
|
|
938
|
-
const {
|
|
1316
|
+
const { canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
|
|
939
1317
|
if (!canStartBuilding) {
|
|
940
|
-
this.
|
|
1318
|
+
this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
|
|
1319
|
+
reason: 'insufficient_txs',
|
|
941
1320
|
blockNumber,
|
|
942
1321
|
slot: this.targetSlot,
|
|
943
|
-
|
|
1322
|
+
checkpointNumber: this.checkpointNumber,
|
|
1323
|
+
indexWithinCheckpoint,
|
|
1324
|
+
minTxs
|
|
1325
|
+
});
|
|
1326
|
+
this.log.verbose(`Not enough age-eligible txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (needs ${minTxs} eligible)`, {
|
|
1327
|
+
reason: 'insufficient_txs',
|
|
1328
|
+
blockNumber,
|
|
1329
|
+
slot: this.targetSlot,
|
|
1330
|
+
checkpointNumber: this.checkpointNumber,
|
|
1331
|
+
indexWithinCheckpoint,
|
|
1332
|
+
minTxs
|
|
944
1333
|
});
|
|
945
1334
|
this.eventEmitter.emit('block-tx-count-check-failed', {
|
|
946
1335
|
minTxs,
|
|
947
|
-
availableTxs,
|
|
948
1336
|
slot: this.targetSlot
|
|
949
1337
|
});
|
|
950
1338
|
this.metrics.recordBlockProposalFailed('insufficient_txs');
|
|
951
|
-
return
|
|
1339
|
+
return {
|
|
1340
|
+
failure: 'insufficient-txs'
|
|
1341
|
+
};
|
|
952
1342
|
}
|
|
953
1343
|
// Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
|
|
954
1344
|
// just in case p2p failed to sync the provisional block and didn't get to remove those txs from the mempool yet.
|
|
955
|
-
|
|
956
|
-
|
|
1345
|
+
// Block building only executes txs, so we skip loading their proofs unless these same tx objects get attached
|
|
1346
|
+
// to the broadcasted proposals via publishTxsWithProposals.
|
|
1347
|
+
const pendingTxs = filter(this.p2pClient.iterateEligiblePendingTxs({
|
|
1348
|
+
includeProof: !!this.config.publishTxsWithProposals
|
|
1349
|
+
}), (tx)=>!txHashesAlreadyIncluded.has(tx.txHash.toString()));
|
|
1350
|
+
this.log.debug(`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.targetSlot}`, {
|
|
957
1351
|
slot: this.targetSlot,
|
|
958
1352
|
blockNumber,
|
|
959
1353
|
indexWithinCheckpoint
|
|
960
1354
|
});
|
|
961
|
-
this.
|
|
1355
|
+
this.setState(SequencerState.CREATING_BLOCK);
|
|
962
1356
|
// Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
|
|
963
1357
|
// by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
|
|
964
1358
|
// minValidTxs is passed into the builder so it can reject the block *before* updating state.
|
|
@@ -969,8 +1363,9 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
969
1363
|
deadline: buildDeadline,
|
|
970
1364
|
isBuildingProposal: true,
|
|
971
1365
|
minValidTxs,
|
|
972
|
-
maxBlocksPerCheckpoint: this.timetable.
|
|
973
|
-
perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier
|
|
1366
|
+
maxBlocksPerCheckpoint: this.timetable.getMaxBlocksPerCheckpoint(),
|
|
1367
|
+
perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier,
|
|
1368
|
+
perBlockDAAllocationMultiplier: this.config.perBlockDAAllocationMultiplier
|
|
974
1369
|
};
|
|
975
1370
|
// Actually build the block by executing txs. The builder throws InsufficientValidTxsError
|
|
976
1371
|
// if the number of successfully processed txs is below minValidTxs, ensuring state is not
|
|
@@ -979,8 +1374,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
979
1374
|
// If any txs failed during execution, drop them from the mempool so we don't pick them up again
|
|
980
1375
|
await this.dropFailedTxsFromP2P(buildResult.failedTxs);
|
|
981
1376
|
if (buildResult.status === 'insufficient-valid-txs') {
|
|
1377
|
+
this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
|
|
1378
|
+
reason: 'insufficient_valid_txs',
|
|
1379
|
+
slot: this.targetSlot,
|
|
1380
|
+
checkpointNumber: this.checkpointNumber,
|
|
1381
|
+
blockNumber,
|
|
1382
|
+
numTxs: buildResult.processedCount,
|
|
1383
|
+
indexWithinCheckpoint,
|
|
1384
|
+
minValidTxs
|
|
1385
|
+
});
|
|
982
1386
|
this.log.warn(`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.targetSlot} has too few valid txs to be proposed`, {
|
|
1387
|
+
reason: 'insufficient_valid_txs',
|
|
983
1388
|
slot: this.targetSlot,
|
|
1389
|
+
checkpointNumber: this.checkpointNumber,
|
|
984
1390
|
blockNumber,
|
|
985
1391
|
numTxs: buildResult.processedCount,
|
|
986
1392
|
indexWithinCheckpoint,
|
|
@@ -991,7 +1397,9 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
991
1397
|
slot: this.targetSlot
|
|
992
1398
|
});
|
|
993
1399
|
this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
|
|
994
|
-
return
|
|
1400
|
+
return {
|
|
1401
|
+
failure: 'insufficient-valid-txs'
|
|
1402
|
+
};
|
|
995
1403
|
}
|
|
996
1404
|
// Block creation succeeded, emit stats and metrics
|
|
997
1405
|
const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
|
|
@@ -1010,12 +1418,17 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1010
1418
|
manaPerSec,
|
|
1011
1419
|
...blockStats
|
|
1012
1420
|
});
|
|
1421
|
+
// `slot` is the target/submission slot (may be one ahead when pipelining),
|
|
1422
|
+
// `buildSlot` is the wall-clock slot during which the block was actually built.
|
|
1013
1423
|
this.eventEmitter.emit('block-proposed', {
|
|
1014
1424
|
blockNumber: block.number,
|
|
1425
|
+
blockHash,
|
|
1426
|
+
checkpointNumber: this.checkpointNumber,
|
|
1427
|
+
indexWithinCheckpoint: block.indexWithinCheckpoint,
|
|
1015
1428
|
slot: this.targetSlot,
|
|
1016
|
-
buildSlot: this.
|
|
1429
|
+
buildSlot: this.getBuildSlot()
|
|
1017
1430
|
});
|
|
1018
|
-
this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe());
|
|
1431
|
+
this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
|
|
1019
1432
|
return {
|
|
1020
1433
|
block,
|
|
1021
1434
|
usedTxs
|
|
@@ -1025,9 +1438,17 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1025
1438
|
reason: err.message,
|
|
1026
1439
|
slot: this.targetSlot
|
|
1027
1440
|
});
|
|
1441
|
+
this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
|
|
1442
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
1443
|
+
slot: this.targetSlot,
|
|
1444
|
+
checkpointNumber: this.checkpointNumber,
|
|
1445
|
+
blockNumber
|
|
1446
|
+
});
|
|
1028
1447
|
this.log.error(`Error building block`, err, {
|
|
1029
|
-
|
|
1030
|
-
slot: this.targetSlot
|
|
1448
|
+
reason: err instanceof Error ? err.message : String(err),
|
|
1449
|
+
slot: this.targetSlot,
|
|
1450
|
+
checkpointNumber: this.checkpointNumber,
|
|
1451
|
+
blockNumber
|
|
1031
1452
|
});
|
|
1032
1453
|
this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
|
|
1033
1454
|
this.metrics.recordFailedBlock();
|
|
@@ -1061,42 +1482,65 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1061
1482
|
const { indexWithinCheckpoint, blockNumber, buildDeadline, forceCreate } = opts;
|
|
1062
1483
|
// We only allow a block with 0 txs in the first block of the checkpoint
|
|
1063
1484
|
const minTxs = indexWithinCheckpoint > 0 && this.config.minTxsPerBlock === 0 ? 1 : this.config.minTxsPerBlock;
|
|
1064
|
-
//
|
|
1065
|
-
const startBuildingDeadline = buildDeadline ? new Date(buildDeadline.getTime() - this.timetable.
|
|
1066
|
-
|
|
1067
|
-
|
|
1485
|
+
// Latest time to keep waiting for txs: wait_for_txs_deadline = block_build_deadline(k) - min_block_duration.
|
|
1486
|
+
const startBuildingDeadline = buildDeadline ? new Date(buildDeadline.getTime() - this.timetable.minBlockDuration * 1000) : undefined;
|
|
1487
|
+
// Gate on age-eligible txs so we don't start building on txs the builder would then filter out for being
|
|
1488
|
+
// too fresh. hasEligiblePendingTxs early-exits once minTxs are eligible instead of counting the whole pool.
|
|
1489
|
+
while(!forceCreate && !await this.p2pClient.hasEligiblePendingTxs(minTxs)){
|
|
1068
1490
|
// If we're past deadline, or we have no deadline, give up
|
|
1069
1491
|
const now = this.dateProvider.nowAsDate();
|
|
1070
1492
|
if (startBuildingDeadline === undefined || now >= startBuildingDeadline) {
|
|
1071
1493
|
return {
|
|
1072
1494
|
canStartBuilding: false,
|
|
1073
|
-
availableTxs,
|
|
1074
1495
|
minTxs
|
|
1075
1496
|
};
|
|
1076
1497
|
}
|
|
1077
1498
|
// Wait a bit before checking again
|
|
1078
|
-
this.
|
|
1079
|
-
this.log.verbose(`Waiting for
|
|
1499
|
+
this.setState(SequencerState.WAITING_FOR_TXS);
|
|
1500
|
+
this.log.verbose(`Waiting for ${minTxs} age-eligible txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot}`, {
|
|
1080
1501
|
blockNumber,
|
|
1081
1502
|
slot: this.targetSlot,
|
|
1082
|
-
indexWithinCheckpoint
|
|
1503
|
+
indexWithinCheckpoint,
|
|
1504
|
+
minTxs
|
|
1083
1505
|
});
|
|
1084
1506
|
await this.waitForTxsPollingInterval();
|
|
1085
|
-
availableTxs = await this.p2pClient.getPendingTxCount();
|
|
1086
1507
|
}
|
|
1087
1508
|
return {
|
|
1088
1509
|
canStartBuilding: true,
|
|
1089
|
-
availableTxs,
|
|
1090
1510
|
minTxs
|
|
1091
1511
|
};
|
|
1092
1512
|
}
|
|
1513
|
+
async getSignedCommitteeAttestations(broadcast) {
|
|
1514
|
+
const { proposal, blockProposedAt } = broadcast;
|
|
1515
|
+
this.setState(SequencerState.COLLECTING_ATTESTATIONS);
|
|
1516
|
+
const attestations = await this.waitForAttestations(proposal);
|
|
1517
|
+
if (!attestations) {
|
|
1518
|
+
return undefined;
|
|
1519
|
+
}
|
|
1520
|
+
this.checkpointMetrics.recordCheckpointAttestationDelay(this.dateProvider.now() - blockProposedAt);
|
|
1521
|
+
// Proposer must sign over the attestations before pushing them to L1
|
|
1522
|
+
const signer = this.proposer ?? this.publisher.getSenderAddress();
|
|
1523
|
+
try {
|
|
1524
|
+
const attestationsSignature = await this.validatorClient.signAttestationsAndSigners(attestations, signer, this.targetSlot, this.checkpointNumber);
|
|
1525
|
+
return {
|
|
1526
|
+
attestations,
|
|
1527
|
+
attestationsSignature
|
|
1528
|
+
};
|
|
1529
|
+
} catch (err) {
|
|
1530
|
+
if (this.handleHASigningError(err, 'Attestations signature')) {
|
|
1531
|
+
return;
|
|
1532
|
+
}
|
|
1533
|
+
this.log.error(`Error signing attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
|
|
1534
|
+
return undefined;
|
|
1535
|
+
}
|
|
1536
|
+
}
|
|
1093
1537
|
/**
|
|
1094
1538
|
* Waits for enough attestations to be collected via p2p.
|
|
1095
1539
|
* This is run after all blocks for the checkpoint have been built.
|
|
1096
1540
|
*/ async waitForAttestations(proposal) {
|
|
1097
1541
|
if (this.config.fishermanMode) {
|
|
1098
1542
|
this.log.debug('Skipping attestation collection in fisherman mode');
|
|
1099
|
-
return CommitteeAttestationsAndSigners.empty();
|
|
1543
|
+
return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
|
|
1100
1544
|
}
|
|
1101
1545
|
const slotNumber = proposal.slotNumber;
|
|
1102
1546
|
const { committee, seed, epoch } = await this.epochCache.getCommittee(slotNumber);
|
|
@@ -1104,25 +1548,29 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1104
1548
|
throw new Error('No committee when collecting attestations');
|
|
1105
1549
|
} else if (committee.length === 0) {
|
|
1106
1550
|
this.log.verbose(`Attesting committee is empty`);
|
|
1107
|
-
return CommitteeAttestationsAndSigners.empty();
|
|
1551
|
+
return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
|
|
1108
1552
|
} else {
|
|
1109
1553
|
this.log.debug(`Attesting committee length is ${committee.length}`, {
|
|
1110
1554
|
committee
|
|
1111
1555
|
});
|
|
1112
1556
|
}
|
|
1113
|
-
const numberOfRequiredAttestations =
|
|
1557
|
+
const numberOfRequiredAttestations = computeQuorum(committee.length);
|
|
1114
1558
|
if (this.config.skipCollectingAttestations) {
|
|
1115
1559
|
this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
|
|
1116
|
-
const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
|
|
1117
|
-
|
|
1560
|
+
const attestations = await this.validatorClient?.collectOwnAttestations(proposal, this.checkpointNumber);
|
|
1561
|
+
this.logCheckpointAttestations('collected', committee, attestations ?? [], numberOfRequiredAttestations, {
|
|
1562
|
+
reason: 'collect_own_only'
|
|
1563
|
+
});
|
|
1564
|
+
return new CommitteeAttestationsAndSigners(orderAttestations(attestations ?? [], committee), this.getSignatureContext());
|
|
1118
1565
|
}
|
|
1119
|
-
|
|
1120
|
-
const
|
|
1121
|
-
|
|
1566
|
+
// Hard attestation-collection cutoff = the single consensus attestation_deadline (target_slot_start + S - 2E).
|
|
1567
|
+
const attestationDeadlineSeconds = this.timetable.getAttestationDeadline(this.targetSlot);
|
|
1568
|
+
const attestationDeadline = new Date(attestationDeadlineSeconds * 1000);
|
|
1569
|
+
this.metrics.recordRequiredAttestations(numberOfRequiredAttestations, Math.max(0, attestationDeadline.getTime() - this.dateProvider.now()));
|
|
1122
1570
|
const collectAttestationsTimer = new Timer();
|
|
1123
1571
|
let collectedAttestationsCount = 0;
|
|
1124
1572
|
try {
|
|
1125
|
-
const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline);
|
|
1573
|
+
const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline, this.checkpointNumber);
|
|
1126
1574
|
collectedAttestationsCount = attestations.length;
|
|
1127
1575
|
// Trim attestations to minimum required to save L1 calldata gas
|
|
1128
1576
|
const localAddresses = this.validatorClient.getValidatorAddresses();
|
|
@@ -1132,20 +1580,55 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1132
1580
|
}
|
|
1133
1581
|
// Rollup contract requires that the signatures are provided in the order of the committee
|
|
1134
1582
|
const sorted = orderAttestations(trimmed, committee);
|
|
1583
|
+
this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
|
|
1584
|
+
submittedCount: trimmed.length
|
|
1585
|
+
});
|
|
1135
1586
|
// Manipulate the attestations if we've been configured to do so
|
|
1136
|
-
if (this.config.injectFakeAttestation || this.config.injectHighSValueAttestation || this.config.injectUnrecoverableSignatureAttestation || this.config.shuffleAttestationOrdering) {
|
|
1587
|
+
if (this.config.injectFakeAttestation || this.config.injectHighSValueAttestation || this.config.injectUnrecoverableSignatureAttestation || this.config.injectYParityAttestation || this.config.shuffleAttestationOrdering) {
|
|
1137
1588
|
return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
|
|
1138
1589
|
}
|
|
1139
|
-
return new CommitteeAttestationsAndSigners(sorted);
|
|
1590
|
+
return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
|
|
1140
1591
|
} catch (err) {
|
|
1141
1592
|
if (err && err instanceof AttestationTimeoutError) {
|
|
1142
1593
|
collectedAttestationsCount = err.collectedCount;
|
|
1594
|
+
this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
|
|
1595
|
+
collectedCount: collectedAttestationsCount,
|
|
1596
|
+
reason: 'timeout'
|
|
1597
|
+
});
|
|
1598
|
+
this.log.error(`Timeout while waiting for attestations for checkpoint proposal at slot ${proposal.slotNumber} (collected ${collectedAttestationsCount}/${numberOfRequiredAttestations})`, err);
|
|
1599
|
+
} else {
|
|
1600
|
+
this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
|
|
1601
|
+
collectedCount: collectedAttestationsCount,
|
|
1602
|
+
reason: err instanceof Error ? err.message : String(err)
|
|
1603
|
+
});
|
|
1604
|
+
this.log.error(`Error collecting attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
|
|
1143
1605
|
}
|
|
1144
|
-
|
|
1606
|
+
return undefined;
|
|
1145
1607
|
} finally{
|
|
1146
1608
|
this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
|
|
1147
1609
|
}
|
|
1148
1610
|
}
|
|
1611
|
+
logCheckpointAttestations(status, committee, attestations, requiredAttestations, opts = {}) {
|
|
1612
|
+
const signedValidators = attestations?.map((attestation)=>attestation.getSender()?.toString()).filter((address)=>address !== undefined) ?? [];
|
|
1613
|
+
const collectedCount = opts.collectedCount ?? new Set(signedValidators).size;
|
|
1614
|
+
const missingValidatorCount = status === 'failed' ? Math.max(0, requiredAttestations - collectedCount) : undefined;
|
|
1615
|
+
this.logCheckpointEvent(`attestations-${status}`, `Checkpoint attestations ${status} for slot ${this.targetSlot}`, {
|
|
1616
|
+
slot: this.targetSlot,
|
|
1617
|
+
checkpointNumber: this.checkpointNumber,
|
|
1618
|
+
committeeSize: committee.length,
|
|
1619
|
+
requiredAttestations,
|
|
1620
|
+
collectedAttestations: collectedCount,
|
|
1621
|
+
...opts.submittedCount !== undefined && {
|
|
1622
|
+
submittedAttestations: opts.submittedCount
|
|
1623
|
+
},
|
|
1624
|
+
...missingValidatorCount !== undefined && {
|
|
1625
|
+
missingValidatorCount
|
|
1626
|
+
},
|
|
1627
|
+
...opts.reason !== undefined && {
|
|
1628
|
+
reason: opts.reason
|
|
1629
|
+
}
|
|
1630
|
+
});
|
|
1631
|
+
}
|
|
1149
1632
|
/** Breaks the attestations before publishing based on attack configs */ manipulateAttestations(slotNumber, epoch, seed, committee, attestations) {
|
|
1150
1633
|
// Compute the proposer index in the committee, since we dont want to tweak it.
|
|
1151
1634
|
// Otherwise, the L1 rollup contract will reject the block outright.
|
|
@@ -1171,7 +1654,15 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1171
1654
|
unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
|
|
1172
1655
|
}
|
|
1173
1656
|
}
|
|
1174
|
-
return new CommitteeAttestationsAndSigners(attestations);
|
|
1657
|
+
return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
|
|
1658
|
+
}
|
|
1659
|
+
if (this.config.injectYParityAttestation) {
|
|
1660
|
+
// Force every non-proposer signed slot's recovery byte to yParity (v ∈ {0, 1}) form in the packed L1
|
|
1661
|
+
// tuple, after packAttestations has canonicalized it. The proposer's own slot is left canonical so
|
|
1662
|
+
// propose() still passes verifyProposer. Models a malicious proposer landing a checkpoint L1 accepts
|
|
1663
|
+
// but that can never be proven (ECDSA.recover rejects v ∉ {27, 28}).
|
|
1664
|
+
this.log.warn(`Injecting yParity attestations in checkpoint for slot ${slotNumber} (proposer #${proposerIndex})`);
|
|
1665
|
+
return new MaliciousYParityCommitteeAttestationsAndSigners(attestations, proposerIndex, this.getSignatureContext());
|
|
1175
1666
|
}
|
|
1176
1667
|
if (this.config.shuffleAttestationOrdering) {
|
|
1177
1668
|
this.log.warn(`Shuffling attestation ordering in checkpoint for slot ${slotNumber} (proposer #${proposerIndex})`);
|
|
@@ -1197,10 +1688,10 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1197
1688
|
shuffled[i]
|
|
1198
1689
|
];
|
|
1199
1690
|
}
|
|
1200
|
-
const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
|
|
1201
|
-
return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
|
|
1691
|
+
const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
|
|
1692
|
+
return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
|
|
1202
1693
|
}
|
|
1203
|
-
return new CommitteeAttestationsAndSigners(attestations);
|
|
1694
|
+
return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
|
|
1204
1695
|
}
|
|
1205
1696
|
async dropFailedTxsFromP2P(failedTxs) {
|
|
1206
1697
|
if (failedTxs.length === 0) {
|
|
@@ -1208,15 +1699,27 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1208
1699
|
}
|
|
1209
1700
|
const failedTxData = failedTxs.map((fail)=>fail.tx);
|
|
1210
1701
|
const failedTxHashes = failedTxData.map((tx)=>tx.getTxHash());
|
|
1211
|
-
|
|
1702
|
+
const failures = failedTxs.map((fail)=>({
|
|
1703
|
+
txHash: fail.tx.getTxHash().toString(),
|
|
1704
|
+
reason: fail.error.message
|
|
1705
|
+
}));
|
|
1706
|
+
this.log.warn(`Dropping ${failedTxs.length} txs from mempool due to failures during block building for slot ${this.targetSlot}`, {
|
|
1707
|
+
slot: this.targetSlot,
|
|
1708
|
+
checkpointNumber: this.checkpointNumber,
|
|
1709
|
+
failures
|
|
1710
|
+
});
|
|
1212
1711
|
await this.p2pClient.handleFailedExecution(failedTxHashes);
|
|
1213
1712
|
}
|
|
1214
1713
|
/**
|
|
1215
1714
|
* Adds the proposed block to the archiver so it's available via P2P.
|
|
1216
1715
|
* Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
|
|
1217
1716
|
* would never receive its own block without this explicit sync.
|
|
1717
|
+
*
|
|
1718
|
+
* In fisherman mode we skip this push: the fisherman builds blocks locally for validation
|
|
1719
|
+
* and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
|
|
1720
|
+
* whenever the real proposer's block arrives from L1.
|
|
1218
1721
|
*/ async syncProposedBlockToArchiver(block) {
|
|
1219
|
-
if (this.config.skipPushProposedBlocksToArchiver
|
|
1722
|
+
if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
|
|
1220
1723
|
this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
|
|
1221
1724
|
blockNumber: block.number,
|
|
1222
1725
|
slot: block.header.globalVariables.slotNumber
|
|
@@ -1229,6 +1732,35 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1229
1732
|
});
|
|
1230
1733
|
await this.blockSink.addBlock(block);
|
|
1231
1734
|
}
|
|
1735
|
+
/**
|
|
1736
|
+
* Adds the proposed checkpoint to the archiver so the proposer's optimistic proposed-checkpoint
|
|
1737
|
+
* tip advances locally. Gossip doesn't echo our own messages back, so without this the proposer
|
|
1738
|
+
* would never see its own proposed checkpoint and couldn't pipeline the next slot.
|
|
1739
|
+
*
|
|
1740
|
+
* Skipped whenever proposed blocks aren't pushed (`skipPushProposedBlocksToArchiver`, fisherman
|
|
1741
|
+
* mode): the archiver derives the checkpoint archive from its stored blocks, so without them the
|
|
1742
|
+
* push would fail. All blocks were already added (and awaited) during block building, so this
|
|
1743
|
+
* needs no retry — they are guaranteed present by the time we get here.
|
|
1744
|
+
*/ async syncProposedCheckpointToArchiver(checkpoint, blockCount, feeAssetPriceModifier) {
|
|
1745
|
+
if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
|
|
1746
|
+
return;
|
|
1747
|
+
}
|
|
1748
|
+
const startBlock = BlockNumber(this.syncedToBlockNumber + 1);
|
|
1749
|
+
this.log.debug(`Syncing proposed checkpoint ${this.checkpointNumber} to archiver`, {
|
|
1750
|
+
checkpointNumber: this.checkpointNumber,
|
|
1751
|
+
slot: this.targetSlot,
|
|
1752
|
+
startBlock,
|
|
1753
|
+
blockCount
|
|
1754
|
+
});
|
|
1755
|
+
await this.blockSink.addProposedCheckpoint({
|
|
1756
|
+
header: checkpoint.header,
|
|
1757
|
+
checkpointNumber: this.checkpointNumber,
|
|
1758
|
+
startBlock,
|
|
1759
|
+
blockCount,
|
|
1760
|
+
totalManaUsed: checkpoint.header.totalManaUsed.toBigInt(),
|
|
1761
|
+
feeAssetPriceModifier
|
|
1762
|
+
});
|
|
1763
|
+
}
|
|
1232
1764
|
/** Runs fee analysis and logs checkpoint outcome as fisherman */ async handleCheckpointEndAsFisherman(checkpoint) {
|
|
1233
1765
|
// Perform L1 fee analysis before clearing requests
|
|
1234
1766
|
// The callback is invoked asynchronously after the next block is mined
|
|
@@ -1268,20 +1800,8 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
|
|
|
1268
1800
|
}
|
|
1269
1801
|
return false;
|
|
1270
1802
|
}
|
|
1271
|
-
/** Waits until a specific time within the current slot */ async waitUntilTimeInSlot(targetSecondsIntoSlot) {
|
|
1272
|
-
const slotStartTimestamp = this.getSlotStartBuildTimestamp();
|
|
1273
|
-
const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
|
|
1274
|
-
await sleepUntil(new Date(targetTimestamp * 1000), this.dateProvider.nowAsDate());
|
|
1275
|
-
}
|
|
1276
1803
|
/** Waits the polling interval for transactions. Extracted for test overriding. */ async waitForTxsPollingInterval() {
|
|
1277
|
-
await
|
|
1278
|
-
}
|
|
1279
|
-
getSlotStartBuildTimestamp() {
|
|
1280
|
-
return getSlotStartBuildTimestamp(this.slot, this.l1Constants);
|
|
1281
|
-
}
|
|
1282
|
-
getSecondsIntoSlot() {
|
|
1283
|
-
const slotStartTimestamp = this.getSlotStartBuildTimestamp();
|
|
1284
|
-
return Number((this.dateProvider.now() / 1000 - slotStartTimestamp).toFixed(3));
|
|
1804
|
+
await this.awaitInterruptibleSleep(TXS_POLLING_MS);
|
|
1285
1805
|
}
|
|
1286
1806
|
getPublisher() {
|
|
1287
1807
|
return this.publisher;
|