Skip to content

Do not rename test files on-the-fly to share functions#7309

Merged
kzys merged 1 commit intocontainerd:mainfrom
kzys:fuzz-no-rename
Aug 24, 2022
Merged

Do not rename test files on-the-fly to share functions#7309
kzys merged 1 commit intocontainerd:mainfrom
kzys:fuzz-no-rename

Conversation

@kzys
Copy link
Copy Markdown
Member

@kzys kzys commented Aug 18, 2022

Instead this commit moves some functions that could be used by fuzzers.

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

@kzys kzys force-pushed the fuzz-no-rename branch 2 times, most recently from 20bace7 to 87d6bc4 Compare August 19, 2022 03:05
Instead this commit moves some functions that could be used by fuzzers.

Signed-off-by: Kazuyoshi Kato <[email protected]>
@kzys kzys marked this pull request as ready for review August 19, 2022 06:09
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

@kzys
Copy link
Copy Markdown
Member Author

kzys commented Aug 22, 2022

@AdamKorcz Could you take a look?

@kzys kzys merged commit d0c99ff into containerd:main Aug 24, 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.

5 participants