Skip to content
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

Enable OSS-Fuzz on Demand via PR Comments. #13120

Merged
merged 10 commits into from
Apr 1, 2025

Conversation

decoNR
Copy link
Collaborator

@decoNR decoNR commented Mar 7, 2025

Adds support for triggering OSS-Fuzz on Demand runs via /gcbrun oss_fuzz_on_demand.py <projects> --sanitizer <sanitizer> --fuzzing-engine <fuzzing_engine> commands in PR comments.

  • Parses PR comments for OSS-Fuzz on Demand commands.
  • ci_trial_build.py renamed to gcb.py to reflect added functionality.

Related to b/401215144 .

@decoNR
Copy link
Collaborator Author

decoNR commented Mar 7, 2025

/gcbrun trial_build.py libaom hermes zstd --sanitizer coverage --fuzzing-engine libfuzzer

@decoNR
Copy link
Collaborator Author

decoNR commented Mar 10, 2025

/gcbrun oss_fuzz_on_demand.py libaom hermes zstd --sanitizer coverage --fuzzing-engine libfuzzer

Copy link
Contributor

@jonathanmetzman jonathanmetzman 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 nits

@@ -0,0 +1,33 @@
# Copyright 2022 Google LLC
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the copyright year is wrong on these.

@jonathanmetzman
Copy link
Contributor

You can merge this now.

else:
mock_trial_build_trial_build_main.assert_not_called()
mock_oss_fuzz_on_demand.assert_called_once_with(
test_case["expected_command"])

Choose a reason for hiding this comment

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

Could you just add this endline here?

ENTRYPOINT ["python3", "/opt/oss-fuzz/infra/build/functions/gcb.py"]

Choose a reason for hiding this comment

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

same

@javanlacerda
Copy link

Approved with nits

@javanlacerda javanlacerda merged commit 35af737 into master Apr 1, 2025
19 checks passed
@javanlacerda javanlacerda deleted the oss_fuzz_on_demand_trigger branch April 1, 2025 20:40
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