<div dir="ltr"><div><div><div><div>So I went ahead and did a few steps that will hopefully make this better.<br><br></div>1) I moved lp:goose into its own directory, we actually had it sharing the source dependencies of juju-core so we could know if we broke anything, but now we can trust dependencies.tsv so we can handle this a bit better.<br>

<br>2) I added a "killall mongod", which would help ensure that we don't have any 
mongo's running to block the flock.<br><br></div>2) Changed the "verify_command" to both run godeps -u, as well as create a TMPDIR that gets cleaned up. The new logic is complex enough, it really feels like it should be a script inside of the source tree that we just run, but anyway, here it is:<br>

      verify_command = export GOPATH=/home/tarmac/trees; godeps -u dependencies.tsv && export TMPDIR=`mktemp -d --tmpdir juju-core-test.XXXXXX` && go fmt ./... && go install ./... && go test ./...; STATUS=$?; rm -rf $TMPDIR; killall -u tarmac mongod; exit $STATUS<br>

<br></div>I made sure to submit a branch that actually fails, to ensure that the exit $STATUS stuff triggers correctly.<br><br></div><div>This probably won't make tests any less flaky, but at least they'll leave a little less cruft behind.<br>
</div><div><br></div><div>John<br>=:-><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 1, 2014 at 1:12 PM, David Cheney <span dir="ltr"><<a href="mailto:david.cheney@canonical.com" target="_blank">david.cheney@canonical.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">or use tmpwatcher ?<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Apr 1, 2014 at 7:47 PM, Andrew Wilkins<br>
<<a href="mailto:andrew.wilkins@canonical.com">andrew.wilkins@canonical.com</a>> wrote:<br>
> Much as I'd like the tests to clean up after themselves, can we just create<br>
> a temporary $TMPDIR and blow it away after the test run?<br>
><br>
><br>
> On Tue, Apr 1, 2014 at 4:40 PM, John Meinel <<a href="mailto:john@arbash-meinel.com">john@arbash-meinel.com</a>> wrote:<br>
>><br>
>> It seems our test suite still leaks /tmp/test-* directories occasionally.<br>
>> Today I was working on setting up tarmac managing the 1.18 branch and<br>
>> noticed it ran out of disk space running the "juju resolved --retry". I<br>
>> ended up cleaning out approx 7GB of dead test directories.<br>
>><br>
>> Anyway, if you ever have to kick the bot, please at least check 'df' to<br>
>> see if we might be low on disk space.<br>
>><br>
>> John<br>
>> =:-><br>
>><br>
>><br>
>> --<br>
>> Juju-dev mailing list<br>
>> <a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
>> Modify settings or unsubscribe at:<br>
>> <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
>><br>
><br>
><br>
> --<br>
> Juju-dev mailing list<br>
> <a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
> Modify settings or unsubscribe at:<br>
> <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
><br>
</div></div></blockquote></div><br></div>