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
tls: splitting config into downstream and upstream #34066
Conversation
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
alwayslink = True, | ||
) | ||
|
||
envoy_cc_extension( |
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.
I'm intentionally leaving a config library which includes both because for legacy servers we always had them under one extension, and there's a lot of build target churn otherwise. I can do the churn if you think it's worth it
@@ -6,7 +6,6 @@ load("@envoy_build_config//:extensions_build_config.bzl", "EXTENSIONS") | |||
_required_extensions = { | |||
"envoy.http.original_ip_detection.xff": "//source/extensions/http/original_ip_detection/xff:config", | |||
"envoy.request_id.uuid": "//source/extensions/request_id/uuid:config", |
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.
removing here is balanced out by adding to the exe explicitly.
I think it's more consistent to do that for required extensions, especially as we believe the openssl buid is stale
/retest |
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Ok this is now passing CI, sorry for noise |
Commit Message:
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:]