milvus-io/milvus-operator

Add check to avoid `null` in config fields

Open

#227 aberto em 17 de fev. de 2023

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Go (30 forks)auto 404
enhancementgood first issue

Métricas do repositório

Stars
 (63 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Veja como os campos de configuração são deserializados do YAML e adicione uma verificação de nulo no código de validação para rejeitar valores nulos.
Pilha de tecnologia
go
Domain
infrastructure
Tipo Issue
Bug
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitGo
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70