feat: neon iris websat for helm #58
Conversation
NeonDaniel
left a comment
NeonDaniel
left a comment
There was a problem hiding this comment.
https://github.com/NeonGeckoCom/neon-diana-utils/blob/dev/neon_diana_utils/templates/neon/Chart.yaml should also be updated here to include this deployment by default.
We should also consider a single ingress spec for both (more in the future?) web services that go with neon-core
#59 will include some refactoring for this. Ingress will support more services that can be enabled/disabled as part of the |
|
|
||
| dependencies: | ||
| - name: base-neon | ||
| version: 0.0.4 |
| @@ -0,0 +1,38 @@ | |||
| {{ if .Values.ingress.enabled }} | |||
There was a problem hiding this comment.
Any reason not to use the base-neon.ingress template?
{{- include "base-neon.ingress" . }}
| @@ -0,0 +1,27 @@ | |||
| serviceName: neon-core-iris-websat | |||
| subdomain: iris | |||
There was a problem hiding this comment.
I'm not sure this is referenced anywhere looking at the neon-iris chart for comparison
| tlsSecretName: tls-letsencrypt-iris-websat | ||
| certIssuer: letsencrypt-private-key | ||
| rules: | ||
| - host: iris |
There was a problem hiding this comment.
neon-iris uses this default host, can this be websat or satellite (or similar) instead?
2 participants
Description
Adds Helm chart for the iris-websat image.
Issues
N/A
Other Notes
I'm not sure if this is everything that's needed, and there definitely needs to be an update to neon.yaml to properly set the websocket URL for it to work. I couldn't find where to do that though.