From 927587cec8d2807f895bdbf91b2a2132b525fe0c Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Tue, 21 Dec 2021 22:38:33 +0200 Subject: [PATCH] Fix comments in tests --- test_runner/batch_others/test_pageserver_restart.py | 5 ++--- test_runner/fixtures/zenith_fixtures.py | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/test_runner/batch_others/test_pageserver_restart.py b/test_runner/batch_others/test_pageserver_restart.py index a483450c54..05564390af 100644 --- a/test_runner/batch_others/test_pageserver_restart.py +++ b/test_runner/batch_others/test_pageserver_restart.py @@ -10,9 +10,8 @@ from fixtures.log_helper import log pytest_plugins = ("fixtures.zenith_fixtures") -# Check that dead minority doesn't prevent the commits: execute insert n_inserts -# times, with fault_probability chance of getting a wal acceptor down or up -# along the way. 2 of 3 are always alive, so the work keeps going. +# Test restarting page server, while safekeeper and compute node keep +# running. def test_pageserver_restart(zenith_env_builder: ZenithEnvBuilder): # One safekeeper is enough for this test. zenith_env_builder.num_safekeepers = 1 diff --git a/test_runner/fixtures/zenith_fixtures.py b/test_runner/fixtures/zenith_fixtures.py index da49e526f6..445532ff33 100644 --- a/test_runner/fixtures/zenith_fixtures.py +++ b/test_runner/fixtures/zenith_fixtures.py @@ -104,7 +104,7 @@ def pytest_configure(config): numprocesses = config.getoption('numprocesses') if numprocesses is not None and BASE_PORT + numprocesses * WORKER_PORT_NUM > 32768: # do not use ephemeral ports - raise Exception('Too many workers configured. Cannot distrubute ports for services.') + raise Exception('Too many workers configured. Cannot distribute ports for services.') # find the base directory (currently this is the git root) global base_dir