[newrelic-logging] Missing mount on daemonset ?

[jodem]

Bug description

I deploy latest version of the chart in the bundle, the logging does not working anymore.

[error] [filter:lua:lua.3] cannot access script '/fluent-bit/scripts/payload.lua'

The configmap exists "newrelic-nri-bundle-newrelic-logging-lua" and does have a payload.lua

however it is not mounted in the daemonset.

volumeMounts:
  - mountPath: /fluent-bit/etc
     name: fluent-bit-config
  - mountPath: /var
     name: logs       

Looking at the template, looks like you need to override ressources to make it work.

{{- end }}
          {{- if .Values.resources }}. <==== problem here captain
            {{- if eq $.Values.fluentBit.fluentBitMetrics "basic" }}

            - name: lua-scripts
              mountPath: /fluent-bit/scripts
              readOnly: true
            {{- end }}
          resources:

I am not a helm rockstar but I think the {{- if .Values.resources }} should be above resources, not in the check on basic fluentbit metric param that decide if you need a volume mount.

Version of Helm and Kubernetes

bundle 6.0.35, kube 1.33

Which chart?

nri-bundle => newrelic-logging

Workaround : make sur you specify limits on logging chart to have the volume mounted

    newrelic-logging:
      fluentBit:
        fluentBitMetrics: "basic"
      enabled: true
      resources:
        limits:
          cpu: 200m
          memory: 128Mi
        requests:
          cpu: 100m
          memory: 64Mi

[Chris-Pohl]

I added a comment here:
d401da7#r176325931
The volumeMount for lua-scripts is under ports instead of volumeMount.