envoyproxy/envoy

Are the lb_policy and lb_config forced to match?

オープン

#25,583 opened on 2023/02/15

GitHub で見るコントリビューターガイド
 (9 件のコメント) (0 件のリアクション) (0 人の担当者)C++ (5,373 件のフォーク)batch import
area/configurationarea/load balancinghelp wantedquestion

Repository metrics

Stars
 (27,997 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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
Issue 種別
調査
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++understanding of Envoy configuration
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70