envoyproxy/envoy

Are the lb_policy and lb_config forced to match?

Open

#25.583 aberto em 15 de fev. de 2023

Ver no GitHub
 (9 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/configurationarea/load balancinghelp wantedquestion

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

Description: Currently in clusterInfoImpl when the lb_configs are loaded, it doesn't check if the lb_policy matches the config that's loaded. We have a separate member for the lb_policy and other members for each config type. Is there anything to stop the user from setting a round_robin config but also setting the lb_policy to least request? If there isn't, should anything be done about that?

Relevant Links: Current implementation of how the lb config members are initialized

Current location of where lb_policy is set

Guia do colaborador

Direção de pesquisa
Inspecione a implementação atual em source/common/upstream/upstream impl.cc para ver se há alguma validação que garanta que lb policy corresponda ao lb config fornecido. Se não houver, considere adicionar uma verificação na inicialização do cluster para avisar ou gerar erro quando houver incompatibilidade. Veja como outras incompatibilidades de configuração semelhantes são tratadas no codebase.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Pesquisa
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
C++understanding of Envoy configuration
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70