Skip to content

Fix CI Test Results workflow failing to find event.json#2555

Open
jbonofre wants to merge 1 commit intoapache:mainfrom
jbonofre:fix/ci-test-results-event-file
Open

Fix CI Test Results workflow failing to find event.json#2555
jbonofre wants to merge 1 commit intoapache:mainfrom
jbonofre:fix/ci-test-results-event-file

Conversation

@jbonofre
Copy link
Copy Markdown
Member

dawidd6/action-download-artifact@v20 with name_is_regexp creates per-artifact subdirectories, so the event file lands at event/event-file-ubuntu-24.04/event.json rather than event/event.json. Download only the ubuntu-24.04 artifact by exact name and update the event_file path accordingly.

dawidd6/action-download-artifact@v20 with name_is_regexp creates per-artifact
subdirectories, so the event file lands at event/event-file-ubuntu-24.04/event.json
rather than event/event.json. Download only the ubuntu-24.04 artifact by exact
name and update the event_file path accordingly.
with:
commit: ${{ github.event.workflow_run.head_sha }}
event_file: event/event.json
event_file: event/event-file-ubuntu-24.04/event.json
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A pity that event_file does not support wildcards... so we need to update this workflow file every time change the Ubuntu release.....

Maybe we could think about an alternative way: use an additional step to run
cp event/event-file-ubuntu-*/event.json event/

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good idea. We can also use wildcard (latest) on Ubuntu matrix.

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