itzg / itzg/minecraft-server-charts

Spawn protection setting bugged

Open
#227 8 comments 0 reactions 0 assignees View on GitHub
Dominant language
Mustache
Stars
406
Forks
164
Avg merge
1h 34m
Merged PRs (30d)
1

Description

The spawn protection config setting cannot be set to "0" as mentioned in the docs as it is currently bugged. Setting it to 0 will cause the envmap to evaluate to false and instead exclude the `SPAWN_PROTECTION` environment variable rather than including it as intended.

https://github.com/itzg/minecraft-server-charts/blob/ba5b3ba7168c3ac6f416e9c00aea35d37c0d9875/charts/minecraft/templates/_helpers.tpl#L28

https://github.com/itzg/minecraft-server-charts/blob/ba5b3ba7168c3ac6f416e9c00aea35d37c0d9875/charts/minecraft/templates/deployment.yaml#L174

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with charts/minecraft/templates/_helpers.tpl at line 28 and charts/minecraft/templates/deployment.yaml at line 174. Render the chart with spawn protection set to 0 and inspect the generated environment variables. Done means the rendered deployment includes SPAWN_PROTECTION when the setting is 0, as described in the issue and documentation.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm, kubernetes
Domain
devops, infrastructure
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.