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

test: Make test_hot_standby_feedback more forgiving of slow initialization #9113

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

hlinnaka
Copy link
Contributor

Don't start waiting for the index to appear in the secondary until it has been created in the primary. Before, if the "pgbench -i" step took more than 60 s, we would give up.

There was a flaky test failure along those lines at: https://neon-github-public-dev.s3.amazonaws.com/reports/pr-9105/10997477941/index.html#suites/950eff205b552e248417890b8b8f189e/73cf4b5648fa6f74/ Hopefully, this avoids such failures in the future.

…ation

Don't start waiting for the index to appear in the secondary until it
has been created in the primary. Before, if the "pgbench -i" step took
more than 60 s, we would give up.

There was a flaky test failure along those lines at:
https://neon-github-public-dev.s3.amazonaws.com/reports/pr-9105/10997477941/index.html#suites/950eff205b552e248417890b8b8f189e/73cf4b5648fa6f74/
Hopefully, this avoids such failures in the future.
Copy link

github-actions bot commented Sep 23, 2024

5012 tests run: 4848 passed, 0 failed, 164 skipped (full report)


Flaky tests (4)

Postgres 17

Postgres 16

Postgres 14

Code coverage* (full report)

  • functions: 32.1% (7457 of 23225 functions)
  • lines: 49.9% (60092 of 120333 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
84e377f at 2024-09-24T01:02:43.291Z :recycle:

@hlinnaka hlinnaka merged commit 70fe007 into main Sep 24, 2024
83 checks passed
@hlinnaka hlinnaka deleted the make-test_hot_standby_feedback-more-forgiving branch September 24, 2024 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants