John Bonham

Hey @Nick-Harvey , thanks for reaching out. Sorry, I'm not sure what part of you first question "or a C&P of the crt & key", means? For TLS support the...

The services use the chart and release name combo as names, which become the host names. You can specify a release name when installing, I think something like `helm install...

Hey @rendhalver we are definitely still up for collaboration- I have been working locally on the chart you are proposing into the official repo, and would like to make a...

@rendhalver how would you like me to add the TLS changes to your PR? I pulled down your branch that is being proposed into the charts repo, so I could...

Yeah sure thing, I'll do one of the two once I get it wrapped up. I'm still doing some finishing touches

@Nick-Harvey correct me if I misinterpreted anything. What domains are covered in `portus.tls`? With the name you specified above it should cover "reg-foobar" "reg-foobar-registry" and "reg-foobar-nginx". The TLS credentials used...

Yes :) I think "ingress.kubernetes.io/ssl-passthrough" might need to be uncommented as well. I realize it is confusing setting up TLS twice, but I really couldn't think of a better way,...

Oh sorry for any confusion- yes it is most definitely possible with this chart! I was just trying to clarify why TLS is needed for the internal services as well...

Thank you :) I'm glad that did the trick

Hey @rendhalver I got an updated version of your chart working with a base non-TLS default, which is passing the chart test locally for me. I ran into some issues-...