Merge lp://staging/~michael.nelson/rnr-server/add-multi-mech into lp://staging/rnr-server

Proposed by Michael Nelson
Status: Merged
Approved by: Łukasz Czyżykowski
Approved revision: 147
Merged at revision: 142
Proposed branch: lp://staging/~michael.nelson/rnr-server/add-multi-mech
Merge into: lp://staging/rnr-server
Diff against target: 201 lines (+139/-1)
7 files modified
.bzrignore (+1/-0)
buildout.cfg (+13/-0)
django_project/load_testing/config.cfg (+14/-0)
django_project/load_testing/test_scripts/rnr_review_stats.py (+56/-0)
fabfile/__init__.py (+1/-0)
fabfile/load_testing.py (+50/-0)
new_file_template.py (+4/-1)
To merge this branch: bzr merge lp://staging/~michael.nelson/rnr-server/add-multi-mech
Reviewer Review Type Date Requested Status
Łukasz Czyżykowski (community) Approve
Review via email: mp+52395@code.staging.launchpad.net

Commit message

[r=lukasz] Adds basic support for multi-mechanize load testing to dev env.

Description of the change

Overview
========

This branch adds a fab target for loadtesting (bug 729769):

 $ fab -f fabfile load_test:localhost:8000

Notes:
 1) I haven't added the code to automatically fire up a devserver if the host is not provided - I thought it better to be explicit here (ie. load_test will fail if the host you point it to isn't running).
 2) I wasn't able to add matplotlib and numpy to the dev (buildout) environment (even with python-numpy installed on the system I get https://pastebin.canonical.com/44341/. So currently both python-numpy and python-matplotlib need to be installed on the system to run this target.
 3) Currently we can't pass a --project-dir to multi-mechanize.py, so have to fiddle around with directories as in the fabfile (I'll check with cgoldberg if it's worth a fix on multi-mechanize).
 4) Currently we can't pass the target host as a run-time arg to multi-mech, so using os.environ to communicate this (and will work on a patch for multi-mech, see https://pastebin.canonical.com/44353/ for conv. with cgoldberg).

To test
=======
1) Run a local dev server: `make;make run`
2) `fab -f fabfile load_test:localhost:8000`

To post a comment you must log in.
Revision history for this message
Łukasz Czyżykowski (lukasz-czyzykowski) wrote :

Works like advertised.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches