Skip to content

Conversation

@guy-starkware
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

guy-starkware commented Nov 23, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@guy-starkware guy-starkware force-pushed the guyn/l1provider/remove_error_handling_func branch from 63b3e91 to b7ff544 Compare November 23, 2025 14:18
@guy-starkware guy-starkware force-pushed the guyn/l1gas/add_test_for_low_number branch from 77f1085 to 159b40f Compare November 23, 2025 14:18
@guy-starkware guy-starkware force-pushed the guyn/l1gas/add_test_for_low_number branch from 159b40f to 2462126 Compare November 23, 2025 15:14
@guy-starkware guy-starkware force-pushed the guyn/l1provider/remove_error_handling_func branch from b7ff544 to 2679f3f Compare November 23, 2025 15:14
@guy-starkware guy-starkware force-pushed the guyn/l1gas/add_test_for_low_number branch from 2462126 to 772a4b0 Compare November 24, 2025 09:03
@guy-starkware guy-starkware force-pushed the guyn/l1provider/remove_error_handling_func branch from 2679f3f to 245a3cc Compare November 24, 2025 09:03
@guy-starkware guy-starkware force-pushed the guyn/l1provider/remove_error_handling_func branch from 245a3cc to bbf0ccb Compare December 2, 2025 11:06
@guy-starkware guy-starkware force-pushed the guyn/l1gas/add_test_for_low_number branch from 772a4b0 to 960a899 Compare December 2, 2025 11:06
Copy link
Collaborator

@ShahakShama ShahakShama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ShahakShama reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @guy-starkware)


crates/apollo_l1_provider/src/l1_scraper.rs line 261 at r1 (raw file):

            .map_err(L1ScraperError::BaseLayerError)?;

        let latest_l1_block = latest_l1_block

Chain these:
let latest_l1_block = self
.base_layer
.l1_block_at(latest_l1_block_number)
.await
.map_err(L1ScraperError::BaseLayerError)?
.ok_or(L1ScraperError::MissingBlock { block_number: latest_l1_block_number })?;


crates/apollo_l1_provider/src/l1_scraper.rs line 433 at r1 (raw file):

    FinalityTooHigh { finality: u64, latest_l1_block_no_finality: L1BlockNumber },
    #[error("Block number {block_number} not found")]
    MissingBlock { block_number: L1BlockNumber },

This error isn't very informative. The fact that we asked for this block is an internal logic of the scraper. Maybe MissingBlockReturnedFromLatestBlockNumber? BaseLayerInconsistency?

@guy-starkware guy-starkware changed the base branch from guyn/l1gas/add_test_for_low_number to graphite-base/10348 December 4, 2025 13:49
Copy link
Contributor Author

@guy-starkware guy-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @ShahakShama)


crates/apollo_l1_provider/src/l1_scraper.rs line 261 at r1 (raw file):

Previously, ShahakShama wrote…

Chain these:
let latest_l1_block = self
.base_layer
.l1_block_at(latest_l1_block_number)
.await
.map_err(L1ScraperError::BaseLayerError)?
.ok_or(L1ScraperError::MissingBlock { block_number: latest_l1_block_number })?;

Done.


crates/apollo_l1_provider/src/l1_scraper.rs line 433 at r1 (raw file):

Previously, ShahakShama wrote…

This error isn't very informative. The fact that we asked for this block is an internal logic of the scraper. Maybe MissingBlockReturnedFromLatestBlockNumber? BaseLayerInconsistency?

LatestL1BlockNumberNoBlockFound

@guy-starkware guy-starkware force-pushed the guyn/l1provider/remove_error_handling_func branch from bbf0ccb to 530dd2e Compare December 4, 2025 13:52
@guy-starkware guy-starkware changed the base branch from graphite-base/10348 to guyn/l1gas/add_test_for_low_number December 4, 2025 13:52
Copy link
Collaborator

@ShahakShama ShahakShama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@ShahakShama reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)

@guy-starkware guy-starkware changed the base branch from guyn/l1gas/add_test_for_low_number to main-v0.14.1 December 7, 2025 14:54
@github-actions
Copy link

github-actions bot commented Dec 7, 2025

Artifacts upload workflows:

Copy link
Contributor Author

@guy-starkware guy-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guy-starkware reviewed 1 of 1 files at r2, 4 of 4 files at r3, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)

@guy-starkware guy-starkware force-pushed the guyn/l1provider/remove_error_handling_func branch from 530dd2e to 12063fb Compare December 7, 2025 15:15
Copy link
Contributor Author

@guy-starkware guy-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guy-starkware reviewed 4 of 4 files at r4, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants