itzg / itzg/minecraft-server-charts

Helm Chart - Passing Namespace not working

Open
#210 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Mustache
Stars
406
Forks
164
Avg merge
1h 34m
Merged PRs (30d)
1

Description

Helm supports -n or --namespace string to set the namespace on Deployments, Services, etc.
This is not working on mc-router helm charts when doing --dry-run. It does deploy to the correct namespace, but you can not see the Namespace when doing --dry-run.

Could we have this fixed?

You can use **namespace: {{ .Release.Namespace }}** to accomplish this.

See examples below:

apiVersion: v1
kind: ServiceAccount
metadata:
name: example
namespace: {{ .Release.Namespace }}

apiVersion: apps/v1
kind: DaemonSet
metadata:
name: example-ds
namespace: {{ .Release.Namespace }}

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect the mc-router Helm chart templates and render them with --dry-run while passing -n or --namespace. Confirm that the rendered Deployments, Services, ServiceAccount, and DaemonSet include the release namespace, and verify that deployment still targets the correct namespace.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm
Domain
devops
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.