itzg/minecraft-server-charts

Missing values in helm templates

Open

#141 aberto em 24 de jan. de 2023

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Mustache (165 forks)github user discovery
good first issuehelp wanted

Métricas do repositório

Stars
 (391 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

In the helm chart, two values are missing, as previously added in Minecraft bedrock. https://github.com/itzg/docker-minecraft-bedrock-server/blob/ab51be217cf5f245fcb3239466f145af3a9422ba/property-definitions.json#L112

https://github.com/itzg/docker-minecraft-bedrock-server/blob/ab51be217cf5f245fcb3239466f145af3a9422ba/property-definitions.json#L116

I did not find those two values can be easily controlled in the helm chart values.

  "emit-server-telemetry": {
    "env": "EMIT_SERVER_TELEMETRY",
    "allowed": ["true","false"]
  },
  "enable-lan-visibility": {
    "env": "ENABLE_LAN_VISIBILITY",
    "allowed": ["true","false"]
  }

I expect this feature can be added.

Guia do colaborador

Direção de pesquisa
Adicione dois mapeamentos de variáveis de ambiente para 'emit server telemetry' e 'enable lan visibility' ao values.yaml e aos templates do gráfico Helm, seguindo o padrão das propriedades existentes.
Pilha de tecnologia
go
Domain
devopsinfrastructure
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
HelmKubernetes
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70