envoyproxy/envoy

Inconsistent naming with clusters.

Open

#37.525 geöffnet am 5. Dez. 2024

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem identifiziert inkonsistente Benennungen von Clustern in der Envoy Codebasis: Einige Dateien verwenden 'envoy.cluster', während andere 'envoy.clusters' verwenden. Konkret sind die verknüpften Dateien extensions metadata.yaml (Zeilen 92-131), logical dns cluster.h (Zeile 90) und redis cluster.h (Zeile 313). Die Aufgabe besteht darin, eine einheitliche Namenskonvention (wahrscheinlich 'envoy.clusters', da sie häufiger vorkommt) festzulegen und alle Vorkommen entsprechend umzubenennen. Der Mitwirkende sollte nach anderen Instanzen von 'envoy.cluster' oder 'envoy.clusters' im Quellbaum suchen, um vollständige Konsistenz sicherzustellen.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80