envoyproxy/envoy

config: allow double registration with different name and deprecation

Open

#8,540 opened on 2019年10月9日

GitHub で見る
 (1 comment) (2 reactions) (1 assignee)C++ (5,373 forks)batch import
api/v3enhancementhelp wanted

Repository metrics

Stars
 (27,997 stars)
PR merge metrics
 (平均マージ 8d) (30d で 378 merged PRs)

説明

Description:

To support extension rename with better deprecation. e.g. allow TLS transport socket to be registered as envoy.transport_socket.tls and tls and the latter should be deprecated.

https://github.com/envoyproxy/envoy/pull/8508#discussion_r332119188

Work Items:

  • double registration
  • warn log
  • deprecated stat
  • DEPRECATED_FEATURE_TEST

コントリビューターガイド

調査方針
PR #8508 の議論（https://github.com/envoyproxy/envoy/pull/8508#discussion r332119188）を確認し、二重登録の背景を理解します。残りの作業項目は、非推奨の統計指標を追加し、DEPRECATED FEATURE TEST を実装することです。コードベース内の既存の非推奨パターン（例：source/common/config/ など）を調べ、非推奨機能に対して統計指標とテストマクロがどのように使用されているかを理解します。参考として 'source/server/config validation/api validation.h' のようなファイルを探します。
技術スタック
cpp
領域
backend
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
GitC++
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
55