-
-
Notifications
You must be signed in to change notification settings - Fork 209
refactor(postgresq): switch to 'include_dir' and then rename conf files to ensure ordering #1820
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…dir' directive and move logging.conf into the postgresql-customer dir
… preserve ordering
* 'develop' of github.com:supabase/postgres: (29 commits) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1818) fix: update Dockerfiles for changes to postgis multiversion (#1817) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1813) feat: support multiple versions of the postgis extension (#1667) Update pgsql-http to handle semver (#1816) feat: add retry policy for auth service routes (#1782) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1811) fix: move tmpdir for SAA to one that always exists (#1799) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1810) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1809) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1807) build: automatically cancel old tests/build on new push (#1808) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1804) docs: small adjust (#1806) docs: getting started guide in README (#1805) fix: templating of SAA service file fix: adjust the qemu vars filename refactor(ansible): bring our ansible up to modern ansible-lint standards feat: support multiple versions of the http extension (#1664) feat: supply a slightly different supabase-admin-agent configuration for qemu artifacts ...
tgallacher
approved these changes
Oct 9, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks sane
hunleyd
added a commit
that referenced
this pull request
Oct 14, 2025
* origin/develop: fix(nix): remove '%' character from Nix trusted-public-keys configuration (#1840) chore: systemd hardening (#1837) Revert "refactor(postgresq): switch to 'include_dir' and then rename conf files to ensure ordering" (#1835) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1821) fix(pgmq): replace drop_queue function if exists (#1828) refactor(postgresq): switch to 'include_dir' and then renames conf files to ensure ordering (#1820) feat: support multiple versions of the pg_jsonschema extension (#1757) feat: bump auth to v2.180.0 (#1829) feat: update supautils confs w/ new tables feat(migrations): predefined role grants (#1815) feat: run pg_regress tests after installing the last version of the extension (#1826) fix(pgmq): add missing helper function in migration script (#1825) feat: support multiple versions of the pgmq extension (#1668) feat: run pg_regress during extension tests (#1812) feat: support multiple versions of the pgrouting extension (#1687) refactor(ansible): bring our ansible up to modern ansible-lint standards (#1819) feat: support multiple versions of the pg-graphql extension (#1761)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
Moved all config files that are included in the
postgresql.conf
into the/etc/postgresql-custom
dir then updatedpostgresql.conf
to useinclude_dir
instead ofinclude
. To ensure the same ordering of the various GUCs (to ensure proper overriding of values), rename files with digits prepended.Finally, split auto_explain and pg_cron out into their own config files instead of them being tacked on the end of the main PG config.