Skip to content

Make CI Fuzz optional#7067

Merged
mxpv merged 1 commit intocontainerd:mainfrom
kzys:fuzz-optional
Jun 16, 2022
Merged

Make CI Fuzz optional#7067
mxpv merged 1 commit intocontainerd:mainfrom
kzys:fuzz-optional

Conversation

@kzys
Copy link
Copy Markdown
Member

@kzys kzys commented Jun 15, 2022

Until we merge cncf/cncf-fuzzing into this repository (see #7066),
we should keep this step optional.

Signed-off-by: Kazuyoshi Kato [email protected]

@k8s-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@mxpv
Copy link
Copy Markdown
Member

mxpv commented Jun 15, 2022

In my case this was failing at "Build step".
https://github.com/containerd/containerd/runs/6906475993
So it probably worth to have continue-on-error in CI yaml.

Until we merge cncf/cncf-fuzzing into this repository (see containerd#7066),
we should keep this step optional.

Signed-off-by: Kazuyoshi Kato <[email protected]>
@kzys kzys marked this pull request as ready for review June 16, 2022 02:40
@mxpv mxpv merged commit 9c6154b into containerd:main Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants