Skip to content

ci: reduce unit test CI time by running fuzz tests in a separate job#5716

Merged
rene merged 1 commit intolf-edge:masterfrom
shjala:move-fuzz-ci
Apr 14, 2026
Merged

ci: reduce unit test CI time by running fuzz tests in a separate job#5716
rene merged 1 commit intolf-edge:masterfrom
shjala:move-fuzz-ci

Conversation

@shjala
Copy link
Copy Markdown
Member

@shjala shjala commented Mar 30, 2026

Description

Fuzz tests were previously part of the main test target, causing unit test results to be blocked until all fuzz functions completed. Moving them into a dedicated fuzz-test Makefile target and a separate GHA workflow so unit tests can pass independently.

PR dependencies

N/A

How to test and validate this PR

N/A

Changelog notes

N/A

PR Backports

N/A

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've tested my PR on arm64 device
  • I've written the test verification instructions
  • I've set the proper labels to this PR

For backport PRs (remove it if it's not a backport):

  • I've added a reference link to the original PR
  • PR's title follows the template

And the last but not least:

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

Please, check the boxes above after submitting the PR in interactive mode.

@rene
Copy link
Copy Markdown
Contributor

rene commented Mar 30, 2026

@shjala , thank you for this change. Have you tested the new workflow in another repo?

@shjala
Copy link
Copy Markdown
Member Author

shjala commented Mar 31, 2026

@shjala , thank you for this change. Have you tested the new workflow in another repo?

@rene not really, fuzz-tests.yml is just a exact copy of go-tests.yml. It is passing green, should I tested elsewhere first?

Fuzz tests were previously part of the main `test` target, causing unit test
results to be blocked until all fuzz functions completed. Moving them into a
dedicated `fuzz-test` Makefile target and a separate GHA workflow so
unit tests can pass independently.

Signed-off-by: Shahriyar Jalayeri <shahriyar@posteo.de>
Copy link
Copy Markdown
Contributor

@christoph-zededa christoph-zededa left a comment

Choose a reason for hiding this comment

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

LGTM, with this we might even increase the timeout in the future

Copy link
Copy Markdown
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM

@rene rene merged commit 5a139fe into lf-edge:master Apr 14, 2026
43 of 55 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.

4 participants