Skip to content

[CI] Move yarpgen to standalone workflow#21731

Merged
KornevNikita merged 3 commits intosyclfrom
separate-yarpgen
Apr 13, 2026
Merged

[CI] Move yarpgen to standalone workflow#21731
KornevNikita merged 3 commits intosyclfrom
separate-yarpgen

Conversation

@KornevNikita
Copy link
Copy Markdown
Contributor

The idea is to make yarpgen a standalone workflow, so it'd possible to launch custom yarpgen runs with other than nightly builds (e.g. release builds).
In general it's a copy-paste of what we had before + some adjustments to make it work with specific artifacts.

@KornevNikita KornevNikita requested a review from a team as a code owner April 10, 2026 16:13
Copy link
Copy Markdown
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

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

only nits, you can merge if we need this soon and address later

if: inputs.toolchain_run_id != ''
env:
GH_TOKEN: ${{ github.token }}
run: |
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.

nit: is there a benefit to using gh run instead of the download artifact workflow we usually use?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes. We don't build the toolchain in this workflow, and the github action can only download artifacts from the same workflow. The idea is to make the workflow able to download artifacts from another runs, e.g. from nightlys.

run: |
cd yarpgen
# A system hits OOM when using all available threads.
python3 scripts/run_gen.py --target dpcpp --std sycl -t 60 -j 2 | tee yarpgen_run.log
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.

nit: do we need 2>&1 here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Nope, I don't think so

@KornevNikita KornevNikita merged commit aa71920 into sycl Apr 13, 2026
19 of 22 checks passed
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.

3 participants