neondatabase / neon

Neon: Serverless Postgres. We separated storage and compute to offer autoscaling, code-like database branching, and scale to zero.
https://neon.tech
Apache License 2.0
13.2k stars 367 forks source link

Increase timeout for wating subscriner caught-up #8118

Open knizhnik opened 1 week ago

knizhnik commented 1 week ago

Problem

test_subscriber_restart has quit large failure rate' https://neonprod.grafana.net/d/fddp4rvg7k2dcf/regression-test-failures?orgId=1&var-test_name=test_subscriber_restart&var-max_count=100&var-restrict=false

I can be caused by too small timeout (5 seconds) to wait until changes are propagated.

Summary of changes

Increase timeout to 30 seconds.

Checklist before requesting a review

Checklist before merging

github-actions[bot] commented 1 week ago

3228 tests run: 3111 passed, 0 failed, 117 skipped (full report)


Flaky tests (2) #### Postgres 15 - `test_subscriber_restart`: [debug](https://neon-github-public-dev.s3.amazonaws.com/reports/pr-8118/9593865194/index.html#suites/8be0c222d5601535470e7e5978bbfb03/e5bda8926b448499/retries) #### Postgres 14 - `test_subscriber_restart`: [debug](https://neon-github-public-dev.s3.amazonaws.com/reports/pr-8118/9593865194/index.html#suites/8be0c222d5601535470e7e5978bbfb03/eacec62eea607a8f/retries)

Code coverage* (full report)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
d0b82bdf1b9d596480e63a0c6e41224b64f46ce3 at 2024-06-20T08:56:39.734Z :recycle: