Test post-release PostgreSQL images - #8721
Conversation
Point main CI at the development images from the-process PR #236 for PostgreSQL 16.14, 17.10, and 18.4. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions CI workflow to validate post-release container images for PostgreSQL 16.14, 17.10, and 18.4, while temporarily pinning CI to the -dev-f47a379 image suffix to exercise newly published tags.
Changes:
- Bump PostgreSQL versions used by CI images to 16.14 / 17.10 / 18.4.
- Switch CI image suffix to
-dev-f47a379for the two-phase validation plan. - Update the upgrade tester composite version string to match the bumped versions.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8721 +/- ##
==========================================
- Coverage 88.70% 88.69% -0.01%
==========================================
Files 289 289
Lines 64888 64888
Branches 8180 8180
==========================================
- Hits 57559 57553 -6
- Misses 4966 4970 +4
- Partials 2363 2365 +2 🚀 New features to boost your workflow:
|
Replace the development image suffix with the stable suffix published from the-process merge e11b596. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: dfb89c55-cd36-46b4-b19e-51307cbaf7d3
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
.github/workflows/build_and_test.yml:40
- The PR description says CI should be pointed at the
-dev-f47a379images for the first validation phase, but the workflow setsimage_suffixto-ve11b596. This makes the workflow run against a different set of images than described (and may skip the intended two-phase validation). Updateimage_suffixto the described dev suffix (or adjust the PR description if-ve11b596is the intended target).
sql_snapshot_pg_version: "18.4"
image_suffix: "-ve11b596"
pg16_version: '{ "major": "16", "full": "16.14" }'
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
.github/workflows/build_and_test.yml:39
- PR description says CI should point at
-dev-f47a379images during the two-phase validation, but the workflow currently setsimage_suffixto-ve11b596. This makes the workflow test a different image set than described (and may skip the intended dev-tagged images).
image_suffix: "-ve11b596"
DESCRIPTION: Validate post-release images for PostgreSQL 16.14, 17.10, and 18.4
Summary
-dev-f47a379images from Bump post-release Citus test versions the-process#236Two-phase validation
This PR intentionally uses the development suffix first. After CI validates these images and citusdata/the-process#236 merges, replace
-dev-f47a379with that merge's stable image suffix in this same PR.Validation
git diff --checkpasses