Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.
This repository has been archived by the owner on May 16, 2023. It is now read-only.

use Release.Name in resources #1644

Open

Description

Hi, it is common practice to use the release name in the created resources, unless the release name is the chart name. I can see that this is not the case with this chart.

This makes it inconvenient to deploy it under a different release name or as a subchart.

I think it would be good to follow the established pattern.

Usually this is done by a helper such as

{{- define "elasticsearch.fullname" -}}

I can see that it's rather using different helpers. For the statefulset it's this for example

{{- define "elasticsearch.uname" -}}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions