travis: If the comparison-tool fails, dump the tail of the debug log #5025

pull theuni wants to merge 1 commits into bitcoin:master from theuni:travis-print-log changing 1 files +4 −0

theuni commented at 8:47 PM on October 1, 2014: member

The entire debug log would be huge, and could cause issues for automated tools like travis. Printing 200 lines is an initial guess at a reasonable number, more may be required.

travis: If the comparison-tool fails, dump the tail of the debug log

The entire debug log would be huge, and could cause issues for automated tools
like travis. Printing 200 lines is an initial guess at a reasonable number,
more may be required.

5ad450a65a

sipa commented at 8:48 PM on October 1, 2014: member

ut ACK
theuni commented at 9:02 PM on October 1, 2014: member

@sipa might be worth pulling this into your headers-first branch and letting it fail so we can see if it is enough to be useful.
TheBlueMatt commented at 10:21 PM on October 1, 2014: member

Why not just cat instead of tail? Maybe the error was right at the beginning and it kept running.

theuni commented at 10:34 PM on October 1, 2014: member

See description. Travis has a limit for how big the output can be. Here's what the log looks like after a successful run:

cory@cory-i7:~/dev/bitcoin(travis-print-log)$ wc -l .bitcoin/regtest/debug.log
26015 .bitcoin/regtest/debug.log
cory@cory-i7:~/dev/bitcoin(travis-print-log)$ du -h .bitcoin/regtest/debug.log 
1.7M    .bitcoin/regtest/debug.log

Things would get ugly if we cat'd that.

TheBlueMatt commented at 10:41 PM on October 1, 2014: member

Well, we should just include the full debug.log with the copy over to bitcoincore.org or so.
sipa commented at 1:58 AM on October 2, 2014: member

This is much easier, and shouldn't hurt anyway.
TheBlueMatt commented at 2:21 AM on October 2, 2014: member

Yes, sorry, utACK on this...but a suggestion for when build output happens: make sure to include debug.log.
laanwj commented at 6:33 AM on October 2, 2014: member

Well, we should just include the full debug.log with the copy over to bitcoincore.org or so.

Once Travis grows support for producing multiple build results and securely uploading them, that would be the idea. But tailing 200 lines is a useful improvement already. (ut)ACK.
laanwj merged this on Oct 2, 2014
laanwj closed this on Oct 2, 2014
laanwj referenced this in commit 3b390723c1 on Oct 2, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors