Merge lp:~dave-cheney/juju-core/016-add-stress-test-script into lp:~juju/juju-core/trunk

Proposed by Dave Cheney
Status: Merged
Approved by: Gustavo Niemeyer
Approved revision: no longer in the source branch.
Merged at revision: 644
Proposed branch: lp:~dave-cheney/juju-core/016-add-stress-test-script
Merge into: lp:~juju/juju-core/trunk
Diff against target: 16 lines (+11/-0)
1 file modified
scripts/stresstest.bash (+11/-0)
To merge this branch: bzr merge lp:~dave-cheney/juju-core/016-add-stress-test-script
Reviewer Review Type Date Requested Status
The Go Language Gophers Pending
Review via email: mp+128620@code.launchpad.net

Description of the change

scripts: add stress test script

Simple script to ferret out concurency based issues.

https://codereview.appspot.com/6641045/

To post a comment you must log in.
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM

https://codereview.appspot.com/6641045/diff/1/scripts/stresstest.bash
File scripts/stresstest.bash (right):

https://codereview.appspot.com/6641045/diff/1/scripts/stresstest.bash#newcode1
scripts/stresstest.bash:1: #!/bin/bash
I suggest putting this under contrib/stresstest/run.sh instead, to make
it more obvious that this directory is unrelated to the functionality of
the system, and to establish the pattern to similar tools.

https://codereview.appspot.com/6641045/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added directory 'scripts'
2=== added file 'scripts/stresstest.bash'
3--- scripts/stresstest.bash 1970-01-01 00:00:00 +0000
4+++ scripts/stresstest.bash 2012-10-09 01:44:20 +0000
5@@ -0,0 +1,11 @@
6+#!/bin/bash
7+
8+# basic stress test
9+
10+set -e
11+
12+while true; do
13+ go get -u -v launchpad.net/juju-core/trivial
14+ export GOMAXPROCS=$[ 1 + $[ RANDOM % 128 ]]
15+ go test launchpad.net/juju-core/... 2>&1
16+done

Subscribers

People subscribed via source and target branches