Skip to content

Conversation

marseel
Copy link
Contributor

@marseel marseel commented Nov 14, 2024

Select should not be performed on the list but boolean value.

Fixes: #35915

Apparently, I am not good at writing jq queries.
Before I've tested positive case only, and now I've tested both positive and negative cases manually.

Select should not be performed on list but boolean value.

Fixes: cilium#35915

Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
@marseel marseel requested review from a team as code owners November 14, 2024 15:18
@marseel marseel requested a review from dlapcevic November 14, 2024 15:18
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 14, 2024
@marseel
Copy link
Contributor Author

marseel commented Nov 14, 2024

/test

@marseel marseel added the release-note/ci This PR makes changes to the CI. label Nov 15, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Nov 15, 2024
@aanm aanm added this pull request to the merge queue Nov 15, 2024
Merged via the queue into cilium:main with commit 6dbbad0 Nov 15, 2024
76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants