[ci] Create cross-repo ci workflow for ot-sku - #30802
Conversation
|
LGTM, but do you have an example run I can look at? |
Here is an example: The build currently fails because the |
This workflow, which will triggered by ot-sku, runs the provisioning tests on FPGA (CW340), using the provided ot-sku repository as the provisioning extension. Signed-off-by: Amaury Pouly <amaury.pouly@opentitan.org>
|
|
||
| # For CI triggered by pull requests, cancel the previous run. | ||
| concurrency: | ||
| group: ot-sku-ci-${{ inputs.pull_request || inputs.branch || inputs.merge_sha }} |
There was a problem hiding this comment.
Very minor nit: I guess this might cause an issue if a pull request shares a branch name? I'm not sure what the exact format of the pull_request input looks like (is it just a number?)
There was a problem hiding this comment.
I think pull_request is a number, so you are theoretically correct. The probability of hitting this seems very very small though. If we wanted to fix, I guess we should use a different prefix for each type?
| execute_ot_sku_tests: | ||
| name: OT-SKU tests | ||
| needs: start | ||
| runs-on: [ubuntu-22.04-fpga, cw340] |
There was a problem hiding this comment.
Do we want to only run CW340 tests? I think there are some orchestrator tests that only support the CW310 right now (e2e_option_flags_test, e2e_ate_individ_test). Albeit perhaps these tests should instead be changed to support the CW340 😄
There was a problem hiding this comment.
Yes that's a good point. Maybe I can create a follow-up PR to add these?
4 participants
This workflow, which will be triggered by ot-sku, runs the provisioning tests on FPGA (CW340), using the provided ot-sku repository as the provisioning extension.
Used by lowRISC/ot-sku#14