[opendal] fix build script and update repo URL#15648
Conversation
- build.sh: copy each fuzz binary to its correct output name instead of duplicating fuzz_reader to all slots; add new fuzz_path target; remove unused .env files since fuzz targets now fall back to a local fs operator when no service is configured (apache/opendal#7723) - Dockerfile: update clone URL from incubator-opendal to opendal - project.yaml: update main_repo URL; add tonghuaroot to auto_ccs
|
tonghuaroot is a new contributor to projects/opendal. The PR must be approved by known contributors before it can be merged. The past contributors are: hunsche, alexcrichton, dqhl76 |
|
can you let us know when the upstream is ready to land this? |
|
Hi @DavidKorczynski — the upstream piece is apache/opendal#7723. It adds the |
| - "dqhl76@gmail.com" | ||
| - "ouyangjun1999@gmail.com" | ||
| - "suyanhanx@gmail.com" | ||
| - "tonghuaroot@gmail.com" |
There was a problem hiding this comment.
Are you a maintainer? If not, can you please get approval from maintainers that you can add your email or just remove your email from the list?
2 participants
Summary
The OpenDAL fuzz integration has several issues that prevent effective fuzzing:
fuzz_readerto all output slots —fuzz_writerbinary was never deployedincubator-opendalwas renamed toopendal.envfiles — fuzz targets now fall back to a local fs operator automatically (see fix(fuzz): fall back to fs operator so fuzz targets produce real coverage apache/opendal#7723), so the env-based service configuration is no longer neededChanges
build.sh: loop over actual target names and copy each binary correctly; add newfuzz_pathtarget; remove.envfile copiesDockerfile: update git clone URL toapache/opendalproject.yaml: updatemain_repo; addtonghuaroot@gmail.comtoauto_ccs.fs.envand.memory.envEvidence
All four OSS-Fuzz targets currently report 0.26% coverage (harness startup only):
https://introspector.oss-fuzz.com/project-profile?project=opendal
Upstream tracking issue: apache/opendal#7722