Skip to content

Use tools v0.8.41 for packaging workflow dependency fixes - #1240

Merged
ibrahim halatci (ihalatci) merged 5 commits into
developfrom
ihalatci-image-health-check-fix
Oct 8, 2026
Merged

ibrahim halatci (ihalatci) merged 5 commits into
developfrom
ihalatci-image-health-check-fix

Conversation

@ihalatci

@ihalatci ibrahim halatci (ihalatci) commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #1241.

Update the test-tools clone reference from v0.8.40 to stable v0.8.41 in Image Health Check, Build Package, and Build package for test images. Stable tag verified at tools merge commit b55decf1ef0de2a18f9edb1249f2030bd5aee442, released after citusdata/tools#438 merged and its package-test rerun passed.

The shared tools fixes address pytest startup with platformdirs==4.12.1 and include the reviewed dependency-security updates. The newer runner environment appeared to mix ambient plugin metadata with old pinned platformdirs==4.2.2, producing ModuleNotFoundError: No module named 'platformdirs.pytest_plugin' in all 22 jobs of the original run; the exact hosted metadata source remains unconfirmed.

Net packaging change: exactly three tools-tag replacements. Original ambient pip/pytest commands, matrices, environments, test selection, and unrelated references remain unchanged; no venv workaround. Exact-content assertions and git diff --check passed.

Final dev-candidate validation on packaging ec16bf2f88d082352a743ca1f4c9a2568a251e5e passed all three workflows (22/22 matrix jobs and test steps each; 66/66 total), plus all 22 approved test-image push steps:

Stable-tag validation passed on exact packaging commit 544ecf84ebd5e389b90096cebca7258c8c6f1b1b (attempt 2): all three workflows have 22/22 successful matrix jobs and test steps (66/66 total), and all 22 approved test-image publication steps succeeded:

Attempt 1 passed 63/63 jobs that reached pytest; three AlmaLinux9 image-build failures occurred before tools clone/pytest (two EPEL metadata HTTP 404 failures and one devtoolset-8 installation failure). Rerunning only those three failed jobs succeeded without code changes.

Test-image publication is approved; production image pushes retain the non-develop branch guard. Review requests remain pending for serhat andic (@serhatandic) and Onur Tirtir (@onurctirtir). Keep this PR open; no packaging merge or further tag creation.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Restore ambient Python test commands and use v0.8.41-dev across packaging test workflows.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ihalatci ibrahim halatci (ihalatci) changed the title Isolate Image Health Check Python dependencies Validate shared platformdirs dependency fix with tools dev tag Oct 5, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ihalatci ibrahim halatci (ihalatci) changed the title Validate shared platformdirs dependency fix with tools dev tag Validate shared dependency fixes with tools security dev tag Oct 5, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ihalatci ibrahim halatci (ihalatci) changed the title Validate shared dependency fixes with tools security dev tag Validate final shared dependency updates with tools dev tag Oct 5, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ihalatci ibrahim halatci (ihalatci) changed the title Validate final shared dependency updates with tools dev tag Use tools v0.8.41 for packaging workflow dependency fixes Oct 8, 2026
@ihalatci
ibrahim halatci (ihalatci) merged commit 675ed6b into develop Oct 8, 2026
151 of 154 checks passed
@ihalatci
ibrahim halatci (ihalatci) deleted the ihalatci-image-health-check-fix branch October 8, 2026 10:02
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.

Image Health Check fails during pytest plugin loading

3 participants