Skip to content

perf: Speed up storage_reservation_test#9514

Merged
alin-at-dfinity merged 2 commits intomasterfrom
alin/speed-up-storage_reservation_test
Mar 23, 2026
Merged

perf: Speed up storage_reservation_test#9514
alin-at-dfinity merged 2 commits intomasterfrom
alin/speed-up-storage_reservation_test

Conversation

@alin-at-dfinity
Copy link
Copy Markdown
Contributor

The test ran with the default subnet memory capacity of 2 TB, meaning that the stable memory growth test required executing many hundreds of ingress messages (4 GB at a time). Plus, because we used the default 4 scheduler cores, each step down in page size required 4 up to allocations / stable_grow calls.

Reduce the memory capacity to 4 GB (threshold at 2 GB). And the number of scheduler cores to 2.

Test runtime drops from 50-100 seconds to under 1 second.

The test ran with the default subnet memory capacity of 2 TB, meaning that the stable memory growth test required executing many hundreds of ingress messages (4 GB at a time). Plus, because we used the default 4 scheduler cores, each stepdown in page size required 4 up to allocations / stable_grow calls.

Reduce the memory capacity to 4 GB (threshold at 2 GB). And the number of scheduler cores to 2.

Test runtime drops from 50-100 seconds to under 1 second.
@alin-at-dfinity alin-at-dfinity requested a review from a team as a code owner March 20, 2026 11:00
@github-actions github-actions bot added the fix label Mar 20, 2026
@alin-at-dfinity alin-at-dfinity changed the title fix: Speed up storage_reservation_test perf: Speed up storage_reservation_test Mar 20, 2026
@github-actions github-actions bot added perf and removed fix labels Mar 20, 2026
@alin-at-dfinity alin-at-dfinity added this pull request to the merge queue Mar 23, 2026
Merged via the queue into master with commit b597009 Mar 23, 2026
38 checks passed
@alin-at-dfinity alin-at-dfinity deleted the alin/speed-up-storage_reservation_test branch March 23, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants