Skip to content

Commit

Permalink
pageserver: add no_sync for use in regression tests (2/2) (#9678)
Browse files Browse the repository at this point in the history
## Problem

Followup to #9677 which enables
`no_sync` in tests. This can be merged once the next release has
happened.

## Summary of changes

- Always run pageserver with `no_sync = true` in tests.
  • Loading branch information
jcsp authored Nov 13, 2024
1 parent 1ff5333 commit 7595d3a
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test_runner/fixtures/neon_fixtures.py
Original file line number Diff line number Diff line change
Expand Up @@ -1065,6 +1065,9 @@ def __init__(self, config: NeonEnvBuilder):
"http_auth_type": http_auth_type,
# Default which can be overriden with `NeonEnvBuilder.pageserver_config_override`
"availability_zone": "us-east-2a",
# Disable pageserver disk syncs in tests: when running tests concurrently, this avoids
# the pageserver taking a long time to start up due to syncfs flushing other tests' data
"no_sync": True,
}
if self.pageserver_virtual_file_io_engine is not None:
ps_cfg["virtual_file_io_engine"] = self.pageserver_virtual_file_io_engine
Expand Down

1 comment on commit 7595d3a

@github-actions
Copy link

Choose a reason for hiding this comment

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

5391 tests run: 5170 passed, 1 failed, 220 skipped (full report)


Failures on Postgres 17

  • test_ancestor_detach_branched_from[False-True-at]: debug-x86-64
# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_ancestor_detach_branched_from[debug-pg17-False-True-at]"

Test coverage report is not available

The comment gets automatically updated with the latest test results
7595d3a at 2024-11-13T10:08:47.031Z :recycle:

Please sign in to comment.