Skip to content

Conversation

jrajahalme
Copy link
Member

Ever since we dropped the separate allow/deny maps the fuzz test has been failing on trying to fuzz the mapState receiver. With the new flatter mapState definition this no longer works, and the value of trying to fuzz the empty map is in itself questionable, so drop it. This allows the fuzz test to proceed with insertions of fuzzed keys and values.

Ever since we dropped the separate allow/deny maps the fuzz test has been
failing on trying to fuzz the mapState receiver. With the new flatter
mapState definition this no longer works, and the value of trying to fuzz
the empty map is in itself questionable, so drop it. This allows the fuzz
test to proceed with insertions of fuzzed keys and values.

Signed-off-by: Jarno Rajahalme <jarno@isovalent.com>
@jrajahalme jrajahalme added kind/bug/CI This is a bug in the testing code. release-note/misc This PR makes changes that have no direct user impact. labels Nov 27, 2024
@jrajahalme jrajahalme requested a review from a team as a code owner November 27, 2024 08:24
@jrajahalme jrajahalme requested a review from doniacld November 27, 2024 08:24
@github-actions github-actions bot added the sig/policy Impacts whether traffic is allowed or denied based on user-defined policies. label Nov 27, 2024
@jrajahalme
Copy link
Member Author

/test

@joestringer joestringer added this pull request to the merge queue Nov 27, 2024
Merged via the queue into cilium:main with commit 78b2e1a Nov 27, 2024
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug/CI This is a bug in the testing code. release-note/misc This PR makes changes that have no direct user impact. sig/policy Impacts whether traffic is allowed or denied based on user-defined policies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants