Skip to content

Conversation

@hunleyd
Copy link
Contributor

@hunleyd hunleyd commented Nov 21, 2025

Moves auto_explain.log_min_duration setting from inline Dockerfile definitions and the main PostgreSQL configuration file to a dedicated auto_explain.conf file within the conf.d directory.

This change centralizes the configuration for better management and consistency across different PostgreSQL versions.

Updates PostgreSQL release versions in ansible/vars.yml with the project-specific suffix.

Moves `auto_explain.log_min_duration` setting from inline Dockerfile definitions and the main PostgreSQL configuration file to a dedicated `auto_explain.conf` file within the `conf.d` directory.

This change centralizes the configuration for better management and consistency across different PostgreSQL versions.

Updates PostgreSQL release versions in `ansible/vars.yml` with the project-specific suffix.
@hunleyd hunleyd self-assigned this Nov 21, 2025
The source path for PostgreSQL configuration templates has been simplified.

Previously, underscores in `ext_item` were removed when constructing the template filename. This change removes that transformation, directly using `{{ ext_item }}.conf`. This implies that template files in `files/postgresql_config/conf.d/` now match the `ext_item` loop variables directly (e.g., `auto_explain.conf`).
@hunleyd hunleyd marked this pull request as ready for review November 25, 2025 19:40
@hunleyd hunleyd requested review from a team as code owners November 25, 2025 19:40
Copy link
Contributor

@jchancojr jchancojr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hunleyd hunleyd added this pull request to the merge queue Nov 25, 2025
Merged via the queue into develop with commit a119df9 Nov 25, 2025
48 of 50 checks passed
@hunleyd hunleyd deleted the INDATA-255 branch November 25, 2025 22:11
hunleyd added a commit that referenced this pull request Dec 1, 2025
* origin/develop:
  chore: bump Wrappers version to 0.5.7 (#1945)
  chore: bump versions (#1949)
  Revert "docs: create an automated changlog file that shows up in release file…" (#1948)
  chore: remove precreated Storage objects (#1911)
  refactor: Move PostgreSQL auto_explain config to conf.d (#1943)
  docs: create an automated changlog file that shows up in release files (#1793)
  fix: removing dead nix code (#1946)
  docs: expansion to help understand anatomy and code api in user here (#1944)
  fix: try to use mirror fallbacks for docker images (#1942)
  feat: need this workflow present to test (#1941)
  feat: Add PostgreSQL conf.d directory (#1929)
  [jj-spr] initial version (#1866)
  chore: upgrade supabase-admin-agent (#1935)
  fix: garbage collect after every profile intall + remove osquery (#1938)
  fix(ci): limit max-jobs of nix to 8 to prevent OOM while running nix flake check (#1933)
  Revert "fix: disable temporary Nix checks on aarch64-linux" (#1934)
  fix: adaptive retry on aws (#1917)
  fix: disable temporary Nix checks on aarch64-linux (#1930)
  feat: release orioledb with rewind functionality (#1880)
  feat: reduciton of closure size for plv8 (#1925)
hunleyd added a commit that referenced this pull request Dec 1, 2025
* origin/develop:
  refactor: Refactor PostgreSQL cron config and deprecations (#1951)
  chore: bump Wrappers version to 0.5.7 (#1945)
  chore: bump versions (#1949)
  Revert "docs: create an automated changlog file that shows up in release file…" (#1948)
  chore: remove precreated Storage objects (#1911)
  refactor: Move PostgreSQL auto_explain config to conf.d (#1943)
  docs: create an automated changlog file that shows up in release files (#1793)
  fix: removing dead nix code (#1946)
  docs: expansion to help understand anatomy and code api in user here (#1944)
  fix: try to use mirror fallbacks for docker images (#1942)
  feat: need this workflow present to test (#1941)
  feat: Add PostgreSQL conf.d directory (#1929)
  [jj-spr] initial version (#1866)
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.

4 participants