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.

Sign up for GitHub

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

Jump to bottom

network_ext_proc: create the boilerplate l4 ext_proc #38877

Merged
merged 12 commits into from
Mar 28, 2025
Merged

network_ext_proc: create the boilerplate l4 ext_proc #38877

merged 12 commits into from
Mar 28, 2025

Conversation

botengyao
Copy link
Member

@botengyao botengyao commented Mar 24, 2025 •
edited
Loading

#38721

Add the boilerplate of the l4 ext_proc filter

Commit Message:
Additional Description:
Risk Level: low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

@botengyao
initial commit
7b252bc 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
add boilerplate for l4 ext_proc
97c1efe 
Signed-off-by: Boteng Yao <[email protected]>
@repokitteh-read-only RepoKitteh - Read Only
Copy link

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: #38877 was opened by botengyao.

see: more, trace.

@botengyao
clean up right now
a8d499c 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao botengyao marked this pull request as ready for review March 24, 2025 20:51
@botengyao
Copy link
Member Author

This could be a quick review, thanks!

/assign @yanavlasov

@botengyao
fix format
58fc8d1 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
Merge remote-tracking branch 'upstream/main' into boilerplate_l4_ext_…
fd15b7f 
…proc

Signed-off-by: Boteng Yao <[email protected]>
@botengyao
fix metadata
aaaf5d9 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
fix ci
18ca347 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
increse coverage
d8b7c0b 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
add wip docs
f9961ca 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
fix ci
c17d1ab 
Signed-off-by: Boteng Yao <[email protected]>
@repokitteh-read-only RepoKitteh - Read Only
Copy link

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

🐱

Caused by: #38877 was synchronize by botengyao.

see: more, trace.

@botengyao
fix ci
b3cfb39 
Signed-off-by: Boteng Yao <[email protected]>
yanavlasov
yanavlasov requested changes Mar 26, 2025
View reviewed changes
Copy link
Contributor

@yanavlasov yanavlasov left a comment

Choose a reason for hiding this comment

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

/wait

@botengyao
update docs
891d602 
Signed-off-by: Boteng Yao <[email protected]>
@botengyao
Copy link
Member Author

/retest

@markdroth
Copy link
Contributor

/lgtm api

@botengyao
Copy link
Member Author

ping

@yanavlasov yanavlasov merged commit ed6c7e3 into envoyproxy:main Mar 28, 2025
26 checks passed
@mathetake mathetake mentioned this pull request Mar 28, 2025
Merged
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

@markdroth markdroth

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@botengyao @markdroth @yanavlasov