Skip to content

[ci] Rework nightly FPGA jobs #27625

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 16, 2025
Merged

[ci] Rework nightly FPGA jobs #27625

merged 5 commits into from
Jul 16, 2025

Conversation

jwnrt
Copy link
Contributor

@jwnrt jwnrt commented Jul 14, 2025

The FPGA jobs in nightly are reworked to use our reusable workflows.

  • Every FPGA Bazel test is now executed.
  • Bitstreams are built if needed.
  • The Bazel cache is now available.

Normal CI only runs one version of each FPGA test. We run all execution environments of each test in nightlies to prevent regressions. Building bitstreams as-needed here lets (eventually) disable the non-HyperDebug bitstream build in regular CI, reducing load.

Example nightly run: https://github.com/lowRISC/opentitan/actions/runs/16312373557

@jwnrt jwnrt requested a review from pamaury July 14, 2025 14:52
@jwnrt jwnrt requested a review from rswarbrick as a code owner July 14, 2025 14:52
@jwnrt jwnrt force-pushed the nightly-build-bitstreams branch from c619490 to 2730ef7 Compare July 14, 2025 15:21
Copy link
Contributor

@pamaury pamaury left a comment

Choose a reason for hiding this comment

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

This looks good to me, although it would be good to have a second opinion, especially on the GHA front

@jwnrt jwnrt force-pushed the nightly-build-bitstreams branch from 2730ef7 to 72fc739 Compare July 15, 2025 09:10
@jwnrt jwnrt requested a review from nbdd0121 July 15, 2025 09:11
@jwnrt jwnrt force-pushed the nightly-build-bitstreams branch from 72fc739 to 44c0e5e Compare July 15, 2025 12:15
@jwnrt jwnrt requested a review from nbdd0121 July 15, 2025 12:18
@jwnrt jwnrt force-pushed the nightly-build-bitstreams branch from 44c0e5e to f9c62e3 Compare July 15, 2025 15:25
jwnrt added 4 commits July 15, 2025 22:31
All tests now support only the non-HyperDebug CW310 environment and
silicon owner environments. They are tagged with `pmod` in both cases
since the test requires some external hardware.

Signed-off-by: James Wainwright <[email protected]>
@jwnrt jwnrt force-pushed the nightly-build-bitstreams branch from f9c62e3 to c9503d7 Compare July 15, 2025 21:31
This job will now build bitstreams if they aren't cached.

Signed-off-by: James Wainwright <[email protected]>
@jwnrt jwnrt force-pushed the nightly-build-bitstreams branch from c9503d7 to 45b6279 Compare July 16, 2025 06:42
@jwnrt
Copy link
Contributor Author

jwnrt commented Jul 16, 2025

I have disabled using cached test results so the nightly will always run from scratch. I've adjusted the timeouts to accommodate. The longest job is ~3.5 hours with 4 hour timeout.

@jwnrt
Copy link
Contributor Author

jwnrt commented Jul 16, 2025

And the nightly run looks okay... I'm going to merge

@jwnrt jwnrt merged commit 3916346 into master Jul 16, 2025
89 of 92 checks passed
@jwnrt jwnrt deleted the nightly-build-bitstreams branch July 16, 2025 10:33
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