envoyproxy/envoy

Inconsistent naming with clusters.

Open

#37.525 aberto em 5 de dez. de 2024

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
O problema identifica uma nomenclatura inconsistente de clusters na base de código do envoy: alguns arquivos usam 'envoy.cluster' enquanto outros usam 'envoy.clusters'. Especificamente, os arquivos vinculados são extensions metadata.yaml (linhas 92-131), logical dns cluster.h (linha 90) e redis cluster.h (linha 313). A tarefa é decidir sobre uma única convenção de nomenclatura (provavelmente 'envoy.clusters', pois aparece com mais frequência) e renomear todas as ocorrências de acordo. O contribuidor deve procurar por outras instâncias de 'envoy.cluster' ou 'envoy.clusters' na árvore de fontes para garantir consistência completa.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80