cert-manager-webhook-pdns icon indicating copy to clipboard operation
cert-manager-webhook-pdns copied to clipboard

Set http/https proxy via env variables

Open bbusioc opened this issue 1 year ago • 4 comments

Hello,

Is there a was to set the http/s proxy via an environment variable by using the helm chart?
For some reason, which I can't really understand to reach the power DNS API I have to go through a proxy.
I manually edited the deployment, added the required HTTP_PROXY, HTTPS_PROXY and NO_PROXY configurations and it worked.
However, I can't find a way to set environment variables via the helm chart.

Thanks, Bogdan

bbusioc avatar Sep 19 '23 09:09 bbusioc

I fixed it locally on my side by updating like this the env part in the deployment template:

          env:
            - name: GROUP_NAME
              value: {{ .Values.groupName | quote }}
            {{- with .Values.http_proxy }}
            - name: HTTP_PROXY
              value: {{ . }}
            {{- end }}
            {{- with .Values.https_proxy }}
            - name: HTTPS_PROXY
              value: {{ . }}
            {{- end }}
            {{- with .Values.no_proxy }}
            - name: NO_PROXY
              value: {{ . }}
            {{- end }}

But I think it would be great if this or a better solution could be integrated in your repo.

Thanks, Bogdan

bbusioc avatar Sep 20 '23 07:09 bbusioc

Hello @bbusioc, thanks - I thought the chart supported arbitrary env vars, but it looks like it does not. I'll add that support soon.

zachomedia avatar Sep 20 '23 08:09 zachomedia

@zachomedia : when will it be possible for you to add env variables ?

stephbat avatar Sep 24 '24 18:09 stephbat

I sent this PR https://github.com/zachomedia/cert-manager-webhook-pdns/pull/54

stephbat avatar Sep 24 '24 18:09 stephbat

Resolved via #54

zachomedia avatar Sep 30 '24 14:09 zachomedia