fix python linter complaints

This commit is contained in:
Christian Schwarz
2022-11-24 07:36:23 -05:00
committed by Christian Schwarz
parent 1022b4b98f
commit c2f5d011c7
2 changed files with 7 additions and 7 deletions

View File

@@ -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")

View File

@@ -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"