Skip to content

CI-6141: Enable packaging for pxf-fdw7 - #58

Open
magf wants to merge 6 commits into
GreengageDB:mainfrom
magf:main
Open

CI-6141: Enable packaging for pxf-fdw7#58
magf wants to merge 6 commits into
GreengageDB:mainfrom
magf:main

Conversation

@magf

@magf magf commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Enable packaging for pxf-fdw7 (#58)

Task

  • add version 7 / ubuntu 22.04 entry to the PR build matrix
  • uncomment the deb-packages-pxf7 matrix entry in the Release workflow
  • remove dh-python build dependency, since no Python code is packaged
    and the dh-python helpers are never invoked

Additional changes (out of scope, done at reviewer's discretion)

  • improve observability for skipped packaging jobs: prefix job name
    with [SKIPPED], write a summary to GITHUB_STEP_SUMMARY, emit a
    workflow notice annotation
  • bump tests/install/deb action v44 to v45 (download-artifact v4 to
    v8, Node 24 support)
  • move the 6 / ubuntu 24.04 prod build from nightly matrix to PR
    matrix only: repeating it nightly against an unchanged image adds
    no value
  • invert build matrix condition so schedule triggers use the nightly
    matrix and everything else (PR/release) uses the PR matrix; skip
    the package job on schedule so nightly runs tests only

Fixes found during release verification

  • enable packaging for 6 / ubuntu 24.04: the skip_package flag was
    left over from the nightly matrix entry it was moved from
  • enable clobber on release asset upload, to prevent upload failures
    when pxf-cli and pxf-server assets already exist as duplicates
    across versions 6 and 7

Task: CI-6141

magf added 6 commits August 12, 2026 12:10
* ci: create pxf-fdw7 deb package

- enable package workflow for version 7
- uncomment deb-packages-pxf7 matrix entry in Release CI

* build(deb): remove dh-python dependency

No Python code is being packaged;
dh-python helpers are not invoked

* ci(package): improve observability for skipped packaging jobs

- Add `[SKIPPED]` prefix to the job name when `skip_package` is true
- Generate a Markdown report in `$GITHUB_STEP_SUMMARY` to provide clear context in the Actions UI
- Emit a `::notice` workflow annotation to highlight skipped matrix configurations

* ci(package): bump tests/install/deb v44 to v45

Update actions/download-artifact v4 to v8 - Node 24 support

* ci(build): move 24.04 prod build from nightly to PR only

There's no point in repeating the nightly build on not changed prod image

Task: GG-682
* ci: create pxf-fdw7 deb package

- enable package workflow for version 7
- uncomment deb-packages-pxf7 matrix entry in Release CI

* build(deb): remove dh-python dependency

No Python code is being packaged;
dh-python helpers are not invoked

* ci(package): improve observability for skipped packaging jobs

- Add `[SKIPPED]` prefix to the job name when `skip_package` is true
- Generate a Markdown report in `$GITHUB_STEP_SUMMARY` to provide clear context in the Actions UI
- Emit a `::notice` workflow annotation to highlight skipped matrix configurations

* ci(package): bump tests/install/deb v44 to v45

Update actions/download-artifact v4 to v8 - Node 24 support

* ci(build): move 24.04 prod build from nightly to PR only

There's no point in repeating the nightly build on not changed prod image

* fix(ci): use nightly matrix only on schedule trigger, skip package job on schedule

Invert build matrix condition: schedule -> nightly, everything else -> PR/release.
Add if: github.event_name != 'schedule' to package job so nightly runs only test

Task: CI-6141
Overwrite existing release assets to prevent upload failures
when pxf-cli and pxf-server already exist as duplicates
across versions 6 and 7
@magf
magf marked this pull request as ready for review August 12, 2026 11:37
@magf
magf requested a review from a team as a code owner August 12, 2026 11:37
@magf
magf enabled auto-merge (squash) August 12, 2026 11:44
@magf

magf commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

Tests

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.

1 participant