Skip to content

Conversation

@ajsutton
Copy link
Contributor

Description

Support file: URLs to download prestates from.
Makes it possible to support multiple prestates without needing a HTTP server.

The source file URL is still treated as a remote source and a copy made into the data-dir. This isn't strictly necessary but it keeps the code reasonably simple and accommodates things like network file systems where caching may still be beneficial.

Tests

Added unit tests.

Builds on #12440 to avoid conflicts.

@ajsutton ajsutton requested review from a team as code owners October 14, 2024 03:59
@ajsutton ajsutton requested review from Inphi and removed request for a team October 14, 2024 03:59
@ajsutton ajsutton force-pushed the aj/support-file-prestates-url branch from b961a0d to a43f7fe Compare October 15, 2024 00:39
Makes it possible to support multiple prestates without needing a HTTP server.
@ajsutton ajsutton force-pushed the aj/support-file-prestates-url branch from a43f7fe to 19d0d4a Compare October 16, 2024 00:02
Base automatically changed from aj/prestates-url to develop October 16, 2024 00:14
@ajsutton ajsutton added this pull request to the merge queue Oct 16, 2024
Merged via the queue into develop with commit 9f24084 Oct 16, 2024
@ajsutton ajsutton deleted the aj/support-file-prestates-url branch October 16, 2024 01:12
samlaf pushed a commit to samlaf/optimism that referenced this pull request Nov 10, 2024
…um-optimism#12441)

* op-challenger: Introduce --prestates-url to specify prestates source for all game types at once.

* op-challenger: Add required config validation for asterisc kona.

Allow specify both prestate and prestates url to allow overriding. Prestate takes priority.

* op-challenger: Introduce vm-flag to make default with overrides pattern easier.

* op-challenger: Add vm type info to override flags.

* op-challenger: Support file: URLs to download prestates from.

Makes it possible to support multiple prestates without needing a HTTP server.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants