Skip to content

Commit

Permalink
tests: use tenant_create() helper in test_bulk_insert (#6064)
Browse files Browse the repository at this point in the history 
## Problem

Since #5449 we enable generations in tests by default. Running
benchmarks was missed while merging that PR, and there was one that
needed updating.

## Summary of changes

Make test_bulk_insert use the proper generation-aware helper for tenant
creation.
  • Loading branch information
@jcsp
jcsp authored Dec 7, 2023
1 parent e89e41f commit 880663f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test_runner/performance/test_bulk_insert.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ def measure_recovery_time(env: NeonCompare):
# when we "create" the Tenant again, we will replay the WAL from the beginning.
client.tenant_delete(env.tenant)
wait_tenant_status_404(client, env.tenant, iterations=60, interval=0.5)
client.tenant_create(new_tenant_id=env.tenant)
env.env.pageserver.tenant_create(tenant_id=env.tenant)

# Measure recovery time
with env.record_duration("wal_recovery"):
Expand Down

1 comment on commit 880663f

@github-actions
Copy link

Choose a reason for hiding this comment

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

2528 tests run: 2423 passed, 0 failed, 105 skipped (full report)

Flaky tests (4)

Postgres 16

  • test_timeline_deletion_with_files_stuck_in_upload_queue: release

Postgres 15

  • test_competing_branchings_from_loading_race_to_ok_or_err: release
  • test_gc_cutoff: release

Postgres 14

Code coverage (full report)

  • functions: 54.8% (9385 of 17112 functions)
  • lines: 82.3% (54247 of 65947 lines)
The comment gets automatically updated with the latest test results
880663f at 2023-12-07T15:50:07.838Z :recycle:

Please sign in to comment.