-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Conversation
/gcbrun trial_build.py libaom hermes zstd --sanitizer coverage --fuzzing-engine libfuzzer |
/gcbrun oss_fuzz_on_demand.py libaom hermes zstd --sanitizer coverage --fuzzing-engine libfuzzer |
There was a problem hiding this 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 |
There was a problem hiding this comment.
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.
You can merge this now. |
infra/build/functions/gcb_test.py
Outdated
else: | ||
mock_trial_build_trial_build_main.assert_not_called() | ||
mock_oss_fuzz_on_demand.assert_called_once_with( | ||
test_case["expected_command"]) |
There was a problem hiding this comment.
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"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
Approved with nits |
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.ci_trial_build.py
renamed togcb.py
to reflect added functionality.Related to b/401215144 .