Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: tolerate pageserver 500s in test_timeline_archival_chaos #9769

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

jcsp
Copy link
Collaborator

@jcsp jcsp commented Nov 15, 2024

Problem

Test exposes cases where pageserver gives 500 responses, causing failures like https://neon-github-public-dev.s3.amazonaws.com/reports/pr-9766/11844529470/index.html#suites/d1acc79950edeb0563fc86236c620898/3546be2ffed99ba6

Summary of changes

  • Tolerate such messages, and link an issue for cleaning up the pageserver not to return such 500s.

@jcsp jcsp added a/test Area: related to testing a/tech_debt Area: related to tech debt labels Nov 15, 2024
@jcsp jcsp requested a review from arpad-m November 15, 2024 12:21
@jcsp jcsp enabled auto-merge (squash) November 15, 2024 12:32
Copy link

5418 tests run: 5175 passed, 0 failed, 243 skipped (full report)


Flaky tests (2)

Postgres 16

Code coverage* (full report)

  • functions: 31.8% (7889 of 24841 functions)
  • lines: 49.4% (62447 of 126383 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
c209e10 at 2024-11-15T13:21:51.407Z :recycle:

@jcsp jcsp merged commit 04938d9 into main Nov 15, 2024
85 checks passed
@jcsp jcsp deleted the jcsp/archival-chaos-500s branch November 15, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a/tech_debt Area: related to tech debt a/test Area: related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants