tests: remove outdated metric check

Checking S3 metrics is too invasive for an integration test
in general anyway: it isn't a functional requirement that the
pagserver does a specific number of S3 ops while attaching
a tenant.
This commit is contained in:
John Spray
2023-10-17 20:18:02 +01:00
parent 9ba9965110
commit e4cd6cde4a

View File

@@ -277,13 +277,6 @@ def test_delete_timeline_exercise_crash_safety_failpoints(
if failpoint == "timeline-delete-after-index-delete":
m = ps_http.get_metrics()
assert (
m.query_one(
"remote_storage_s3_request_seconds_count",
filter={"request_type": "get_object", "result": "err"},
).value
== 2 # One is missing tenant deletion mark, second is missing index part
)
assert (
m.query_one(
"remote_storage_s3_request_seconds_count",