fix(e2e): avoid parsing stage logs as final artifact paths - #1427
Merged
Merged
Conversation
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
fangyangci
enabled auto-merge (squash)
September 18, 2026 02:39
Yue Sun (KayMKM)
approved these changes
Sep 18, 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.
Summary
Final artifact:orExisting artifact found:, preferring the final artifact over reused or intermediate outputs.OSError/ValueErrorwhen checking candidate files.Root Cause
The wrapped-path reconstruction introduced in #1411 also parsed stage-level
Artifact:lines. Those lines include a size suffix, such as..._export.onnx (335.3 MB), so the.endswith(".onnx")stop condition never matched. Subsequent warning and Optimize output was appended to the candidate filename. On the affected Windows agent,Path.is_file()raisedWinError 1113after the build subprocess had already exited successfully.Stage artifacts are not necessarily the deployment-ready model and must not be selected as a fallback for the final build summary.
Validation
git diff --checkpassed.openai/clip-vit-base-patch32with QNN package 2.2480.49.0, matching the version reported in the failing logs:All three runs used
--eval-type perf --timeout 1800 --raw-outputwith isolated temporary output/cache directories. Builds exercised the stage-output parsing path, and all eight component perf subprocesses exited 0. No path-parser exceptions occurred. Saved result JSON confirms all five jobs passed.This validates the perf workflows from the three reported failures; accuracy was not rerun. The portable unit regression uses actual Rich output and injects the affected Windows filesystem exception; the hardware E2E reruns use real build/perf subprocesses. Temporary models, raw logs, and historical evaluation results are not part of this PR.