Make CI Fuzz less noisy#7065
Conversation
|
Skipping CI for Draft Pull Request. |
|
@AdamKorcz Does it make sense to move https://github.com/cncf/cncf-fuzzing/blob/58a5397633815046cccfd2f2e4940e463b443b74/projects/containerd/remotes_docker_fuzzer.go from the repository to this repository? This one is really noisy and I want to change its logging level. |
|
Looks great. We can increase the log level even further. The logs are not needed for the fuzzers when they run continuously. |
Yes |
kzys
force-pushed
the
fuzz-quiet
branch
2 times, most recently
from
b944dd6 to
37429aa
Compare
kzys
force-pushed
the
fuzz-quiet
branch
3 times, most recently
from
23c8921 to
b06020d
Compare
containerd/containerd#7065 will move the fuzzer to the upstream. Signed-off-by: Kazuyoshi Kato <[email protected]>
|
This one is ready for review. The result of "Run Fuzzers" from this PR is still noisy but better than what we have in main. |
|
@AdamKorcz @mxpv Can you take a look? |
LGTM, but as a suggestion, the log level can be upped to panic. |
Signed-off-by: Kazuyoshi Kato <[email protected]>
It is too noisy and not really useful if the input is random. Signed-off-by: Kazuyoshi Kato <[email protected]>
Signed-off-by: Kazuyoshi Kato <[email protected]>
This test is one of the noisiest and should be moved from cncf/cncf-fuzzing first. Signed-off-by: Kazuyoshi Kato <[email protected]>
Signed-off-by: Kazuyoshi Kato <[email protected]>
|
@AdamKorcz Changed in 50f1a4e and f91616e. |
4 participants
This PR makes CR Fuzz less noisy. The fuzzers in cncf-fuzzing are untouched yet. They will be merged later (see #7066).