milvus-io/milvus-operator

Add check to avoid `null` in config fields

Open

#227 ouverte le 17 févr. 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Go (30 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (63 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

For manifest like this:

spec:
  config:
    common:
    #  security:
    #    authorizationEnabled: true
    etcd:
      rootPath: my-cluster

would assign null to spec.config.common.

Operator should be able to deny the call.

Guide contributeur

Direction de recherche
Examinez comment les champs de configuration sont désérialisés à partir du YAML et ajoutez une vérification de null dans le code de validation pour rejeter les valeurs nulles.
Stack technique
go
Domaine
infrastructure
Type d'issue
Bug
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitGo
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70