ci: Split out native fuzz jobs for macOS and windows #31073

pull dergoegge wants to merge 1 commits into bitcoin:master from dergoegge:2024-10-native-fuzz changing 3 files +142 −2
  1. dergoegge commented at 2:51 pm on October 11, 2024: member

    Split out two new CI jobs (for native macOS and windows) that run the fuzz tests on the qa-assets input corpora.

    In both jobs the fuzz binary is built with -DBUILD_FOR_FUZZING to enable Assume assertions as well as FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION.

  2. [ci] Split out native fuzz jobs for macOS and windows 107a0df198
  3. DrahtBot commented at 2:51 pm on October 11, 2024: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #31158 (build, ci: Fix linking bitcoin-chainstate.exe to bitcoinkernel.dll on Windows by hebasto)
    • #31157 (Cleanups to port mapping module post UPnP drop by darosior)
    • #31130 (Drop miniupnp dependency by darosior)
    • #28710 (Remove the legacy wallet and BDB dependency by achow101)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. dergoegge marked this as ready for review on Oct 11, 2024
  5. DrahtBot added the label Tests on Oct 11, 2024
  6. maflcko commented at 3:32 pm on October 14, 2024: member
    Not sure about rushing this. Maybe wait for a fix for #31057 first, after which this may or may not be needed?
  7. maflcko commented at 11:30 am on October 28, 2024: member
    Can this be closed?
  8. dergoegge commented at 11:53 am on October 28, 2024: member
    I still think this would make sense but there seems to be no interest. Also #30950 has been resolved.
  9. dergoegge closed this on Oct 28, 2024

  10. ryanofsky referenced this in commit 03cff2c142 on Nov 5, 2024
  11. maflcko commented at 11:42 am on November 5, 2024: member
    This may be re-considered after a rebase. However, I wonder if the bloat can be reduced, assuming GHA has a matrix feature, so that the non-fuzz and fuzz-build can share most of the GHA config and steps?

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-21 15:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me