Skip to content

feat: allow templating in secrets.name - #247

Merged
as51340 merged 1 commit into
memgraph:mainfrom
Linkurious:feat/secrets-name-templating
May 22, 2026
Merged

feat: allow templating in secrets.name#247
as51340 merged 1 commit into
memgraph:mainfrom
Linkurious:feat/secrets-name-templating

Conversation

@enys

@enys enys commented May 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Allow Go template expressions in secrets.name values by wrapping references with the tpl function.

Example

secrets:
  enabled: true
  name: "{{ .Release.Name }}-secrets"

Changes

Uses tpl for secrets.name in:

  • charts/memgraph/templates/statefulset.yaml
  • charts/memgraph-lab/templates/deployment.yaml
  • charts/memgraph-high-availability/templates/coordinators.yaml
  • charts/memgraph-high-availability/templates/data.yaml

Plain string values (e.g. memgraph-secrets) continue to work unchanged.

Validation

All three charts verified with helm template:

  • Templated name ({{ .Release.Name }}-secrets) resolves correctly
  • Default plain string values render as before

Use tpl function for secrets.name references so users can use
Go template expressions like {{ .Release.Name }}-secrets.
Plain string values continue to work unchanged.

Affected charts:
- memgraph
- memgraph-lab
- memgraph-high-availability

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@as51340 as51340 added memgraph-ha Memgraph HA chart issue memgraph-lab Memgraph Lab chart issue memgraph Memgraph standalone issue labels May 21, 2026
@as51340 as51340 added this to the 17th June 2026 milestone May 21, 2026
@as51340 as51340 added feature New non-breaking feature docs-not-needed Docs not needed for such a change labels May 21, 2026
@as51340
as51340 merged commit e0c86de into memgraph:main May 22, 2026
7 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-needed Docs not needed for such a change feature New non-breaking feature memgraph Memgraph standalone issue memgraph-ha Memgraph HA chart issue memgraph-lab Memgraph Lab chart issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants