-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add policy fuzzers #22038
add policy fuzzers #22038
Conversation
d91775f
to
e1f4762
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you share also instructions on how to use these fuzzers, ideally in the commit message?
Added in the commit message. |
The CODEOWNERS check complains:
Should this be added to .gitignore? Also, I don't see the instructions to test in the commit message, looks like it is only in the PR description. Could you put it into the commit message too? The other failing tests look like known flakes #22217 and #22168 . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requesting changes for the above.
Commit e40d99b9860c8821e42c0c54ec6494239818825a does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
e40d99b
to
294418a
Compare
The test failure looks like it is caused by a breakage on the main branch, that is expected to be fixed here: #22337 . CODEOWNERS checks seem to be OK now so the testing aspect otherwise seems fine. Given that this is not hooked into CI on the Cilium repository side, I think this is OK to merge now. |
Adds fuzzers that test whether cilium can crash after sanitizing a rule. To test these fuzzers locally, run go test -fuzz=FuzzTestName, for example go test -fuzz=FuzzCiliumNetworkPolicyParse Signed-off-by: AdamKorcz <adam@adalogics.com>
294418a
to
e8702f6
Compare
Signed-off-by: AdamKorcz adam@adalogics.com
Adds fuzzers that test whether cilium can crash after sanitizing a rule.
To test these fuzzers locally, run
go test -fuzz=FuzzTestName
, for examplego test -fuzz=FuzzCiliumNetworkPolicyParse