Skip to content

Zephyr 4.x workflow: stabilize CI for renamed forks and slashed branch names#10414

Merged
dgarske merged 1 commit intowolfSSL:masterfrom
night1rider:zephyr/4.x-workflow-fixes
May 6, 2026
Merged

Zephyr 4.x workflow: stabilize CI for renamed forks and slashed branch names#10414
dgarske merged 1 commit intowolfSSL:masterfrom
night1rider:zephyr/4.x-workflow-fixes

Conversation

@night1rider
Copy link
Copy Markdown
Contributor

The Zephyr 4.x CI workflow occasionally failed in two ways:
log file creation crashed when the branch name contained a / (e.g. feat/sbom-bomsh was interpreted as a subdirectory), and west update hung on a credential prompt when a PR's fork was renamed, deleted, or made private.

The workflow now fetches the wolfSSL revision via the upstream repo's refs/pull/<N>/head ref instead of pull_request.head.repo.clone_url.

The test script simplifies the log filename to <board>_<timestamp>.log so slashes in branch names can no longer break the path.

It also exports GIT_TERMINAL_PROMPT=0 before west update so any future auth issue fails fast with a clear error instead of the cryptic "could not read Username" hang.

@night1rider night1rider changed the title stabilize CI for renamed forks and slashed branch names Zepjyr 4.x workflow: stabilize CI for renamed forks and slashed branch names May 6, 2026
@night1rider night1rider changed the title Zepjyr 4.x workflow: stabilize CI for renamed forks and slashed branch names Zephyr 4.x workflow: stabilize CI for renamed forks and slashed branch names May 6, 2026
@night1rider night1rider self-assigned this May 6, 2026
@dgarske dgarske merged commit c38e6ca into wolfSSL:master May 6, 2026
363 of 364 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

MemBrowse Memory Report

No memory changes detected for:

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.

2 participants