Nunc-stans is targeted to high load, high stress environments.
A inbuilt stress tested should be added that can exercise various parts of the nunc-stans library.
attachment 0001-Ticket-58-Add-stress-test.patch
{{{ [==========] Running 1 test(s). [ RUN ] ns_stress_test min idle threads: 0 max work q size: 201 max event q size: 326 min idle threads: 0 max work q size: 1000 max event q size: 463 3 job_count: 1000 client_fail_count: 0 server_fail_count: 0 client_success_count: 1000 server_success_count: 1000 [ OK ] ns_stress_test [==========] 1 test(s) run. [ PASSED ] 1 test(s). }}}
It's a great idea to add a stress test.
I have a couple of questsions/requests... Could you tell me how to run the stress test? :) It'd be nice to have a comment section or a readme file how much nunc-stans is stressed and what is being tested... Probably, for the other cmocka test cases, as well? cmocka already has test cases. Are they supposed to be in the libnunc-stans.so? (I guess not?) Currently, we are building nunc-stans in the 389-ds-base build which takes a tarball of nunc-stans. Are we changing the method of releasing nunc-stans? For instance, are we planning to turn it to an independent package? You may want to run autoconf/automake and push the generated files, as well? ;)
Okay, so I will update the readme? You do make; make check; ./cmocka_stress_test.
At the moment it just loads in a whole bunch of jobs to run and sees if any are missed. I will add more tests soon (IE timer jobs, etc). I will document this too.
I don't know what you mean about the presence of them in libnunc-stans. They should be external? We would only need the tests for development.
I will add the updated autotools files :)
Updated autoconf files 0001-Ticket-58-Add-stress-test.2.patch
This updates building.md to reflect "make; make check; ./cmocka_stress_test"
This also updates the autotools files.
Please don't use c++/java style comments, the rest looks good.
Fixed comment style.
commit a33c41f Writing objects: 100% (17/17), 8.21 KiB | 0 bytes/s, done. Total 17 (delta 12), reused 0 (delta 0) To ssh://git.fedorahosted.org/git/nunc-stans.git 7a4ce78..a33c41f master -> master
Replying to [comment:7 firstyear]:
Thanks!
Login to comment on this ticket.