Skip to content

Sync omnibus submodule: add external PostgreSQL 17 support - #4230

Open
jashaik wants to merge 1 commit into
mainfrom
shahid/pg-17-support
Open

jashaik wants to merge 1 commit into
mainfrom
shahid/pg-17-support

Conversation

@jashaik

@jashaik jashaik commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Widens external PostgreSQL support in preflight validation from PostgreSQL 14 to PostgreSQL 17, and syncs the omnibus (chef-server-omnibus-config) submodule pointer to include this change.

Changes Made

  • In chef-server-omnibus-config (submodule), bumped PostgresqlPreflightValidator::SUPPORTED_VERSION from 14 to 17 in files/server-ctl-cookbooks/infra-server/libraries/preflight_postgres_validator.rb, mirroring the pattern used for the prior PG14 change (commit 458ef0d).
  • Updated preflight_postgres_validator_spec.rb boundary tests: version 14.0 now warns (no longer at the max boundary), version 17.0 passes without failure, and versions > 17.x fail with CSPG014.
  • Updated the omnibus submodule pointer in this repo to the new commit containing the PG17 change.

The embedded PostgreSQL shipped with Chef Server remains unchanged at 13.x (default['private_chef']['postgresql']['version']); this only widens the range of external PostgreSQL major versions accepted without triggering a CSPG014 preflight failure.

Testing

  • Verified the version-comparison logic (major-version match against REQUIRED_VERSION/SUPPORTED_VERSION, warn vs fail boundaries) with a standalone Ruby script covering 9.6, 13.3, 14.0, 17.0, 18.0, and 8.100 — all produced expected outcomes (ok/warn/fail).
  • Local rspec run for the submodule spec was blocked by a pre-existing, unrelated macOS bcrypt native-extension code-signing issue in this environment.

This work was completed with AI assistance following Progress AI policies.

@jashaik
jashaik requested review from a team as code owners August 14, 2026 06:49
@jashaik
jashaik force-pushed the shahid/pg-17-support branch from 8465fdf to df0593a Compare August 19, 2026 07:28
talktovikas
talktovikas previously approved these changes Aug 25, 2026

@talktovikas talktovikas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

vkarve-chef
vkarve-chef previously approved these changes Aug 25, 2026
Signed-off-by: sreepuramsudheer <ssudheer@progress.com>
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