envoyproxy/envoy

AsyncClientFactory getThreadLocalCluster failure under validation_server

Ouverte

#15 447 ouverte le 11 mars 2021

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
area/grpcbughelp wanted

Métriques du dépôt

Stars
 (27 997 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

We're seeing an issue validating our grpc access logging config

[external/envoy/source/server/config_validation/server.cc:60] error initializing configuration 'generated/config/envoy_canary.yaml': Unknown gRPC client cluster 'grpceventsink'

I think these lines should be using the helper ClusterInfoMaps.getCluster() instead of iterating the map manually.

https://github.com/envoyproxy/envoy/blob/e20957c011b11a4f9f935ec7556ed89d1d8c22d7/source/common/grpc/async_client_manager_impl.cc#L41-L42

Guide contributeur

Direction de recherche
Reproduisez l'erreur avec une configuration utilisant le journal d'accès gRPC. Examinez le code dans async client manager impl.cc. Remplacez l'itération manuelle par ClusterInfoMaps.getCluster() pour résoudre l'erreur de cluster inconnu.
Stack technique
cppgrpc
Domaine
backendinfrastructurenetworking
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
Claire
Prérequis
C++GitBazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75