Skip to content

ci: avoid using pouchdb-server in testing#9202

Closed
espy wants to merge 1 commit intoapache:masterfrom
neighbourhoodie:ci-avoid-pouchdb-server
Closed

ci: avoid using pouchdb-server in testing#9202
espy wants to merge 1 commit intoapache:masterfrom
neighbourhoodie:ci-avoid-pouchdb-server

Conversation

@espy
Copy link
Contributor

@espy espy commented Mar 25, 2026

Destroying databases is flaky in pouchdb-server

Overview

PouchDB integration tests were very flaky, with random timeouts. We (NH) identified this being caused by an issue with pouchdb-server, where the destroy() method used to clean up test DBs didn’t actually remove the test databases. The tests no longer seem to time out when using a CouchDB via COUCHD_HOST instead of pouchdb-server. This PR changes the CI setup so we can verify this assumption by running the CI repeatedly.

We moved the node PERF tests out into their own nodejs-perf Job , since our changes conflicted with that particular bit of the node job.

We’re also trying out randomly selecting Ubuntu mirrors other than the terribly slow Azure one, see actions/runner-images#7048 (comment)

Testing recommendations

We want to re-run the test suite multiple times to se whether the timeouts are resolved, and if so, submit this as a fix.

Checklist

  • I am not a bot
  • This is my own work, I did not use AI, LLM's or similar technology for code or docs generation
  • Code is written and works correctly
  • Changes are covered by tests
  • Documentation changes were made in the docs folder

@espy espy force-pushed the ci-avoid-pouchdb-server branch 7 times, most recently from 0fe3883 to 9121490 Compare March 25, 2026 16:12
@espy espy marked this pull request as draft March 25, 2026 16:22
Destroying databases is flaky in pouchdb-server
@espy espy force-pushed the ci-avoid-pouchdb-server branch from 9121490 to ae896c8 Compare March 25, 2026 16:42
@espy
Copy link
Contributor Author

espy commented Mar 26, 2026

So this all seems to work very well now, but the Github PR UI is stuck with a pending test that no longer exist because we renamed a command. The tests all pass in CI, but because we want the PR to be green too, I’m going to close this and open a new one on the same branch.

@espy
Copy link
Contributor Author

espy commented Mar 26, 2026

New PR is #9204

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.

1 participant