Skip to content

Pgbouncer config healthcheck#256

Merged
Nospamas merged 4 commits into
masterfrom
pgbouncer-config-healthcheck
Jun 10, 2026
Merged

Pgbouncer config healthcheck#256
Nospamas merged 4 commits into
masterfrom
pgbouncer-config-healthcheck

Conversation

@Nospamas

@Nospamas Nospamas commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

follows from #255.

  • Allows customisation of the Sqlalchemy connection pool for use with PGBouncer
  • Adds a healthcheck route with optional db check.

- Add devcontainer setup (Dockerfile, config, post-install scripts)
- Fix rasterio_thredds_helper to use vsicurl and convert dodsC to fileServer URLs
- Bump rasterio dependency to >=1.4
- Refactor open_nc to use context manager instead of try/finally
- Expand CI matrix to Python 3.11–3.14 with timeout and concurrency settings
- Tweak tests to work with simplified Thredds server urls
@Nospamas Nospamas force-pushed the pgbouncer-config-healthcheck branch 3 times, most recently from 93eb76e to 95fb886 Compare June 6, 2026 03:54
@Nospamas Nospamas requested a review from QSparks June 8, 2026 18:10

@QSparks QSparks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Nospamas Nospamas force-pushed the pgbouncer-config-healthcheck branch from 95fb886 to 31c0397 Compare June 8, 2026 22:14
@Nospamas Nospamas force-pushed the pgbouncer-config-healthcheck branch from 31c0397 to 943a942 Compare June 8, 2026 22:18
@Nospamas Nospamas merged commit a5fd9f0 into master Jun 10, 2026
6 checks passed
@Nospamas Nospamas deleted the pgbouncer-config-healthcheck branch June 10, 2026 20:40
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