From d78d168096fbbf39a7a6604a38844f8a8b2b8bb1 Mon Sep 17 00:00:00 2001 From: Tuomas Tynkkynen Date: Sat, 20 Jun 2015 21:15:39 +0300 Subject: [PATCH] travis-nox-review-pr: Try to detect if the build ran out of memory Travis builds sometimes fail because Travis doesn't have much memory. When it happens, the failure reason is often hard to see in the logs and is confusing (especially for new contributors). Try to detect OOM errors in dmesg and give a warning when it happens. --- maintainers/scripts/travis-nox-review-pr.sh | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/maintainers/scripts/travis-nox-review-pr.sh b/maintainers/scripts/travis-nox-review-pr.sh index 3c3f9151d83..54549955747 100755 --- a/maintainers/scripts/travis-nox-review-pr.sh +++ b/maintainers/scripts/travis-nox-review-pr.sh @@ -31,7 +31,15 @@ elif [[ $1 == build ]]; then echo "=== Not a pull request" else echo "=== Checking PR" - nox-review pr ${TRAVIS_PULL_REQUEST} + + if ! nox-review pr ${TRAVIS_PULL_REQUEST}; then + if sudo dmesg | egrep 'Out of memory|Killed process' > /tmp/oom-log; then + echo "=== The build failed due to running out of memory:" + cat /tmp/oom-log + echo "=== Please disregard the result of this Travis build." + fi + exit 1 + fi fi # echo "=== Checking tarball creation" # nix-build pkgs/top-level/release.nix -A tarball