itzg/minecraft-server-charts

Missing values in helm templates

Open

#141 ouverte le 24 janv. 2023

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)Mustache (165 forks)github user discovery
good first issuehelp wanted

Métriques du dépôt

Stars
 (391 stars)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Ajoutez deux correspondances de variables d'environnement pour 'emit server telemetry' et 'enable lan visibility' au fichier values.yaml et aux modèles du chart Helm, en suivant le modèle des propriétés existantes.
Stack technique
go
Domaine
devopsinfrastructure
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
HelmKubernetes
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70