Skip to content

Conversation

jwnrt
Copy link

@jwnrt jwnrt commented Sep 2, 2025

Adds a CI workflow which runs a pull request's QEMU against the latest OpenTitan earlgrey_1.0.0 tests. This takes ~30 minutes at the moment.

There is a list of tests in the script which it compares against. The workflow fails if:

  • Expected tests fail (a regression has happened)
  • Unexpected tests pass (the list needs updating)

It runs on pull requests or can be manually run on any OpenTitan branch from the Actions tab.

@jwnrt jwnrt force-pushed the opentitan-bazel-tests branch 3 times, most recently from 36e5b04 to efd0fd4 Compare September 2, 2025 12:46
@jwnrt
Copy link
Author

jwnrt commented Sep 2, 2025

@loiclefort and @rivos-eblot: This is Earlgrey only at the moment, but we can do the same with Darjeeling once we get proper support in the OpenTitan repo.

Let me know what you think of this approach. I don't want to slow down your Darjeeling QEMU work with Earlgrey problems. It runs against the earlgrey_1.0.0 branch where the hardware is frozen, so I don't expect regressions on the OpenTitan side.

@jwnrt jwnrt marked this pull request as ready for review September 2, 2025 13:36
@rivos-eblot
Copy link

Let me know what you think of this approach. I don't want to slow down your Darjeeling QEMU work with Earlgrey problems. It runs against the earlgrey_1.0.0 branch where the hardware is frozen, so I don't expect regressions on the OpenTitan side.

This approach seems ok to me and a great move toward a real CI, thanks.

@rivos-eblot
Copy link

This takes ~30 minutes at the moment.
Might be worth considering a local runner at some point (?)

Copy link

@rivos-eblot rivos-eblot left a comment

Choose a reason for hiding this comment

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

Some suggestions

@jwnrt jwnrt force-pushed the opentitan-bazel-tests branch 3 times, most recently from 9b914b1 to b73a43b Compare September 2, 2025 14:45
Copy link

@AlexJones0 AlexJones0 left a comment

Choose a reason for hiding this comment

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

Thanks @jwnrt, this looks all good from my side! (apart from the CI failure from the unsorted tests)

@jwnrt jwnrt force-pushed the opentitan-bazel-tests branch from b73a43b to da28311 Compare September 2, 2025 15:31
Signed-off-by: James Wainwright <james.wainwright@lowrisc.org>
Signed-off-by: James Wainwright <james.wainwright@lowrisc.org>
@jwnrt jwnrt force-pushed the opentitan-bazel-tests branch from da28311 to e26c9fc Compare September 2, 2025 16:05
@jwnrt jwnrt merged commit d7a5696 into lowRISC:ot-earlgrey-9.2.0 Sep 2, 2025
8 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