-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Ingest Manager] Enforce maximum open connections for some Fleet routes #71221
Conversation
The incrementing and decrementing aren't balanced and the 50 connections / max 100
logs 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 minutes
log 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) |
f24ac84
to
8001d72
Compare
💚 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