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.

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

Set max concurrent stream option for grpc server #308

Closed
wants to merge 1 commit into from

Conversation

Lookyan
Copy link
Contributor

@Lookyan Lookyan commented Mar 31, 2018

Fixes #306
Fixes #299
For large clusters in current implementation when envoy creates new EDS stream for each cluster we need to increase this limit.

Signed-off-by: Alexander Lukyanchenko <digwnews@gmail.com>
@davecheney
Copy link
Contributor

davecheney commented Apr 3, 2018

Thanks for this. I will land this with some comments from my version of this patch once I've built a repro case.

@davecheney
Copy link
Contributor

Fixed by ec106a1

Sorry for the weird commit, I added a commit to your branch before rebasing on master and merging.

Thanks again for working on this.

@davecheney
Copy link
Contributor

Fixes #306
Fixes #299

sunjayBhatia pushed a commit that referenced this pull request Jan 30, 2023
Fixes some inaccurate log statements. Also fixes a bug where
the default namespace for Contours was being inadvertently
reassigned during the two-Contour test, impacting other tests.

Signed-off-by: Steve Kriss <krisss@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants