envoyproxy/envoy

protobuf: upstream_bind_config is a core.v3.BindConfig instead of an UpsteamBindConfig

Open

#15.516 aberto em 16 de mar. de 2021

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/configurationbughelp wanted

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: I'm not sure if it's just "misspelt" or it has to be like this, but cluster.upstream_bind_config is a core.v3.BindConfig. I think it should be a UpstreamBindConfig. This way UpstreameBindConfig message is totally unused. And using the current state does nothing.

Additional questions: How exactly should this field work? With this upsteam_bind_config field should I be able to tell envoy which port_value should it use when sending datagrams to the upstream host? I mean we want to control the port value from our service gets its datagrams. Instead of letting envoy open a socket from a random port, we want to tell which specific port to use when connecting to an upstream host. Is it possible?

Guia do colaborador

Direção de pesquisa
Inspecione as definições de Protobuf para confirmar a incompatibilidade de tipo e atualize o tipo do campo.
Pilha de tecnologia
cpp
Domain
backendapinetworking
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
ProtobufC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40