envoyproxy/envoy

Inconsistent naming with clusters.

Open

#37.525 aperta il 5 dic 2024

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
enhancementhelp wanted

Metriche repository

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

Descrizione

Sometimes, we name our clusters envoy.cluster.xx but other times use envoy.clusters.xx

https://github.com/envoyproxy/envoy/blob/850c7f10cbf5e1132de4eed6b5131d2928ddb95f/source/extensions/extensions_metadata.yaml#L92-L131

https://github.com/envoyproxy/envoy/blob/850c7f10cbf5e1132de4eed6b5131d2928ddb95f/source/extensions/clusters/logical_dns/logical_dns_cluster.h#L90

https://github.com/envoyproxy/envoy/blob/850c7f10cbf5e1132de4eed6b5131d2928ddb95f/source/extensions/clusters/redis/redis_cluster.h#L313

Guida contributor

Direzione di ricerca
Il problema identifica una denominazione incoerente dei cluster nella codebase di envoy: alcuni file utilizzano 'envoy.cluster' mentre altri utilizzano 'envoy.clusters'. Nello specifico, i file collegati sono extensions metadata.yaml (righe 92-131), logical dns cluster.h (riga 90) e redis cluster.h (riga 313). Il compito è decidere una singola convenzione di denominazione (probabilmente 'envoy.clusters' poiché appare più frequentemente) e rinominare di conseguenza tutte le occorrenze. Il contributore dovrebbe cercare altre istanze di 'envoy.cluster' o 'envoy.clusters' nell'albero dei sorgenti per garantire una completa coerenza.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80