fix(release): use ref_name for gh run list branch so RUN_ID is resolved#870
Draft
fix(release): use ref_name for gh run list branch so RUN_ID is resolved#870
Conversation
Nightly release uses gh run list --branch= to get the latest z/OS build run. github.ref is refs/heads/main; gh CLI expects short branch name (main). Using refs/heads/main returns no runs, so RUN_ID was empty and gh run download grabbed the latest artifact from any branch (e.g. member-ispf-stats with old zowed naming), causing FSUM7351 when SDK runs ~/.zowe-server/zowex. Signed-off-by: Petr Plavjanik <petr.plavjanik@broadcom.com> Made-with: Cursor
|
Client artifacts: |
…s PR Signed-off-by: Petr Plavjanik <petr.plavjanik@broadcom.com> Made-with: Cursor
5 tasks
|
Member
|
The z/OS Build workflow is temporarily broken after our switch to Unfortunately this means that at the moment we have no way of building the ZRS binaries and packaging new releases with public infrastructure. I've disabled the z/OS Build workflow for the time being. As a backup, we should be able to build using internal CI for now - I'll follow up offline with more details. |
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.



Summary
Nightly SDK packages (e.g.
zowe-native-proto-sdk-0.3.0-2026-03-18-014027.tgz) were bundling a server PAX that contained the wrong binary names (zowed/libzowed.soinstead ofzowex). After install, the SDK runs~/.zowe-server/zowex server, so the shell reported FSUM7351 not found. The PAX also had stale build timestamps (e.g. March 6) instead of a recent main build.This change fixes the root cause: the nightly release workflow was resolving
RUN_IDwithgh run list --branch=${{ github.ref }}. For scheduled runs,github.refisrefs/heads/main, butgh run list --branch=refs/heads/mainreturns no runs (the CLI expects the short branch namemain). SoRUN_IDwas empty.What happened when RUN_ID was empty
The "Download Server" step runs:
With
RUN_IDempty, this becomesgh run download --name zowe-server-bin(and similarly for the second). Perghbehavior, that downloads the latest artifact with that name from any workflow run, not from a specific branch.The most recent such artifact was from run 22771182462 on branch
member-ispf-stats(March 6, 2026). That branch'sbuildTools.tsstill packaged the old layout:zowedandlibzowed.so. So the nightly was shipping a PAX from a feature branch with old naming instead of from a main z/OS build.Fix
Use
github.ref_nameinstead ofgithub.refwhen callinggh run list:github.ref=refs/heads/main→gh run listreturns nothing →RUN_IDempty.github.ref_name=main→gh run listreturns the latest z/OS build run formain→RUN_IDset correctly.So the nightly now resolves
RUN_IDfrom the main branch's z/OS build (when one exists).Verification
gh run list --branch=refs/heads/main --workflow=zos-build.yml --limit=1returns[], whilegh run list --branch=main --workflow=zos-build.yml --limit=1returns a run ID.main, that run's artifacts will be used for the SDK/server PAX.mainhas been failing/cancelled since around the ibm-clang migration (Migrate native C++ code to useibm-clangcompiler #812). Fixing those failures is separate; this PR only ensures the release workflow uses the correct run (main) when one exists, instead of falling back to the latest artifact from any branch.Follow-up
mainhas been failing (so nightly can ship a current server from main).Checklist
.github/workflows/release.yml(line 62).Made with Cursor