Skip to content

grpc_stats: switch default of stats_for_all_methods#14815

Merged
ggreenway merged 2 commits intoenvoyproxy:mainfrom
ggreenway:grpc-stats-config-default
Jan 26, 2021
Merged

grpc_stats: switch default of stats_for_all_methods#14815
ggreenway merged 2 commits intoenvoyproxy:mainfrom
ggreenway:grpc-stats-config-default

Conversation

@ggreenway
Copy link
Member

@ggreenway ggreenway commented Jan 25, 2021

Signed-off-by: Greg Greenway ggreenway@apple.com

Commit Message: grpc_stats: switch default of stats_for_all_methods
Additional Description: This change was announced (and previous value deprecated) in v1.14.0 (#10467).
Risk Level: Low
Testing: Adjusted unittests
Docs Changes: Updated
Release Notes: Added
Platform Specific Features: None
Optional Runtime guard: envoy.deprecated_features.grpc_stats_filter_enable_stats_for_all_methods_by_default
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

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/.
API shepherd assignee is @htuch
CC @envoyproxy/api-watchers: FYI only for changes made to api/envoy/.

🐱

Caused by: #14815 was opened by ggreenway.

see: more, trace.

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

@snowp snowp left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@lizan
Copy link
Member

lizan commented Jan 26, 2021

/lgtm v2-freeze

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.

5 participants