[FIX]: Set correct indentation for included env from .Values.extraEnv

[ptitmouton]

Hi,

I found an issue with the extraEnv option introduced in #4

Adding the following option to values.yaml:

extraEnv:
  - name: NAME
    value: VALUE
  - name: OTHER
    value: VALUE2

Leads to the following deployment.yaml in the template (which fails to deploy):

<...>
            - name: CLICKHOUSE_DATABASE_URL
              valueFrom:
                secretKeyRef:
                  key: CLICKHOUSE_DATABASE_URL
                  name: plausible-plausible-analytics
                        - name: NAME
            value: VALUE
          - name: OTHER
            value: VALUE2
<...>

There are two problems with

helm-plausible-analytics/templates/deployment.yaml

Line 225 in f480d86

{{ toYaml .Values.extraEnv | indent 10 }}

1. The first line is too much indented. This is because the spaces before {{ toYaml .Values.extraEnv | indent 10 }} are included in the output template
2. The required indentation should be 12, not 10

In the hope I can help with this PR.

Checklist

• Chart Version bumped

[alexnuttinck]

Thanks @ptitmouton!
I also made a fix, I changed the indent function into the nindentfunction for extraEnv.
Let me now if all work for you.

[ptitmouton]

I did not know about nindent.
Works perfectly, thank you!