[newrelic-logging] Invalid yaml generated when .Values.exposedPorts, .Values.resources, and .Values.fluentBit.fluentBitMetrics are all set

[aisling-mls]

Bug description

When setting .Values.exposedPorts, .Values.resources, and .Values.fluentBit.fluentBitMetrics is "basic" invalid yaml is generated for daemonset.yaml

Version of Helm and Kubernetes

Helm version 3.16.3
K8s version 1.33.5

Which chart?

newrelic-logging 1.33.0

What happened?

When these values are set the block
- name: lua-scripts mountPath: /fluent-bit/scripts readOnly: true
is placed under ports instead of volume mounts.

The relevant yaml generated was
` volumeMounts:
- name: fluent-bit-config
mountPath: /fluent-bit/etc
- name: logs
# We mount /var by default because container logs could be on /var/log or /var/lib/docker/containers (symlinked to /var/log)
mountPath: /var
ports:
- containerPort: 2020
hostPort: 2020
name: metrics
protocol: TCP

        - name: lua-scripts
          mountPath: /fluent-bit/scripts
          readOnly: true`

What you expected to happen?

The lua-scripts entry should be under volumeMounts

How to reproduce it?
Set .value.exposedPorts and leave the .Values.fluentBit.fluentBitMetrics set to the default "basic"

Anything else we need to know?

https://github.com/newrelic/helm-charts/blob/master/charts/newrelic-logging/templates/daemonset.yaml#L169-L174

This is the offending block

[aisling-mls]

While working on resolving this I also realized that if .values.Resources is not set then the volume mount is not created at all.