Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Jun 23, 2025
Merged

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

merged 17 commits into from
Jun 23, 2025

Conversation

samrose
Copy link
Collaborator

@samrose samrose commented Jun 18, 2025

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
Copy link

@laurenceisla laurenceisla left a comment

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
Copy link
Collaborator Author

samrose commented Jun 23, 2025

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 <[email protected]>
Co-authored-by: samrose <[email protected]>
Co-authored-by: Div Arora <[email protected]>
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