-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
infra: add proj name to env for fuzz introspector #7856
infra: add proj name to env for fuzz introspector #7856
Conversation
need to bump too when the fuzz-introspector PR is merged |
/gcbrun trial_build.py skcms --sanitizer address --fuzzing-engine libfuzzer |
Quick heads up -- the testing should fail as this line https://github.com/google/oss-fuzz/pull/7856/files#diff-43e6fb06733014791deb946732e00f7aa84e62bd1d856b363e5be922573a0e0dR243 depends on merging ossf/fuzz-introspector#327 -- i.e. the |
@Navidem updated now |
This is more to test regression on other builds. I expect this trial_build to succeed, as it is not an introspector build. |
Ah, sounds good |
Is it normal |
Yeah it should take more than 1 hour, but less than 2. You see no progress? what's the log/output and what about the processes running? Also make sure to have this update ossf/fuzz-introspector#328 -- am not 100% sure if it's needed but it could be. |
Went on successfully after removing the previous |
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
/gcbrun trial_build.py skcms --sanitizer address --fuzzing-engine libfuzzer |
* infra: add proj name to env for fuzz introspector Depends on ossf/fuzz-introspector#327 Ref: ossf/fuzz-introspector#317 * lint fix * update introspector commit Co-authored-by: Navidem <navid.emamdoost@gmail.com>
Depends on ossf/fuzz-introspector#327
Ref: ossf/fuzz-introspector#317