Skip to content

Conversation

samrose
Copy link
Collaborator

@samrose samrose commented Aug 8, 2025

What kind of change does this PR introduce?

Build multiple versions of the pg_net extension to on different PostgreSQL versions.
Add test for the extensions and their upgrade on PostgreSQL 15 and 17. bumps version to 0.19.5 for testing as well

@samrose
Copy link
Collaborator Author

samrose commented Aug 8, 2025

testing pause/restore and other e2e scenarios now

@samrose samrose force-pushed the multi-version-ext/pg-net branch from f7e0208 to 90f8953 Compare August 11, 2025 21:33
@samrose
Copy link
Collaborator Author

samrose commented Aug 12, 2025

Just waiting on a branch of infra to test pause/restore dynamics on

@samrose samrose force-pushed the multi-version-ext/pg-net branch from 90f8953 to fac1f9b Compare August 13, 2025 16:15
@samrose samrose requested a review from jfroche August 15, 2025 20:27
@samrose samrose marked this pull request as ready for review August 15, 2025 20:27
@samrose samrose requested review from a team as code owners August 15, 2025 20:27
samrose and others added 2 commits August 18, 2025 17:54
limit using verison in extension creation to admin user for all extensions
by using supautils and before-create hooks
add test for this in pg_net
For pg_net background worker, we need to switch the version of the
extension on the file system (both pg_net.so and pg_net.control need to point
to the desired version).

As the extension is in the Nix store, we cannot simply symlink to the
desired version, as the Nix store is read-only. 
To work around this, we use overlayfs to create a writable layer on top
of the pg_net store path.
@jfroche jfroche force-pushed the multi-version-ext/pg-net branch from b83b127 to 91bbbb1 Compare August 18, 2025 22:27
This reverts commit 4996d81.

This change would not resolve the issue it was targeting.
@samrose samrose requested a review from jfroche August 22, 2025 13:35
Copy link
Contributor

@valigula valigula left a comment

Choose a reason for hiding this comment

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

LGTM

@samrose samrose merged commit 37e8528 into develop Aug 26, 2025
14 checks passed
@samrose samrose deleted the multi-version-ext/pg-net branch August 26, 2025 19:15
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.

6 participants