-
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
Enable ext_proc MxN API #37774
base: main
Are you sure you want to change the base?
Enable ext_proc MxN API #37774
Conversation
Signed-off-by: Yanjun Xiang <[email protected]>
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
Signed-off-by: Yanjun Xiang <[email protected]>
/assign @yanavlasov @tyxia |
/assign @jmarantz |
Signed-off-by: Yanjun Xiang <[email protected]>
Signed-off-by: Yanjun Xiang <[email protected]>
I'll defer to Yan for this as I feel there might be a good process for evaluating this. |
@yanjunxiang-google AFAIK, we have not done the load test for this feature? If we want to expose this API now, do you want to add a warning comment, followed by a TODO action to load test it. |
@yanjunxiang-google please resolve merge conflict. /wait |
Signed-off-by: Yanjun Xiang <[email protected]>
@yanjunxiang-google Seems like there is a bit of whack-a-mole with the current.yaml /wait |
Enable ext_proc MxN API