oss-fuzz: [CIFuzz]: CIFuzz seems to be incompatible with forks
I opened a PR against a fork and CIFuzz failed there with
2020-04-29 05:10:21,600 - root - DEBUG - Stderr of command 'git clone https://github.com/systemd/systemd systemd' is Cloning into 'systemd'...
.
2020-04-29 05:10:21,845 - root - DEBUG - Stderr of command 'git fetch origin refs/pull/1/merge' is fatal: Couldn't find remote ref refs/pull/1/merge
.
Running: docker build -t gcr.io/oss-fuzz/systemd projects/systemd
2020-04-29 05:10:21,846 - root - ERROR - Can not check out requested state.
2020-04-29 05:10:21,846 - root - ERROR - Error building fuzzers for project systemd with pull request refs/pull/1/merge.
I’m not sure what to do about it but given that GitHub Actions are turned on by default everywhere I’d say it should probably be skipped if the target repository is different from what CIFuzz expects. On Travis CI something like [[ "$TRAVIS_REPO_SLUG" = "systemd/systemd" ]] is used to protect the scripts from forks.
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 15 (14 by maintainers)
Commits related to this issue
- cifuzz: protect forks from CIFuzz CIFuzz isn't compatible with forks: https://github.com/google/oss-fuzz/issues/3731 — committed to evverx/systemd by evverx 4 years ago
- cifuzz: protect forks from CIFuzz CIFuzz isn't compatible with forks: https://github.com/google/oss-fuzz/issues/3731 — committed to evverx/systemd by evverx 4 years ago
- Add workaround for CIFuzz build bug CIFuzz currently does not support fuzzing of forks. See https://github.com/google/oss-fuzz/issues/3731. This changes the configuration so that it CIFuzz does not r... — committed to jfoote/h2o by jfoote 4 years ago
- ci: turn off CIFuzz for forks Unfortunately CIFuzz isn't compatible with forks: https://github.com/google/oss-fuzz/issues/3731 Signed-off-by: Evgeny Vereshchagin <evvers@ya.ru> — committed to evverx/selinux by evverx 3 years ago
- Workaround for google/oss-fuzz#3731 Run CIFuzz docker manually — committed to ailin-nemui/irssi by deleted user 3 years ago
- [elfutils] point CIFuzz to evverx/elfutils I created https://github.com/evverx/elfutils where I test elfutils with gcc/clang, asan/ubsan, LGTM and Coverity. It would be great if it was possible to te... — committed to evverx/oss-fuzz by evverx 3 years ago
- Workaround for google/oss-fuzz#3731 Run CIFuzz docker manually — committed to ahf/irssi by deleted user 3 years ago
- irssi: Autospec creation for update from version 1.2.3 to version 1.4.2 = (1): Used Tabs instead of spaces and increased ABI_VERSION Ailin Nemui (149): fix irssi-version link Merge... — committed to clearlinux-pkgs/irssi by phmccarty 2 years ago
hi, confusingly CI-Fuzz checks out the wrong PR on forks, I guess this is related to this bug report?
If I open PR 1 on my fork, it instead builds PR 1 of the original repo.