-
Notifications
You must be signed in to change notification settings - Fork 8.5k
[Ingest Manager] Enforce maximum open connections for some Fleet routes #71221
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
Conversation
|
The incrementing and decrementing aren't balanced and the 50 connections / max 100logs show 0 open connections after the test has stopped. Hit /enroll with 500 connections for 5 minutes.log from above run (max connections set to 100) logs show 100 (not 0) open connections after the test has stopped. Hit /enroll with 500 connections for 5 minuteslog from above run (max connections set to 1000) Note the logs show 440 (not 0) open connections after the test has stopped. In that case, it's notable that the load test program showed 440 @kobelb @restrry Any ideas why concurrent requests keeps climbing? I don't know our/hapi's lifecycle very well. Are there different/additional hooks we should use? Can we change something about the places we increment/decrement now? |
|
lot of changes here! cc @roncohen |
|
@kobelb If you can take a look that would be great! |
|
Hey @ph, I took a look at what was likely causing the missing decrements this morning. The relevant conversation starts here: #70495 (comment) |
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
Start with #70775 for platform lifecycle hooks and add code from #70495
Fleet-specific code is in f24ac84
Draft because it's missing some of the decrements and doesn't return to 0 connections. See comments