cat all logs in sandcastle output

Summary:
Since we enabled parallelism in valgrind the logs now go to
t/valgrind_log-*, which doesn't match our pattern t/log-*, so those files don't
get printed in the sandcastle output. For this regex we really just want
to exclude t/run-* (these are shell scripts), so let's do that explicitly.

Test Plan:
ran the commands locally, will also look at sandcastle
results on this diff which will include these changes

Reviewers: arahut, sdong, IslamAbdelRahman, kradhakrishnan, wanning

Reviewed By: wanning

Subscribers: andrewkr, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D61557
main
Andrew Kryczka 8 years ago
parent 9253767a6a
commit 7cc0dbd66f
  1. 4
      arcanist_util/config/RocksDBCommonHelper.php

@ -127,8 +127,8 @@ function getSteps($applyDiff, $diffID, $username, $test) {
} }
$cmd = $cmd . " cat /tmp/precommit-check.log" $cmd = $cmd . " cat /tmp/precommit-check.log"
. "; for f in `ls t/log-*`; do echo \$f; cat \$f; done;" . "; shopt -s extglob; for f in `ls t/!(run-*)`; do echo \$f"
. "[[ \$exit_code -eq 0 ]]"; . "; cat \$f; done; shopt -u extglob; [[ \$exit_code -eq 0 ]]";
assert(strlen($cmd) > 0); assert(strlen($cmd) > 0);
$run_test = array( $run_test = array(

Loading…
Cancel
Save