-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
runtime: remove HCM stream error runtime override #16040
Conversation
Remove envoy.reloadable_features.hcm_stream_error_on_invalid_message now that it is old enough. Signed-off-by: Alex Konradi <akonradi@google.com>
Signed-off-by: Alex Konradi <akonradi@google.com>
/assign @alyssawilk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Small nit
/wait
Signed-off-by: Alex Konradi <akonradi@google.com>
Signed-off-by: Alex Konradi <akonradi@google.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Needs a main merge :(
/wait
…e-runtime-guard-16007 Signed-off-by: Alex Konradi <akonradi@google.com>
I think something conflicted again! (a merged PR in the meantime?) :( Release note merges are the worst |
…e-runtime-guard-16007 Signed-off-by: Alex Konradi <akonradi@google.com>
It was probably my other cleanup PR, #16041 |
* Remove HCM stream error runtime override. Remove envoy.reloadable_features.hcm_stream_error_on_invalid_message now deprecated Signed-off-by: Alex Konradi <akonradi@google.com> Signed-off-by: Gokul Nair <gnair@twitter.com>
Commit Message: Remove HCM stream error runtime override
Additional Description:
Remove envoy.reloadable_features.hcm_stream_error_on_invalid_message now
that it is old enough.
Risk Level: low
Testing: ran affected tests
Docs Changes: none
Release Notes: document removed override
Platform Specific Features: none
Fixes #16007