Skip to content

thrift_proxy: remove alpha status#9903

Merged
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
rgs1:thrift-proxy-remove-alpha-status
Feb 1, 2020
Merged

thrift_proxy: remove alpha status#9903
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
rgs1:thrift-proxy-remove-alpha-status

Conversation

@rgs1
Copy link
Member

@rgs1 rgs1 commented Feb 1, 2020

We've been using this in prod for a couple of months and for
a few different services.

A few critical bugs have been fixed (#9089 and #6549) and some
necessary stats/features have been added (#9203 and #8994), so
it's probably a good time to graduate at least the proxy
and the router filter.

Signed-off-by: Raul Gutierrez Segales rgs@pinterest.com

We've been using this in prod for a couple of months and for
a few different services.

A few critical bugs have been fixed (envoyproxy#9089 and envoyproxy#6549) and some
necessary stats/features have been added (envoyproxy#9203 and envoyproxy#8994), so
it's probably a good time to graduate at least the proxy
and the router filter.

Signed-off-by: Raul Gutierrez Segales <rgs@pinterest.com>
@rgs1 rgs1 requested a review from zuercher as a code owner February 1, 2020 01:29
@rgs1 rgs1 changed the title thrift_proxy: migrate from v2alpha to v2 thrift_proxy: remove alpha status Feb 1, 2020
Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit 5176116 into envoyproxy:master Feb 1, 2020
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