search

cloudflare / helm-charts

https://developers.cloudflare.com
Apache License 2.0
85 stars 66 forks source link

Configurable default ingress service http_status:400 #70

Open juliusiqbal opened 8 months ago

juliusiqbal commented 8 months ago

I was trying to use my own ingress service instead of http_status:400. There is no way to do that with current implementation. Either add default service wtih ingress map or separate variable to change or modify.

https://github.com/cloudflare/helm-charts/blob/b85225eb49e1b0b40328c70d4541a27ec5e2b82e/charts/cloudflare-tunnel/templates/configmap.yaml#L29

Below is what I do now to update value on the fly and after remove pods.

kubectl get cm/cloudflare-tunnel -n cloudflare-tunnel -o yaml |
        sed 's/http_status:404/http:\/\/nginx-ingress-controller.nginx-ingress.svc.cluster.local.:80/g' |
        kubectl apply -f -

kubectl -n cloudflare-tunnel delete pods --all

Would have been better if there was way to modify default ingress service or include that as default for ingress service map.

lexfrei commented 8 months ago

implemented in #69 currently, you can use it from my repo: https://lexfrei.github.io/charts

juliusiqbal commented 8 months ago

Thanks @lexfrei

NeuronButter commented 1 month ago

Is there any update on this being merged in? It's been a few months just waiting for one last review...

lexfrei commented 1 month ago

@NeuronButter this repo looks abandoned. You can use it from my repo as mentioned above. Here is the argo-cd installation example https://github.com/lexfrei/k8s/blob/4b7c1e11c38eb63bbad1c871dc445fd58c561a7c/argocd/infra/cloudflared.yaml

NeuronButter commented 1 month ago

Yeah seems like there are also a few people (myself included) who would be willing to keep it going but we seem to be at the graces of CF employees with free time 😬