Skip to content

xDS: change ext_proc drain process to work directionally#45901

Open
markdroth wants to merge 4 commits into
envoyproxy:mainfrom
markdroth:xds_ext_proc_directional_drain
Open

xDS: change ext_proc drain process to work directionally#45901
markdroth wants to merge 4 commits into
envoyproxy:mainfrom
markdroth:xds_ext_proc_directional_drain

Conversation

@markdroth

Copy link
Copy Markdown
Contributor

Commit Message: xDS: change ext_proc drain process to work directionally
Additional Description: Allow draining request and response bodies independently of each other in FULL_DUPLEX_STREAMED and GRPC body send modes.
Risk Level: Low
Testing: N/A
Docs Changes: Included in PR
Release Notes: N/A
Platform Specific Features: N/A

CC @yanjunxiang-google

Signed-off-by: Mark D. Roth <roth@google.com>
@repokitteh-read-only

Copy link
Copy Markdown

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @adisuissa
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #45901 was opened by markdroth.

see: more, trace.

Signed-off-by: Mark D. Roth <roth@google.com>
Signed-off-by: Mark D. Roth <roth@google.com>
Signed-off-by: Mark D. Roth <roth@google.com>
@paul-r-gall

Copy link
Copy Markdown
Contributor

@yanjunxiang-google and @tyxia PTAL.

@yanjunxiang-google

Copy link
Copy Markdown
Contributor

We might have some other changes when we actually support FULL_DUPLEX_STREAMED mode graceful drain in Envoy. Thus I would rather hold off any API changes for now until we have a PoC which clearly demonstrates what needs to be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants