Uploaded image for project: 'Data Collection, Analytics, and Events'
  1. Data Collection, Analytics, and Events
  2. DCAEGEN2-3093

Improper test for log directory in dcaegen2 common deployment template

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Medium Medium
    • Jakarta Release
    • Jakarta Release
    • dcae-services
    • None

      The dcaegen2-services-common deployment template checks the chart's values.yaml for a log directory.  It looks for "log.path".  A chart that does not have the "log" property in it will not pass helm lint.

      The problem is with this line in oom/kubernetes/dcaegen2-services/common/dcaegen2-services-common/templates/_deployment.tpl:

      - $logDir := default "" .Values.log.path -

      The default function doesn't nest: if .Values.log doesn't exist, it will complain about a nil pointer.

      DCAEGEN2-3087 reports this problem in the context of charts generated by the DCAEMOD Helm generator.  (There is an additional problem with the Helm generator, in that it is using the wrong property for the log directory path in the values.yaml files that it generates.)

       

            jackl Jack Lucas
            jackl Jack Lucas
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: