Skip to content

Moved PostgreSQL to WORKDIR/PostgreSQL/<psql-major-version> to help with hub package upgrades migration - #1510

Closed
craigcomstock wants to merge 2 commits into
cfengine:masterfrom
craigcomstock:ENT-12383-2/master
Closed

Moved PostgreSQL to WORKDIR/PostgreSQL/<psql-major-version> to help with hub package upgrades migration#1510
craigcomstock wants to merge 2 commits into
cfengine:masterfrom
craigcomstock:ENT-12383-2/master

Conversation

@craigcomstock

Copy link
Copy Markdown
Contributor

pg_upgrade will work better if we have a major-version specific directory in which libraries are contained.

Without this we end up with commands like psql linked to /var/cfengine/lib/libpq.so in both old and new versions during upgrades.
libpq.so can easily change in a breaking way and no amount of LD_LIBRARY_PATH can fix pg_upgrade because it checks BOTH old and new bin dirs with the same execution and has not argument to specify a different library dir for old and new versions.

Ticket: ENT-12383
Changelog: title

…ith hub package upgrades migration

pg_upgrade will work better if we have a major-version specific directory in which libraries are contained.

Without this we end up with commands like psql linked to /var/cfengine/lib/libpq.so in both old and new versions during upgrades.
libpq.so can easily change in a breaking way and no amount of LD_LIBRARY_PATH can fix pg_upgrade because it checks BOTH old and new bin dirs with the same execution and has not argument to specify a different library dir for old and new versions.

Ticket: ENT-12383
Changelog: title
@craigcomstock
craigcomstock marked this pull request as draft October 15, 2024 15:52
@craigcomstock

Copy link
Copy Markdown
Contributor Author

Build Status

@craigcomstock

Copy link
Copy Markdown
Contributor Author

Build Status

@craigcomstock

Copy link
Copy Markdown
Contributor Author

prefer #1511

@craigcomstock
craigcomstock deleted the ENT-12383-2/master branch October 15, 2024 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant