envoyproxy/envoy

UDP cluster support for statsd addresses

Open

#1 028 ouverte le 30 mai 2017

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/udpenhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

From @jylin in https://github.com/lyft/envoy/pull/1019:

Would it make sense to try to make the "statsd_tcp_cluster_name" then support UDP too (i.e. a "statsd_udp_cluster_name")?

(in reference to avoiding hardcoding IP addresses in config files for statsd)

This seems reasonable from a config/API perspective, but will involve a bit of work to make work. Currently the cluster implementations such as LogicalDnsCluster are assuming TCP connections.

This is also related to #492 - if there was general support for UDP proxying we would be able to leverage that to do have UDP addressable clusters for statsd.

Guide contributeur

Direction de recherche
Examinez LogicalDnsCluster et les autres implémentations de cluster pour comprendre comment ils supposent TCP ; concevez un moyen de prendre en charge UDP pour les adresses statsd.
Stack technique
cpp
Domaine
networkingbackend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitC++Envoy clustering concepts
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35