-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add H2 stress tests #12610
Add H2 stress tests #12610
Conversation
…re not left stuck on locks Signed-off-by: Ludovic Orban <[email protected]>
Thanks, that exactly the issue we've been observing! |
Having the main branch builds (not PRs or other branches) run with |
Setup done here https://jenkins.webtide.net/job/nightlies/job/stress/ 12.0.x 12.1.x setup to run Build with this branch: https://jenkins.webtide.net/job/nightlies/job/stress/job/jetty-stress-run/12/ |
...etty-ee10-test-http2-webapp/src/test/java/org/eclipse/jetty/test/webapp/StressHTTP2Test.java
Outdated
Show resolved
Hide resolved
...etty-ee10-test-http2-webapp/src/test/java/org/eclipse/jetty/test/webapp/StressHTTP2Test.java
Outdated
Show resolved
Hide resolved
...etty-ee10-test-http2-webapp/src/test/java/org/eclipse/jetty/test/webapp/StressHTTP2Test.java
Outdated
Show resolved
Hide resolved
...etty-ee10-test-http2-webapp/src/test/java/org/eclipse/jetty/test/webapp/StressHTTP2Test.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Ludovic Orban <[email protected]>
…tty-12.0.x/h2-reset-stress-test
Add H2 input and output stress test that uses client aborts and asserts that threads are not left stuck on locks.
Ideally, this test (and probably all other stress tests) should be ran at least once before every release; but probably they should be running nightly, and be excluded of PR builds. @olamy what's the current policy?