envoyproxy/envoy

Inconsistent naming with clusters.

Open

#37,525 创建于 2024年12月5日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)C++ (5,373 fork)batch import
enhancementhelp wanted

仓库指标

Star
 (27,997 star)
PR 合并指标
 (平均合并 8天) (30 天内合并 378 个 PR)

描述

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

贡献者指南

研究方向
该问题识别出 Envoy 代码库中集群命名不一致的情况：部分文件使用 'envoy.cluster'，而其他文件使用 'envoy.clusters'。具体涉及的文件包括 extensions metadata.yaml（第 92-131 行）、logical dns cluster.h（第 90 行）和 redis cluster.h（第 313 行）。任务在于确定一个统一的命名约定（可能采用出现频率更高的 'envoy.clusters'），并据此重命名所有相关实例。贡献者应在源代码树中搜索 'envoy.cluster' 或 'envoy.clusters' 的其他出现位置，以确保完全的一致性。
技术栈
cpp
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
1
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GitC++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
80