kadaster-labs / sensrnet-helm-charts

Other
0 stars 0 forks source link

Install backend in 1 step #39

Closed kad-busses closed 2 years ago

kad-busses commented 2 years ago

Set default backend replicaCount to 1, as installation should be more robust now. This simplifies the installation procedure

Update documentation: 1) Combine the previous separate steps for installing the databases and backend. 2) Delete warning for >=3 nodes, the charts also work for 1 3) Update Ingresses to correct format -> Closes https://github.com/kadaster-labs/sensrnet-helm-charts/issues/31

kad-busses commented 2 years ago

Testing can be done on a local Docker Desktop K8s cluster with using the updated script from https://github.com/kadaster-labs/sensrnet-ops/pull/45

kad-busses commented 2 years ago

Thanks voor het testen. Kon de slash niet reproduceren onder WSL met Bash/ZSH. Heb je dat probleem met alle slashes, dus ook oidc_issuer en oidc_jwks_url?

kad-floriw commented 2 years ago

Nee, alleen als de value begint met een slash. Git bash probeert er dan een absoluut pad van te maken (omdat het denkt dat het een relatief pad is). Maar is niet relevant voor deze PR, maar misschien wel handig om dat ergens in het script te noteren (als commentaar) voor als andere het ook uitvoeren met git bash

marcvanandel commented 2 years ago

This is a setting of MSYS: MSYS_NO_PATHCONV=1 (see StackOverflow)