-
Notifications
You must be signed in to change notification settings - Fork 6
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
Helm Chart corrections and additions #2
base: main
Are you sure you want to change the base?
Conversation
43bac07
to
c000437
Compare
…abase name and redis authentication Signed-off-by: Katie Bohnenkamper <[email protected]>
…management outside of chart Signed-off-by: Katie Bohnenkamper <[email protected]>
Signed-off-by: Katie Bohnenkamper <[email protected]>
…nger requiring SSL communication beween load balancer and services Signed-off-by: Katie Bohnenkamper <[email protected]>
Signed-off-by: Katie Bohnenkamper <[email protected]>
Signed-off-by: Katie Bohnenkamper <[email protected]>
Signed-off-by: Katie Bohnenkamper <[email protected]>
c000437
to
40aa14e
Compare
Only after looking through https://github.com/kubectyl/panel/blob/develop/.github/docker/entrypoint.sh am I realizing that I could have just not specified an email and would have accomplished the non-encrypted I am of the opinion that the load balancer should handle SSL termination and that there's really no reason to add the complexity of encrypting the traffic between the load balancer and the service, so I will be keeping that in mind and modifying the the |
Perhaps this will fix the issues I was having with setting Kubectyl up. I'll spin up a cluster and see if this works - thanks! |
Chart Additions:
values.yaml
.Chart Fixes:
latest
tag for either the panel or kuber images. This switches them to the correctdevelop
tag.UTC
).I'll be making another PR on the source to correct the generated Kuber configuration as the above changes alone will need manual follow-up intervention to disable SSL, change the port, and use the correct
sftp-server
image tag.