envoyproxy/envoy

Are the lb_policy and lb_config forced to match?

开放

#25,583 创建于 2023年2月15日

在 GitHub 查看贡献者指南
 (9 条评论) (0 个反应) (0 位负责人)C++ (5,373 个派生)batch import
area/configurationarea/load balancinghelp wantedquestion

仓库指标

星标
 (27,997 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

Description: Currently in clusterInfoImpl when the lb_configs are loaded, it doesn't check if the lb_policy matches the config that's loaded. We have a separate member for the lb_policy and other members for each config type. Is there anything to stop the user from setting a round_robin config but also setting the lb_policy to least request? If there isn't, should anything be done about that?

Relevant Links: Current implementation of how the lb config members are initialized

Current location of where lb_policy is set

贡献者指南

研究方向
检查source/common/upstream/upstream impl.cc中的当前实现，看是否有任何验证确保lb policy与提供的lb config匹配。如果没有，考虑在集群初始化中添加一个检查，在不匹配时发出警告或错误。查看代码库中其他类似配置不匹配的处理方式。
技术栈
cpp
领域
backend
议题类型
调研
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++understanding of Envoy configuration
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70