test(pageserver): add small tenant compaction #11049
Conversation
Signed-off-by: Alex Chi Z <[email protected]>
7777 tests run: 7398 passed, 0 failed, 379 skipped (full report)Code coverage* (full report)
* collected from Rust tests only The comment gets automatically updated with the latest test results
7b02756 at 2025-03-03T17:23:46.832Z :recycle: |
skyzh
requested review from
problame and
arpad-m
and removed request for
problame
There was a problem hiding this comment.
Could you add a comment why we are enabling the compaction period and sleeping? Generally we don't like sleeps in our code as they bring unreliability.
For exceptional cases, it's okay I think but then we should make sure to document why the case is exceptional (for example, to test the compaction timer or something.
Signed-off-by: Alex Chi Z <[email protected]>
The test creates a very small tenant and see if everything works with it. Given that the amount of data written is small and the operation is short, I put a timeout here to let tasks run in the background (i.e., gc/compaction run using the timeout). But now I feel a better way is to simply use the compaction API, which doesn't require a timeout. Ready for review again :) |
Problem
close #10881
Summary of changes
Mock a tenant with very small amount of data.