Skip to content

Update helm chart to reflect newer config params added since 1.13.0 #90

@makeittotop

Description

@makeittotop

The current helm chart is suitable only up to app. version 1.13.0. However the latest app sits on v1.1.5.7. When I attempted to retro-fit the config suitable for the latest version into the existing helm charts, it erred out.

For example -

$ helm upgrade --install --namespace default cortex-tenant .  --dry-run=client
Release "cortex-tenant" does not exist. Installing it now.
Error: values don't meet the specifications of the schema(s) in the following chart(s):
cortex-tenant:
- config: Additional property metrics_include_tenant is not allowed
- config: Additional property listen_metrics_address is not allowed

Would it be OK if I update the related helm chart components - values-schema.json, and the templates to make this change, and raise a PR?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions