All Helm Chart Fails to Render emptyDir

[blanchardma]

Describe the bug
Rendering a Helm chart that includes an emptyDir volume fails due to a YAML parsing error. This issue occurs starting from version v0.50.0 of the code-generator, and can be reproduced with the acm-chart version 1.0.14.

Steps to reproduce
Run the following Helm command:

$ helm template acm oci://public.ecr.aws/aws-controllers-k8s/acm-chart --version 1.0.14 --set deployment.extraVolumes[0].name=test --set deployment.extraVolumes[0].emptyDir.sizeLimit=1Mi

Output:

Pulled: public.ecr.aws/aws-controllers-k8s/acm-chart:1.0.14
Digest: sha256:361345f6800041b88ddbacdfe12949a6133b7fe5fa03f0d9debac0868eab01b2
Error: YAML parse error on acm-chart/templates/deployment.yaml: error converting YAML to JSON: yaml: line 127: did not find expected key

Use --debug flag to render out invalid YAML

Expected outcome
That it works correctly as in previous versions, such as 1.0.13:

$ helm template acm oci://public.ecr.aws/aws-controllers-k8s/acm-chart --version 1.0.13 --set deployment.extraVolumes[0].name=test --set deployment.extraVolumes[0].emptyDir.sizeLimit=1Mi

Output:

...
      hostIPC: false
      hostPID: false
      hostNetwork: false
      dnsPolicy: ClusterFirst
      volumes:
        - emptyDir:
            sizeLimit: 1Mi
          name: test

Side note
I already preparing a PR to solve the issue.

[github-actions]

Hello @blanchardma 👋 Thank you for opening an issue in ACK! A maintainer will triage this issue soon.

We encourage community contributions, so if you're interested in tackling this yourself or suggesting a solution, please check out our Contribution and Code of Conduct guidelines.

You can find more information about ACK on our website.

[blanchardma]

The PR to solve this issue will follow from aws-controllers-k8s/code-generator@main...blanchardma:code-generator:fix/render-extra-volumes after some local testing.

[rushmash91]

Hi @blanchardma , Feel free to create the PR & we can run tests on it.
Thank you for contributing!