Skip to content
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

Remove runtime guard copy_response_code_to_downstream_stream_info #33639

Conversation

jbohanon
Copy link
Contributor

Fixes #33615
Commit Message: Remove runtime guard copy_response_code_to_downstream_stream_info
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Copy link

CC @envoyproxy/runtime-guard-changes: FYI only for changes made to (source/common/runtime/runtime_features.cc).

🐱

Caused by: #33639 was opened by jbohanon.

see: more, trace.

Signed-off-by: Jacob Bohanon <[email protected]>
@jbohanon jbohanon force-pushed the jbohanon/remove-response-code-to-downstream-guard branch from 50281c5 to 5926fc0 Compare April 18, 2024 00:09
adisuissa
adisuissa previously approved these changes Apr 18, 2024
Copy link
Contributor

@adisuissa adisuissa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@adisuissa adisuissa self-assigned this Apr 18, 2024
@adisuissa
Copy link
Contributor

/retest

@jbohanon
Copy link
Contributor Author

cc @yanavlasov (as approver of original PR) have a chance to look at this?

I'll fix the changelog once I have the approvals; no need to thrash CI until then

@adisuissa
Copy link
Contributor

Author requested Yan's approval:
/assign @yanavlasov

@yanavlasov
Copy link
Contributor

LGTM. Please resolve merge conflict.

/wait

Copy link
Contributor

@adisuissa adisuissa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jbohanon
Copy link
Contributor Author

Can someone with write access please merge? @adisuissa @yanavlasov

@adisuissa adisuissa merged commit 835ed26 into envoyproxy:main Apr 25, 2024
53 checks passed
cainelli pushed a commit to cainelli/envoy that referenced this pull request Apr 25, 2024
alyssawilk pushed a commit to alyssawilk/envoy that referenced this pull request Apr 29, 2024
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.

envoy_reloadable_features_copy_response_code_to_downstream_stream_info deprecation
3 participants