Skip to content

Remove unused directory fields from archive_output.storage and stream_output.storage in values.yaml #2074

@junhaoliao

Description

@junhaoliao

Bug

archive_output.storage.directory and stream_output.storage.directory in the Helm chart's
values.yaml are dangling references — they are defined but never consumed by any template.

The configmap template hardcodes the directories instead:

  • configmap.yaml:25: directory: "/var/data/archives" (ignores values.yaml's
    /tmp/clp/var/data/archives)
  • configmap.yaml:169: directory: "/var/data/streams" (ignores values.yaml's
    /tmp/clp/var/data/streams)

These hardcoded paths match the PVC mount points, so the directory fields in values.yaml have no
effect. They appear to be leftovers from when the clp-package used host-mounted directories (removed
in #2023).

The directory fields and their comments should be removed from values.yaml to avoid confusion.

CLP version

dbc1799 (HEAD of main)

Environment

N/A (Helm chart issue, not environment-specific)

Reproduction steps

  1. Set clpConfig.archive_output.storage.directory or clpConfig.stream_output.storage.directory
    to a custom value in values.yaml (or via --set).
  2. Run helm template or deploy the chart.
  3. Inspect the rendered configmap — the directory fields are hardcoded to /var/data/archives and
    /var/data/streams respectively, ignoring the user-provided values.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions