mirror of
https://github.com/neondatabase/neon.git
synced 2025-12-22 21:59:59 +00:00
30a7dd630c
## Problem `TYPE_CHECKING` is used inconsistently across Python tests. ## Summary of changes - Update `ruff`: 0.7.0 -> 0.11.2 - Enable TC (flake8-type-checking): https://docs.astral.sh/ruff/rules/#flake8-type-checking-tc - (auto)fix all new issues
72 lines
2.5 KiB
Python
72 lines
2.5 KiB
Python
from __future__ import annotations
|
|
|
|
import threading
|
|
import time
|
|
from contextlib import closing
|
|
from typing import TYPE_CHECKING
|
|
|
|
import psycopg2.errors
|
|
from fixtures.log_helper import log
|
|
|
|
if TYPE_CHECKING:
|
|
from fixtures.neon_fixtures import NeonEnv, PgBin
|
|
|
|
|
|
# Test updating neon.pageserver_connstring setting on the fly.
|
|
#
|
|
# This merely changes some whitespace in the connection string, so
|
|
# this doesn't prove that the new string actually takes effect. But at
|
|
# least the code gets exercised.
|
|
def test_pageserver_reconnect(neon_simple_env: NeonEnv, pg_bin: PgBin):
|
|
env = neon_simple_env
|
|
env.create_branch("test_pageserver_restarts")
|
|
endpoint = env.endpoints.create_start("test_pageserver_restarts")
|
|
n_reconnects = 1000
|
|
timeout = 0.01
|
|
scale = 10
|
|
|
|
def run_pgbench(connstr: str):
|
|
log.info(f"Start a pgbench workload on pg {connstr}")
|
|
pg_bin.run_capture(["pgbench", "-i", "-I", "dtGvp", f"-s{scale}", connstr])
|
|
pg_bin.run_capture(["pgbench", f"-T{int(n_reconnects * timeout)}", connstr])
|
|
|
|
thread = threading.Thread(target=run_pgbench, args=(endpoint.connstr(),), daemon=True)
|
|
thread.start()
|
|
|
|
with closing(endpoint.connect()) as con:
|
|
with con.cursor() as c:
|
|
c.execute("SELECT setting FROM pg_settings WHERE name='neon.pageserver_connstring'")
|
|
connstring = c.fetchall()[0][0]
|
|
for i in range(n_reconnects):
|
|
time.sleep(timeout)
|
|
c.execute(
|
|
"alter system set neon.pageserver_connstring=%s",
|
|
(connstring + (" " * (i % 2)),),
|
|
)
|
|
c.execute("select pg_reload_conf()")
|
|
|
|
thread.join()
|
|
|
|
|
|
# Test handling errors during page server reconnect
|
|
def test_pageserver_reconnect_failure(neon_simple_env: NeonEnv):
|
|
env = neon_simple_env
|
|
env.create_branch("test_pageserver_reconnect")
|
|
endpoint = env.endpoints.create_start("test_pageserver_reconnect")
|
|
|
|
con = endpoint.connect()
|
|
cur = con.cursor()
|
|
|
|
cur.execute("set statement_timeout='2s'")
|
|
cur.execute("SELECT setting FROM pg_settings WHERE name='neon.pageserver_connstring'")
|
|
connstring = cur.fetchall()[0][0]
|
|
cur.execute(
|
|
f"alter system set neon.pageserver_connstring='{connstring}?some_invalid_param=xyz'"
|
|
)
|
|
cur.execute("select pg_reload_conf()")
|
|
try:
|
|
cur.execute("select count(*) from pg_class")
|
|
except psycopg2.errors.QueryCanceled:
|
|
log.info("Connection to PS failed")
|
|
assert not endpoint.log_contains("ERROR: cannot wait on socket event without a socket.*")
|