Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

Update loadtests to speak onepw protocol. #508

Merged
merged 1 commit into from Jan 21, 2014
Merged

Conversation

rfk
Copy link
Contributor

@rfk rfk commented Jan 21, 2014

Fortunately, this vastly simplifies the code!

@rfk
Copy link
Contributor Author

rfk commented Jan 21, 2014

@dannycoates r?

@rfk
Copy link
Contributor Author

rfk commented Jan 21, 2014

$ make bench
./loadsrunner.sh --users=20 --duration=1800 --agents=5 --include-file=./loadtests.py --python-dep=hawkauthlib loadtests.LoadTest.test_auth_server
[========================================================================] 100%
Duration: 1809.12 seconds
Hits: 1037195
Started: 2014-01-21 05:24:15.030781
Approximate Average RPS: 573
Average request time: 0.17s
Opened web sockets: 0
Bytes received via web sockets : 0

Success: 17700
Errors: 0
Failures: 6

6 occurrences of: 
    <type 'exceptions.AssertionError'>: 500 != 400  File "/usr/lib64/python2.6/unittest.py", line 278, in run    

That's not bad.

@rfk
Copy link
Contributor Author

rfk commented Jan 21, 2014

The 6 errors are all from calls to /account/create and all occurred pretty early in the loadtest run, when it would have been creating users at a higher rate in order to ramp up the pool of existing users. Possibly scrypt worker overload, possibly database overload - we need way to ship the full "tracebacks" for these to the logbox for analysis.

@rfk
Copy link
Contributor Author

rfk commented Jan 21, 2014

Yep, a second 30-minute test runs clean, so the error must have been due to extra account-creation load on an empty database.

@dannycoates
Copy link
Contributor

Awesome! r+

I'm very interested in those errors too.

rfk added a commit that referenced this pull request Jan 21, 2014
Update loadtests to speak onepw protocol.
@rfk rfk merged commit e1ad70d into master Jan 21, 2014
@jbonacci
Copy link

👍

@shane-tomlinson shane-tomlinson deleted the rfk/onepw-load-tests branch April 18, 2018 12:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants