ci: zephyr: fix Linux/Windows artifact comparison - #11149
Conversation
Preserve the complete artifact suffix when deriving Linux artifact names from Windows artifact names. Fixes thesofproject#11148 Signed-off-by: Piotr Hoppe <piotr.hoppe@intel.com>
There was a problem hiding this comment.
Pull request overview
This PR fixes the compare-linux-win job in the Zephyr CI workflow by correctly deriving the corresponding Linux artifact directory name from each Windows artifact directory name, preserving the full suffix so directory comparisons work reliably again (per #11148).
Changes:
- Adjust artifact directory name mapping logic from
windows-build*tolinux-build*by stripping the fullwindows-buildprefix and prependinglinux-build. - Add an inline comment clarifying the intended prefix transformation.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
added a comment #11148 (comment) to the bug to explain what actually should be looked at. |
PR 11149: test resultsRun date: 2026-09-01 14:50 UTC Tested commit: 75f7997830b04297b68449015b5a4276d21a66f3
|
75f7997 to
3661a33
Compare
kv2019i
left a comment
kv2019i
left a comment
There was a problem hiding this comment.
The test is not passing yet. Marking -1 just in case this doesn't get merged before the test passes.
|
Seems #11160 helps with the issue |
Preserve the complete artifact suffix when deriving Linux artifact names from Windows artifact names.
Fixes #11148