envoyproxy / envoyproxy/envoy

tools/api: add support for 1) deprecating message types and 2) moving nested message types up

Open
#10,740 2 comments 0 reactions 0 assignees View on GitHub
api/v3 help wanted
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 22h
Merged PRs (30d)
430

Description

*Title*: add support for 1) deprecating message types and 2) moving nested message types up

*Description*:
* at the moment, there are already a few message types that have been deprecated, e.g.
* `bootstrap.v2.Runtime`
* `tools/api` already support removing it from the next API version by adding `option deprecated = true;`
* `trace.v3.Tracing`
* if `option deprecated = true;` is added, `tools/api` will remove the message along with its nested types, which is not desired (see next point)
* additionally, there is a case where a nested message type needs to be moved up, e.g.
* `trace.v3.Tracing.Http` => `trace.v4.HttpTracer`
* and, probably, even `trace.v3.Tracing.Http` => `trace.v3.HttpTracer`

*Context*:
* #10405

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.