Dict all 3 3 - #213
Draft
stutiibm wants to merge 27 commits into
Draft
Conversation
Refactor installation commands to remove unnecessary sudo usage and streamline the installation of dependencies for Python versions. Update CVE scan handling to improve clarity and functionality.
Refactor read_buildinfo.sh to improve readability and structure. Added a new function to read the 'Tested on' value from build scripts and updated the handling of build_script to support both string and array formats.
Added logic to determine version based on changed .sh files.
…able.sh variable.sh always exports BUILD_SCRIPT and TESTED_ON for the first/backward- compat script (UBI8), clobbering the correct per-job values set in the GHA env: block before build_wheels.sh runs. This caused all wheel jobs (ubi9_*, ubi10_*) to pull the UBI8 image and run the UBI8 build script. Fix: in every wheel Run build_wheels.sh step, capture THIS_SCRIPT and THIS_TESTED_ON from the GHA env before sourcing variable.sh, then re-export them as BUILD_SCRIPT and TESTED_ON after the source calls. Applied to all 11 wheel_build_ubiN_pyXXX jobs.
…ariable.sh All 12 wheel_build_ubiN_pyXXX jobs (ubi8: py310-313, ubi9: py310-314, ubi10: py312-314) now capture the GHA job-level BUILD_SCRIPT and TESTED_ON before sourcing variable.sh (which overwrites them with the UBI8 backward- compat values), then restore them after. This ensures each wheel job uses the correct UBI image and build script for its assigned UBI version.
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
1 participant
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.
Checklist
set -eoption enabled and observe success ?