Lowers test's account data size to avoid hitting MaxLoadedAccountsDataSizeExceed #28814
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
@jeffwashington informed me that #28690 was failing in CI due to an error in
test_accounts_data_size_and_resize_transactions()
:https://buildkite.com/solana-labs/solana/builds/85042#01847837-100f-41b7-af5a-fe94c67358f6
Running locally, I observed this test failing intermittently will this error:
This test grows and shrinks accounts to ensure their data size is tracked correctly by the bank. The initial data size and the grow/shrink size are randomly generated, which explains how this test could fail intermittently.
Zooming in, we see the error is
MaxLoadedAccountsDataSizeExceeded
, which is new as of #27840.So
test_accounts_data_size_and_resize_transactions()
fails when the account size triggersMaxLoadedAccountsDataSizeExceeded
.Summary of Changes
Lower the bounds for the accounts size to ensure
MaxLoadedAccountsDataSizeExceeded
is not hit.