Skip to content
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

Warning: EndPort has not been implemented. #28737

Closed
wants to merge 1 commit into from

Conversation

chaunceyjiang
Copy link
Member

@chaunceyjiang chaunceyjiang commented Oct 23, 2023

EndPort has not been implemented yet, add some warns to let users understand why EndPort is not working.

Fixes #28684

Add some warns to let users understand why EndPort is not working.

EndPort has not been implemented yet, add some warns to let users understand why EndPort is not working.

Fixes cilium#28684

Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
@chaunceyjiang chaunceyjiang requested a review from a team as a code owner October 23, 2023 11: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 Oct 23, 2023
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Oct 23, 2023
@joamaki
Copy link
Contributor

joamaki commented Oct 23, 2023

This is a duplicate of #28704.

@chaunceyjiang
Copy link
Member Author

/close

@chaunceyjiang chaunceyjiang deleted the endport branch October 24, 2023 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. kind/community-contribution This was a contribution made by a community member.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cilium silently accepts k8s networkpolicy with endPort without implementing it
2 participants