envoyproxy/envoy

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

Open

#15.516 aperta il 16 mar 2021

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/configurationbughelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Ispeziona le definizioni Protobuf per confermare la mancata corrispondenza del tipo e aggiorna il tipo del campo.
Tech stack
cpp
Dominio
backendapinetworking
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
ProtobufC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40