Skip to content

grpc_stats: removed runtime feature after deprecation period#28144

Merged
jmarantz merged 2 commits intoenvoyproxy:mainfrom
ggreenway:remove-deprecated
Jul 13, 2023
Merged

grpc_stats: removed runtime feature after deprecation period#28144
jmarantz merged 2 commits intoenvoyproxy:mainfrom
ggreenway:remove-deprecated

Conversation

@ggreenway
Copy link
Member

Commit Message: Removing
envoy.deprecated_features.grpc_stats_filter_enable_stats_for_all_methods_by_default which has been deprecated since 2021.

Additional Description:
Risk Level: low
Testing: existing unit tests
Docs Changes: docs update
Release Notes: added
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Removing
`envoy.deprecated_features.grpc_stats_filter_enable_stats_for_all_methods_by_default`
which has been deprecated since 2021.

Signed-off-by: Greg Greenway <ggreenway@apple.com>
@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy[\w/]*/(v1alpha\d?|v1|v2alpha\d?|v2))|(api/envoy/type/(matcher/)?\w+.proto).
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @lizan
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #28144 was opened by ggreenway.

see: more, trace.

@ggreenway
Copy link
Member Author

This has been deprecated since #14815 but somehow removing the old codepath was missed.

@ggreenway
Copy link
Member Author

/retest

@ggreenway
Copy link
Member Author

/assign-from @envoyproxy/envoy-maintainers

@repokitteh-read-only
Copy link

@envoyproxy/envoy-maintainers assignee is @lizan

🐱

Caused by: a #28144 (comment) was created by @ggreenway.

see: more, trace.

@alyssawilk
Copy link
Contributor

@lizan ping!

@ggreenway
Copy link
Member Author

/assign-from @envoyproxy/envoy-maintainers

@repokitteh-read-only
Copy link

@envoyproxy/envoy-maintainers assignee is @jmarantz

🐱

Caused by: a #28144 (comment) was created by @ggreenway.

see: more, trace.

@envoyproxy envoyproxy deleted a comment from repokitteh-read-only bot Jul 11, 2023
@jmarantz jmarantz merged commit 7e68d70 into envoyproxy:main Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants