Skip to content

Commit

Permalink
fix(pageserver): unique test harness name for merge_in_between (#8366)
Browse files Browse the repository at this point in the history 
As title, there should be a way to detect duplicated harness names in
the future :(

Signed-off-by: Alex Chi Z <[email protected]>
  • Loading branch information
@skyzh
skyzh authored Jul 12, 2024
1 parent 411a130 commit 4184685
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pageserver/src/tenant/storage_layer/merge_iterator.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,7 @@ mod tests {
use crate::repository::Value;
use bytes::Bytes;

let harness = TenantHarness::create("merge_iterator_delta_merge").unwrap();
let harness = TenantHarness::create("merge_iterator_merge_in_between").unwrap();
let (tenant, ctx) = harness.load().await;

let tline = tenant
Expand Down

1 comment on commit 4184685

@github-actions
Copy link

Choose a reason for hiding this comment

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

3141 tests run: 3013 passed, 1 failed, 127 skipped (full report)

Failures on Postgres 14

  • test_basebackup_with_high_slru_count[github-actions-selfhosted-sequential-10-13-30]: release
# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_basebackup_with_high_slru_count[release-pg14-github-actions-selfhosted-sequential-10-13-30]"
Flaky tests (1)

Postgres 14

  • test_pageserver_lsn_wait_error_safekeeper_stop: debug

Code coverage* (full report)

  • functions: 32.7% (6983 of 21338 functions)
  • lines: 50.1% (54991 of 109719 lines)

* collected from Rust tests only

The comment gets automatically updated with the latest test results
4184685 at 2024-07-12T14:54:51.618Z :recycle:

Please sign in to comment.