Skip to content

fix: pin version of libpq to 17.x#1649

Merged
samrose merged 17 commits into
developfrom
sam/libpq-version-fix
Jun 23, 2025
Merged

fix: pin version of libpq to 17.x#1649
samrose merged 17 commits into
developfrom
sam/libpq-version-fix

Conversation

@samrose

@samrose samrose commented Jun 18, 2025

Copy link
Copy Markdown
Collaborator

What kind of change does this PR introduce?

The apt package system started sourcing libpq from an earlier version. This PR will attempt to pin the libpq version to 17.x version on all 3 variants of AMI

@samrose samrose changed the title fix: pin version of libpq to the major version installed fix: pin version of libpq to the major version installed or 15 Jun 18, 2025
@samrose
samrose force-pushed the sam/libpq-version-fix branch from 0bd772e to 9e1a7c6 Compare June 19, 2025 20:14
@samrose
samrose marked this pull request as ready for review June 19, 2025 20:31
@samrose
samrose requested review from a team as code owners June 19, 2025 20:31
@samrose
samrose requested a review from laurenceisla June 19, 2025 20:32
Comment thread testinfra/test_ami_nix.py Outdated
Comment thread testinfra/test_ami_nix.py Outdated

@laurenceisla laurenceisla 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.

Aside from those changes needed, it all LGTM.

@samrose
samrose requested a review from laurenceisla June 20, 2025 00:42
@samrose samrose changed the title fix: pin version of libpq to the major version installed or 15 fix: pin version of libpq to 17.x Jun 23, 2025
@samrose

samrose commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator Author

pg 17.x Read replica testing passed. Testing pg 15

@samrose
samrose merged commit 51ddf76 into develop Jun 23, 2025
24 of 25 checks passed
@samrose
samrose deleted the sam/libpq-version-fix branch June 23, 2025 21:22
Crispy1975 pushed a commit that referenced this pull request Jun 24, 2025
* fix: pin version of libpq to the major version installed

* fix: the variable is postgresql_major_version

* fix: using the correct var

* fix: default to postgres 15 major version

* fix: default to 15

* fix: supposedly main contains all releases for libpq5

* fix: syntax

* fix: different syntax

* fix: rely on ppa

* fix: ppa removal

* fix: change to trigger workflow

* fix: adding a test to see postgres version

* fix: testing for failure related to libpq5 version issue

* test: include read-only test for postgrest

* test: reduce log access time

* test: make sure to fail if version wrong

* chore: bump vars
Crispy1975 added a commit that referenced this pull request Jun 25, 2025
* feat: Install gandalf, salt-wrapper for infra

* chore: version bumps

* bumping oriole build also

* fix: create the Gandalf group

* fix: implicitly create the gandalf group

* fix: correct gandalf archive path

* fix: unpack gandalf build correctly

* fix: reorganise order for gandalf installs

* fix: point to the correct path

* chore: bump vars due to conflicts

* fix: pin version of libpq to 17.x (#1649)

* fix: pin version of libpq to the major version installed

* fix: the variable is postgresql_major_version

* fix: using the correct var

* fix: default to postgres 15 major version

* fix: default to 15

* fix: supposedly main contains all releases for libpq5

* fix: syntax

* fix: different syntax

* fix: rely on ppa

* fix: ppa removal

* fix: change to trigger workflow

* fix: adding a test to see postgres version

* fix: testing for failure related to libpq5 version issue

* test: include read-only test for postgrest

* test: reduce log access time

* test: make sure to fail if version wrong

* chore: bump vars

* ci: no longer build Oriole QEMU artifact

Said artifact is no longer being used. If needed in the future, it can
be trivially reintroduced, but in the meantime there's no reason to
expend resources on them.

* chore: disable auth config reloading within qemu artifact

The QEMU artifact is not running with a filesystem that supports the
inotify API, resulting in spurious error messages, and broken config
reloading.

* chore: strip out unnecessary dependencies from QEMU builds

Also upgrades to Alpine 3.22.

* chore: version bumps

* tests: update user and group test adding gandalf

* chore: add gandalf user to user list

---------

Co-authored-by: Crispy1975 <12525875+Crispy1975@users.noreply.github.com>
Co-authored-by: samrose <samuel@supabase.io>
Co-authored-by: Div Arora <root@darora.com>
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.

3 participants