Skip to content

ci: archive the test log of failed verify builds - #1564

Open
joaodinissf wants to merge 1 commit into
masterfrom
ci/fix-test-log-upload
Open

joaodinissf wants to merge 1 commit into
masterfrom
ci/fix-test-log-upload

Conversation

@joaodinissf

Copy link
Copy Markdown
Collaborator

Why the change

Failed verify builds never kept the Eclipse test log: the upload path expanded to /com.avaloq.tools.ddk.xtext.test/... at the file-system root, so the step found nothing.

Special things to note

  • ${{ env.GITHUB_WORKSPACE }} is always empty: the env context only holds variables the workflow sets itself (env: or $GITHUB_ENV), not the runner's built-in ones. Seen on run 36723122740: "No files were found with the provided path: /com.avaloq.tools.ddk.xtext.test/target/work/data/.metadata/.log. No artifacts will be uploaded", and the run has no artifacts.
  • The relative path matches snapshot.yml, whose failure upload already works; upload-artifact resolves it against the workspace.
  • The step only runs when the build fails, so this PR's green build shows the corrected path: but cannot exercise the upload.

Change outline

 .github/workflows/verify.yml   (Archive Tycho Surefire Plugin)
-  path: ${{ env.GITHUB_WORKSPACE }}/com.avaloq.tools.ddk.xtext.test/target/work/data/.metadata/.log
+  path: com.avaloq.tools.ddk.xtext.test/target/work/data/.metadata/.log

🤖 Generated with Claude Code

The upload path started with ${{ env.GITHUB_WORKSPACE }}, but the env
context only holds variables the workflow sets itself, so it expanded to
nothing and the step looked for /com.avaloq.tools.ddk.xtext.test/...,
found no file and uploaded nothing. Use the path relative to the
workspace, as snapshot.yml does.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@joaodinissf
joaodinissf enabled auto-merge (rebase) September 30, 2026 16:20

This branch has not been deployed

No deployments
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.

1 participant