Code review comment for lp:~dave-cheney/juju-core/016-add-stress-test-script

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/

« Back to merge proposal