-
Notifications
You must be signed in to change notification settings - Fork 595
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
docs: add KongServiceFacade docs and examples #5298
Conversation
0049594
to
a3e2379
Compare
a3e2379
to
904aabf
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5298 +/- ##
=======================================
- Coverage 75.9% 75.8% -0.1%
=======================================
Files 170 170
Lines 19458 19447 -11
=======================================
- Hits 14769 14756 -13
- Misses 3858 3860 +2
Partials 831 831 ☔ View full report in Codecov by Sentry. |
904aabf
to
5d578c2
Compare
5d578c2
to
d4a46ee
Compare
d4a46ee
to
19e3059
Compare
14bb238
to
506c2cf
Compare
506c2cf
to
fca78c7
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.
Looking good 👍
Only the kustomize command comment is blocking (as the original URL did not work for me with a spurious kustomize error).
Co-authored-by: Michał Flendrich <[email protected]>
@mheap @mflendrich Thanks for your comments, PTAL again 👍 |
What this PR does / why we need it:
Adds a new
FEATURE_GATES.md
section "Using KongServiceFacade" explaining the basics ofKongServiceFacade
. It also references a new example manifest that demonstrates complete usage ofKongServiceFacade
to set a different set of plugins for the same KubernetesService
.The instructions in the example manifest depend on Kong/charts#963 which adds RBACs for
KongServiceFacade
.PR also adds an integration test that verifies the new example manifest to make sure it doesn't get outdated.
Which issue this PR fixes:
Closes #5152.
Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as
ready to review
:CHANGELOG.md
release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR