Skip to content

ci: fix error failing atheris-libprotobuf-mutator build #4818

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 2 commits into from
Feb 26, 2025

Conversation

its403
Copy link
Contributor

@its403 its403 commented Feb 14, 2025

Summary:

Fix failing atheris-libprotobuf-mutator build in GitHub Actions by adding necessary C++ build dependencies. While the package builds successfully in local environments, it failed in GitHub Actions even after downgrading to Bazel 7.4.1. Added build-essential, GCC 10, and proper compiler configurations to ensure consistent builds across environments.

Changes:

  • Install build-essential package with gcc, g++ and cmake
  • Add explicit compiler environment variable setup
  • Install and configure GCC 10 for better C++ feature support
  • Enable verbose pip installation for better debugging

uses: actions/[email protected]
with:
python-version: 3.9
python-version: 3.9
Copy link
Contributor

Choose a reason for hiding this comment

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

Does Bazel need 3.9 or should we consider upgrading this to 3.13 as we've done with some of our other workflows?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Atheris is only supported till python3.11 version, but atheris-libprotobuf-mutator is throwing error on 3.11 version. I've tested python3.10 so I'm changing version to 3.10.16

Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

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

Merge time! I'm going to try kicking off a manual run of the fuzzer after this merges so we can see if it is actually behaving now. Thank you!

@terriko terriko merged commit 45b9ec3 into intel:main Feb 26, 2025
24 checks passed
@terriko
Copy link
Contributor

terriko commented Feb 26, 2025

Run logs will be here when they finish: https://github.com/intel/cve-bin-tool/actions/runs/13550667833

@terriko
Copy link
Contributor

terriko commented Feb 26, 2025

For the record: looks like it is indeed working as expected -- ran for an hour before termination. I see some cache errors we might want to look into later, but I think we're in business again as far as fuzzing goes!

@its403 its403 deleted the fix-fuzz-jobs branch March 3, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants