Repository navigation
Use tools v0.8.41 for packaging workflow dependency fixes - #1240
Merged
ibrahim halatci (ihalatci) merged 5 commits intoOct 8, 2026
Merged
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This was referenced Oct 3, 2026
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>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
ibrahim halatci (ihalatci)
requested review from
Onur Tirtir (onurctirtir) and
serhat andic (serhatandic)
October 5, 2026 15:11
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
ibrahim halatci (ihalatci)
enabled auto-merge (squash)
October 8, 2026 09:59
serhat andic (serhatandic)
approved these changes
Oct 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #1241.
Update the test-tools clone reference from
v0.8.40to stablev0.8.41in Image Health Check, Build Package, and Build package for test images. Stable tag verified at tools merge commitb55decf1ef0de2a18f9edb1249f2030bd5aee442, released after citusdata/tools#438 merged and its package-test rerun passed.The shared tools fixes address pytest startup with
platformdirs==4.12.1and include the reviewed dependency-security updates. The newer runner environment appeared to mix ambient plugin metadata with old pinnedplatformdirs==4.2.2, producingModuleNotFoundError: 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 --checkpassed.Final dev-candidate validation on packaging
ec16bf2f88d082352a743ca1f4c9a2568a251e5epassed 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.