-
Notifications
You must be signed in to change notification settings - Fork 702
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
Update chart to support TLS for pinniped-proxy #4959
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.
LGTM
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.
Thanks!
@@ -1581,6 +1581,14 @@ pinnipedProxy: | |||
## @param pinnipedProxy.defaultPinnipedAPISuffix Specify the (default) API suffix | |||
## | |||
defaultPinnipedAPISuffix: pinniped.dev | |||
## @param pinnipedProxy.TLSSecret Specify an optional TLS secret with which to proxy requests |
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 guess we have to run the readmenator to add those params to the chart docs.
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.
Yeah - I'll do that. It does now get run automatically upstream, but no need to wait.
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.
Yes, fortunately, there's no need anymore, but it's nice to have our chart up to date, especially if we, want to give a try to the brand new TLS config. Thanks!
ee2141c
to
9dfad4c
Compare
0729ef2
to
b0a4c4a
Compare
1532feb
to
5962adc
Compare
Signed-off-by: Michael Nelson <minelson@vmware.com>
Signed-off-by: Michael Nelson <minelson@vmware.com>
Signed-off-by: Michael Nelson <minelson@vmware.com>
Signed-off-by: Michael Nelson <minelson@vmware.com>
b0a4c4a
to
3581d13
Compare
✅ Deploy Preview for kubeapps-dev ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Description of the change
Following on from #4958, this PR updates the kubeapps chart to:
Benefits
The pinniped-proxy service can run as a TLS web service, when configured so.
Possible drawbacks
Applicable issues
Additional information