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

Gateway API: Implement generic route checks #25885

Merged
merged 1 commit into from Jul 13, 2023

Conversation

meyskens
Copy link
Member

@meyskens meyskens commented Jun 5, 2023

This refactors the Gateway API code to re-use the route checks functions
into a separate package with route checks which can be used for any Gateway API route type.
This allows for easier code re-use for later route implementations as well as better readability.

Fixes: #25130

Blocked as needed changes from #25711

Gatewap API: Implement generic route checks

@meyskens meyskens added dont-merge/preview-only Only for preview or testing, don't merge it. release-note/misc This PR makes changes that have no direct user impact. labels Jun 5, 2023
@meyskens meyskens force-pushed the meyskens/refactor-gateway branch 2 times, most recently from 5745d95 to 0a189e1 Compare June 6, 2023 15:40
@meyskens meyskens force-pushed the meyskens/refactor-gateway branch 3 times, most recently from d265a18 to 27dbc33 Compare June 12, 2023 14:39
@meyskens meyskens added area/servicemesh GH issues or PRs regarding servicemesh and removed dont-merge/preview-only Only for preview or testing, don't merge it. labels Jun 12, 2023
@meyskens meyskens marked this pull request as ready for review June 12, 2023 14:58
@meyskens meyskens requested a review from a team as a code owner June 12, 2023 14:58
@youngnick youngnick requested review from youngnick and removed request for mhofstetter June 13, 2023 08:28
@jrajahalme
Copy link
Member

/test

@meyskens
Copy link
Member Author

/test

@meyskens
Copy link
Member Author

/test

This refactors the Gateway API code to re-use the route checks functions
into a separate package with route checks which can be used for any
Gateway API route type.
This allows for easier code re-use for later route implementations as
well as better readability.

Signed-off-by: Maartje Eyskens <maartje.eyskens@isovalent.com>
@meyskens
Copy link
Member Author

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 13, 2023
@julianwiedmann julianwiedmann merged commit 74d3f37 into cilium:main Jul 13, 2023
65 checks passed
@meyskens meyskens deleted the meyskens/refactor-gateway branch July 17, 2023 09:36
@sayboras sayboras added backport/author The backport will be carried out by the author of the PR. needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Aug 23, 2023
@sayboras sayboras mentioned this pull request Aug 23, 2023
2 tasks
@sayboras sayboras added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Aug 23, 2023
@sayboras sayboras mentioned this pull request Aug 23, 2023
2 tasks
@sayboras sayboras changed the title Gatewap API: Implement generic route checks Gateway API: Implement generic route checks Aug 23, 2023
@sayboras sayboras added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/servicemesh GH issues or PRs regarding servicemesh backport/author The backport will be carried out by the author of the PR. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make TLSRoute validation functions generic to re-use them on other routes
5 participants