Skip to content

Fix minecraft-proxy template references in extraPorts#281

Closed
cpfarhood wants to merge 1 commit intoitzg:masterfrom
cpfarhood:fix-minecraft-proxy-extraports-template
Closed

Fix minecraft-proxy template references in extraPorts#281
cpfarhood wants to merge 1 commit intoitzg:masterfrom
cpfarhood:fix-minecraft-proxy-extraports-template

Conversation

@cpfarhood
Copy link
Copy Markdown
Contributor

  • Fix extraports-svc.yaml: use 'proxy.fullname' instead of 'minecraft.fullname'
  • Fix extraports-ing.yaml: use 'proxy.fullname' instead of 'minecraft.fullname'

The extraPorts feature was referencing the wrong chart template name, causing deployment failures when extraPorts were configured:

Error: template "minecraft.fullname" not defined

This fix corrects the template references to use the proper 'proxy.fullname' template defined in the minecraft-proxy chart.

- Fix extraports-svc.yaml: use 'proxy.fullname' instead of 'minecraft.fullname'
- Fix extraports-ing.yaml: use 'proxy.fullname' instead of 'minecraft.fullname'

The extraPorts feature was referencing the wrong chart template name,
causing deployment failures when extraPorts were configured:

  Error: template "minecraft.fullname" not defined

This fix corrects the template references to use the proper
'proxy.fullname' template defined in the minecraft-proxy chart.
Copy link
Copy Markdown
Owner

@itzg itzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again

Copy link
Copy Markdown
Owner

@itzg itzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, the chart version needs to be bumped.

@cpfarhood cpfarhood closed this by deleting the head repository Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants