Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

local-cluster: Custom genesis size #2427

Merged

Conversation

ilya-bobyr
Copy link

Problem

Local cluster tests put additional accounts into the genesis, in order to speed up the setup phase. But as the genesis size is limited, certain setups can not be constructed without increasing the genesis size.

Summary of Changes

Allow configuration of the genesis size in the function and macros used by the local-cluster tests.

@ilya-bobyr ilya-bobyr marked this pull request as ready for review August 3, 2024 08:44
KirillLykov
KirillLykov previously approved these changes Aug 3, 2024
bw-solana
bw-solana previously approved these changes Aug 5, 2024
Copy link

@bw-solana bw-solana left a comment

Choose a reason for hiding this comment

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

LGTM. One nit

Some local-cluster tests may require larger than the default genesis, in
order to speed up the initial test setup.
@ilya-bobyr ilya-bobyr dismissed stale reviews from bw-solana and KirillLykov via e0e2d84 August 6, 2024 01:39
@ilya-bobyr ilya-bobyr force-pushed the pr/local-cluster-custom-genesis branch from 7dffe72 to e0e2d84 Compare August 6, 2024 01:39
@ilya-bobyr ilya-bobyr requested a review from bw-solana August 6, 2024 01:39
@ilya-bobyr ilya-bobyr merged commit 6668685 into anza-xyz:master Aug 7, 2024
42 checks passed
@ilya-bobyr ilya-bobyr deleted the pr/local-cluster-custom-genesis branch August 7, 2024 02:32
ray-kast pushed a commit to abklabs/agave that referenced this pull request Nov 27, 2024
Some local-cluster tests may require larger than the default genesis, in
order to speed up the initial test setup.
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.

3 participants