Skip to content

Upgrade containerd/cgroups to remove github.com/cilium/ebpf's fuzzer#7304

Merged
estesp merged 3 commits intocontainerd:mainfrom
kzys:upgrade-cgroups
Aug 18, 2022
Merged

Upgrade containerd/cgroups to remove github.com/cilium/ebpf's fuzzer#7304
estesp merged 3 commits intocontainerd:mainfrom
kzys:upgrade-cgroups

Conversation

@kzys
Copy link
Copy Markdown
Member

@kzys kzys commented Aug 16, 2022

The fuzzer is broken and it breaks OSS-Fuzz according to #7288.

@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

@kzys kzys marked this pull request as ready for review August 16, 2022 23:35
@estesp
Copy link
Copy Markdown
Member

estesp commented Aug 17, 2022

/retest-required

@dims
Copy link
Copy Markdown
Member

dims commented Aug 17, 2022

/test pull-containerd-build

@kzys kzys force-pushed the upgrade-cgroups branch from a590ab0 to 1f8bea4 Compare August 17, 2022 19:14
@dims
Copy link
Copy Markdown
Member

dims commented Aug 17, 2022

/retest

1 similar comment
@chaodaiG
Copy link
Copy Markdown
Contributor

/retest

Kazuyoshi Kato added 3 commits August 17, 2022 21:43
The fuzzer is broken and it breaks OSS-Fuzz according to containerd#7288.

Signed-off-by: Kazuyoshi Kato <[email protected]>
The package doesn't have the file anymore.

Signed-off-by: Kazuyoshi Kato <[email protected]>
Otherwise, go-digest panics (panic: sha256 not available).

Signed-off-by: Kazuyoshi Kato <[email protected]>
@kzys kzys force-pushed the upgrade-cgroups branch from 1f8bea4 to 9d286ba Compare August 17, 2022 21:43
Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@estesp estesp merged commit 4e37eb3 into containerd:main Aug 18, 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.

6 participants