Skip to content

api: stop generating v4alpha protos#17842

Merged
lizan merged 4 commits intoenvoyproxy:mainfrom
lizan:stop_v4alpha_generation
Aug 25, 2021
Merged

api: stop generating v4alpha protos#17842
lizan merged 4 commits intoenvoyproxy:mainfrom
lizan:stop_v4alpha_generation

Conversation

@lizan
Copy link
Copy Markdown
Member

@lizan lizan commented Aug 25, 2021

Stop generating v4alpha protos as it won't land in foreseeable future.

This fixes go-control-plane sync because of it fails to generate contrib API correctly.

Risk Level: Medium
Testing: CI
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to api/envoy/.
envoyproxy/api-shepherds assignee is @adisuissa
CC @envoyproxy/api-watchers: FYI only for changes made to api/envoy/.

🐱

Caused by: #17842 was opened by lizan.

see: more, trace.

lizan added 3 commits August 25, 2021 02:00
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, a much smaller change to tooling than I anticipated. Thanks!

@lizan lizan merged commit e453c6c into envoyproxy:main Aug 25, 2021
@lizan lizan deleted the stop_v4alpha_generation branch August 25, 2021 20:47
tyxia added a commit to tyxia/envoy that referenced this pull request Aug 26, 2021
Signed-off-by: Tianyu Xia <tyxia@google.com>
tyxia added a commit to tyxia/envoy that referenced this pull request Aug 26, 2021
Signed-off-by: Tianyu Xia <tyxia@google.com>
tyxia added a commit to tyxia/envoy that referenced this pull request Aug 26, 2021
Signed-off-by: Tianyu Xia <tyxia@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants