-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Support dynamic_action in composite filter #33511
Support dynamic_action in composite filter #33511
Conversation
Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
…site_dynamic_action Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
/assign @htuch @tyxia @wbpcode @yanavlasov |
test/extensions/filters/http/composite/composite_filter_integration_test.cc
Show resolved
Hide resolved
test/extensions/filters/http/composite/composite_filter_integration_test.cc
Show resolved
Hide resolved
/lgtm api |
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.
/lgtm api
--------- Signed-off-by: Yanjun Xiang <yanjunxiang@google.com> Signed-off-by: Fernando Cainelli <fernando.cainelli-external@getyourguide.com>
--------- Signed-off-by: Yanjun Xiang <yanjunxiang@google.com>
This is a follow up commit of #33013 to support dynamic_action in composite filter.
In #33013, the dynamic_action support is intentionally left out as there is
cluster manager initialization sequence issue (#33218), which is addressed by #33221.
This PR also extended the composite_filter_integration_test.cc:CompositeFilterIntegrationTest to support three parameters:
Note, this combination is skipped: downstream only filter / config the filter in upstream, as this won't work.
Also, as per-route config is added in HCM, i.e, downstream, thus all the per-route tests are only done with filter configured in downstream.
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:]