diff --git a/test_runner/regress/test_tenant_detach.py b/test_runner/regress/test_tenant_detach.py index 9a0f7f7313..bafddc7721 100644 --- a/test_runner/regress/test_tenant_detach.py +++ b/test_runner/regress/test_tenant_detach.py @@ -120,8 +120,6 @@ def test_detach_while_attaching( tenant_id = TenantId(pg.safe_psql("show neon.tenant_id")[0][0]) timeline_id = TimelineId(pg.safe_psql("show neon.timeline_id")[0][0]) - checkpoint_numbers = range(1, 3) - # Create table, and insert some rows. Make it big enough that it doesn't fit in # shared_buffers, otherwise the SELECT after restart will just return answer # from shared_buffers without hitting the page server, which defeats the point @@ -143,11 +141,11 @@ def test_detach_while_attaching( # run checkpoint manually to be sure that data landed in remote storage pageserver_http.timeline_checkpoint(tenant_id, timeline_id) - log.info(f"waiting for upload") + log.info("waiting for upload") # wait until pageserver successfully uploaded a checkpoint to remote storage wait_for_upload(client, tenant_id, timeline_id, current_lsn) - log.info(f"upload is done") + log.info("upload is done") # Detach it pageserver_http.tenant_detach(tenant_id) @@ -166,6 +164,6 @@ def test_detach_while_attaching( # Attach it again. If the GC and compaction loops from the previous attach/detach # cycle are still running, things could get really confusing.. pageserver_http.tenant_attach(tenant_id) - + with pg.cursor() as cur: - cur.execute("SELECT COUNT(*) FROM foo"); + cur.execute("SELECT COUNT(*) FROM foo") diff --git a/test_runner/regress/test_tenants.py b/test_runner/regress/test_tenants.py index f6d0b5f31f..0b20afefc3 100644 --- a/test_runner/regress/test_tenants.py +++ b/test_runner/regress/test_tenants.py @@ -259,4 +259,6 @@ def test_pageserver_with_empty_tenants( ), f"Tenant {tenant_without_timelines_dir} without timelines dir should be broken" [loaded_tenant] = [t for t in tenants if t["id"] == str(tenant_with_empty_timelines_dir)] - assert loaded_tenant["state"] == "Active", "Tenant {tenant_with_empty_timelines_dir} with empty timelines dir should be active and ready for timeline creation" + assert ( + loaded_tenant["state"] == "Active" + ), "Tenant {tenant_with_empty_timelines_dir} with empty timelines dir should be active and ready for timeline creation"