milvus-io/milvus-operator

Add check to avoid `null` in config fields

Aperta

#227 aperta il 17 feb 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Go (30 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (63 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina come i campi di configurazione vengono unmarshalizzati da YAML e aggiungi un controllo null nel codice di validazione per rifiutare i valori null.
Tech stack
go
Dominio
infrastructure
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitGo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70