[asterisk-dev] [Code Review] 4007: testsuite: pick test suite temp dir based on free space

[Matt Jordan]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4007/#review13366
-----------------------------------------------------------

Ship it!


Blech. Thanks CentOS 7. :-(

This patch will work just fine for the most of the testsuite, but there will be some tests that still try to stick themselves in /tmp/asterisk-testsuite (namely, the Lua ones). That shouldn't prevent this patch from going in, as we're slowly converting those over to Python anyway.

- Matt Jordan


On Sept. 19, 2014, 8:23 a.m., Scott Griepentrog wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4007/
> -----------------------------------------------------------
> 
> (Updated Sept. 19, 2014, 8:23 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> On CentOS 7, the /tmp path is now a ramdisk which doesn't have enough space for a full Asterisk testsuite run.  This change allows causes the testsuite to switch to using /var/tmp/asterisk-testsuite in the case where the free space is larger there.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/lib/python/asterisk/asterisk.py 5608 
> 
> Diff: https://reviewboard.asterisk.org/r/4007/diff/
> 
> 
> Testing
> -------
> 
> Checked operation on C7 and C6 and the correct directory was used on both.
> 
> 
> Thanks,
> 
> Scott Griepentrog
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140919/8e4796a6/attachment-0001.html>