search

kadaster-labs / sensrnet-helm-charts

Other
0 stars 0 forks source link

Update ingresses #13

Closed kad-busses closed 3 years ago

kad-busses commented 3 years ago

Alle Ingresss maakten gebruik van Traefik Custom Resource Definitions, terwijl dat momenteel alleen nodig is voor multichain. Gezien Traefik ook prima de standaard Ingress oppakt, kunnen we de overige componenten (frontend, viewer, geoserver en backend) beter omschrijven naar Ingress, ipv IngressRoute. Hiervoor zijn de ingress.ymls en values.ymls aangepast. De Readme's en values.schema.json reflecteren deze wijziging.

Er bestond nog wat onduidelijkheid over hoe de multichain te installeren. Instructies hiervoor heb ik toegevoegd in de top-level README.md, met een verwijzing in de charts/multichain-node/README.md, om de instructies voor installatie zoveel mogelijk bijeen te houden.

Multichain-node heeft, vanwege de niet-natively ondersteunde TCP route, nog steeds extra's nodig. In ons geval is dat nu Traefik v2's IngressRouteTCP, maar dat kan ook met extra configfiles voor Nginx gedaan worden. De keuze hiervoor hangt af van de IngressController die gebruikt wordt op het cluster. Implementerende partijen worden nu verplicht Traefik te gebruiken, we kunnen dus overwegen om ook deployment files voor nginx toe te voegen, als alternatief naast traefik.

kad-busses commented 3 years ago

Oops, had de PR nog op Draft staan. Anyway, bedankt voor de review!