Skip to content

network: added network ext proc basic callout functions#39185

Merged
yanavlasov merged 18 commits intoenvoyproxy:mainfrom
botengyao:network_ext_proc_basic
Apr 23, 2025
Merged

network: added network ext proc basic callout functions#39185
yanavlasov merged 18 commits intoenvoyproxy:mainfrom
botengyao:network_ext_proc_basic

Conversation

@botengyao
Copy link
Copy Markdown
Member

@botengyao botengyao commented Apr 21, 2025 •
edited
Loading

Added the basic ext_proc gRPC callout functions without the data loss that was addressed in #38778.
gRPC client was shared with HTTP ext_proc that was templated by #38898

Pending:

  • COUNTER will be in the followup RPs.
  • filter_metadata handling is pending.
  • log info is pending.
  • timeout setting is pending
  • flow control is pending

Commit Message:
Additional Description:
Risk Level: low
Testing:

#38721

@botengyao
inititail
6b9f373 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
minor fix
37852c4 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
fix build
7bbd788 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
Merge remote-tracking branch 'upstream/main' into network_ext_proc_basic
d62613b 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
added tests
23c0c16 
Signed-off-by: Boteng Yao <boteng@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

repokitteh-read-only Bot commented Apr 21, 2025

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #39185 was opened by botengyao.

see: more, trace.

@botengyao
clean up
769c9ec 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
add more tests
8de8518 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
change comments
ea0f359 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao botengyao changed the title [Draft]network: added network ext proc basic callout functions network: added network ext proc basic callout functions Apr 21, 2025
@botengyao
clean up
f3c07b7 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
clean up
3da770c 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
add more test
7d512f7 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
minor improvemnt
0e83593 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
fix spell
279ff12 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
fix tests and format
dce0687 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
increase coverage
03d679c 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
increase coverage
0577640 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao botengyao marked this pull request as ready for review April 22, 2025 16:08
@botengyao botengyao requested a review from yanavlasov as a code owner April 22, 2025 16:08
@botengyao
fix comments
c831aca 
Signed-off-by: Boteng Yao <boteng@google.com>
@botengyao
Copy link
Copy Markdown
Member Author

botengyao commented Apr 22, 2025

/retest

@botengyao
fix test
e54ef5f 
Signed-off-by: Boteng Yao <boteng@google.com>
@yanavlasov yanavlasov merged commit a17ff84 into envoyproxy:main Apr 23, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yanavlasov yanavlasov yanavlasov approved these changes

Assignees

@yanavlasov yanavlasov

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@botengyao @yanavlasov