Revert "refactor(postgresq): switch to 'include_dir' and then rename conf files to ensure ordering"

[samrose]

Reverts #1820

Likely this change will also need the following

diff --git a/ansible/files/postgresql_config/supautils.conf.j2 b/ansible/files/postgresql_config/supautils.conf.j2
index 43875c19..da3a6562 100644
--- a/ansible/files/postgresql_config/supautils.conf.j2
+++ b/ansible/files/postgresql_config/supautils.conf.j2
@@ -1,3 +1,4 @@
+session_preload_libraries = 'supautils'
 supautils.extensions_parameter_overrides = '{"pg_cron":{"schema":"pg_catalog"}}'
 supautils.policy_grants = '{"postgres":["auth.audit_log_entries","auth.flow_state","auth.identities","auth.instances","auth.mfa_amr_claims","auth.mfa_challenges","auth.mfa_factors","auth.oauth_clients","auth.one_time_tokens","auth.refresh_tokens","auth.saml_providers","auth.saml_relay_states","auth.sessions","auth.sso_domains","auth.sso_providers","auth.users","realtime.messages","realtime.subscription","storage.buckets","storage.buckets_analytics","storage.objects","storage.prefixes","storage.s3_multipart_uploads","storage.s3_multipart_uploads_parts"]}'
 supautils.drop_trigger_grants = '{"postgres":["auth.audit_log_entries","auth.flow_state","auth.identities","auth.instances","auth.mfa_amr_claims","auth.mfa_challenges","auth.mfa_factors","auth.oauth_clients","auth.one_time_tokens","auth.refresh_tokens","auth.saml_providers","auth.saml_relay_states","auth.sessions","auth.sso_domains","auth.sso_providers","auth.users","realtime.messages","realtime.subscription","storage.buckets","storage.buckets_analytics","storage.objects","storage.prefixes","storage.s3_multipart_uploads","storage.s3_multipart_uploads_parts"]}'
diff --git a/ansible/tasks/internal/supautils.yml b/ansible/tasks/internal/supautils.yml
index 38e911f5..7dc813b3 100644
--- a/ansible/tasks/internal/supautils.yml
+++ b/ansible/tasks/internal/supautils.yml
@@ -32,13 +32,6 @@
     target: install
   become: yes
 
-- name: supautils - add supautils to session_preload_libraries
-  become: yes
-  replace:
-    path: /etc/postgresql/postgresql.conf
-    regexp: "#session_preload_libraries = ''"
-    replace: session_preload_libraries = 'supautils'
-
 - name: supautils - write custom 05-supautils.conf
   template:
     src: "files/postgresql_config/supautils.conf.j2"

reverting to unblock and will follow up to see if we can apply the above fix and get better config testing coverage too

[linear]

Issue reopened: INDATA-152 configure PG to use include_dir and move logging.conf to the dir

[Copilot]

Pull Request Overview

This PR reverts a previous change that switched PostgreSQL configuration files to use include_dir with numbered prefixes for file ordering. The revert restores the previous approach of using explicit include statements for individual configuration files without numerical prefixes.

• Removes the include_dir approach and returns to explicit include statements in postgresql.conf
• Removes numerical prefixes from configuration file names (e.g., 05-supautils.conf → supautils.conf)
• Consolidates auto_explain and pg_cron settings directly into the main postgresql.conf file

Reviewed Changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
nix/tools/run-server.sh.in	Reverts Nix build script to use individual config files without numbered prefixes
nix/packages/lib.nix	Updates Nix package configuration to remove references to deleted config files
ansible/tasks/*.yml	Updates Ansible tasks to use config files without numerical prefixes
ansible/files/postgresql_config/postgresql.conf.j2	Adds explicit include statements and moves auto_explain/pg_cron settings inline
ansible/files/postgresql_config/*.conf	Removes separate auto_explain and pg_cron config files
Dockerfile-*	Updates Docker builds to use config files without numerical prefixes

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}