Problem Statement

Since it's including the entire body enabled would have been part of the rendered thing in Kubernetes manifest leading to a potential failure:

        enabled: true
        initialDelaySeconds: 10
        periodSeconds: 10
        timeoutSeconds: 10
        failureThreshold: 10
        successThreshold: 10
        httpGet:
          path: /healthz
          port: "8080"
          scheme: HTTP

enabled: true here is invalid. We need an internal field that can be used for entire field parsing here:

          {{- if .Values.livenessProbe.enabled }}
          livenessProbe:
          {{- toYaml .Values.livenessProbe.spec | nindent 12 }}
          {{- end }}

Note that spec.template.spec.containers[0].livenessProbe.httpGet.port should still be the path and not spec.template.spec.containers[0].livenessProbe.spec.httpGet.port. Because that would mean it includes spec.

Related Issue

Fixes #...

Proposed Changes

How do you like to solve the issue and why?

Format

Please ensure that your PR follows the following format for the title:

feat(scope): add new feature
fix(scope): fix bug
docs(scope): update documentation
chore(scope): update build tool or dependencies
ref(scope): refactor code
clean(scope): provider cleanup
test(scope): add tests
perf(scope): improve performance
desig(scope): improve design

Where scope is optionally one of:

• charts
• release
• testing
• security
• templating

Checklist

• I have read the contribution guidelines
• All commits are signed with git commit --signoff
• My changes have reasonable test coverage
• All tests pass with make test
• I ensured my PR is ready for review with make reviewable