-
Notifications
You must be signed in to change notification settings - Fork 47
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
Multi tenant kubernetes example #510
Conversation
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.
I think removing the connection to SNI is a good idea but I'm not entirely sold on the idea of passing the virtual cluster name via the context.
(n.b. I am aware that I did it first but there is a reason I didn't include it in the branch we merged adding the name to the model)
kroxylicious-api/src/main/java/io/kroxylicious/proxy/filter/KrpcFilterContext.java
Show resolved
Hide resolved
473a401
to
7922bec
Compare
7922bec
to
a6effe9
Compare
SonarCloud Quality Gate failed. 0 Bugs 66.7% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Merged despite the Sonar Cloud gate. Justification:
|
Type of change
Description
(Revived from #481)
Simple kubernetes example showing multi-tenant in action. Kroxylicious exposed using NodePort.
The illustrative command used to demonstrate multitent use kaf. You could use any client,
kaf
just has a nice CLI experience.Additional Context
Why are you making this pull request?
Checklist
Please go through this checklist and make sure all applicable tasks have been done