Travis log too long

[martinpopel]

Travis CI has 4MB limit on the log file, but current T2T tests exceed this limit.
As a result I cannot see the real cause why a given build failed.
Most of the log (if downloaded as a raw file) is full of download progress bars, e.g.

  Downloading mpmath-1.0.0.tar.gz (511kB)
�[?25l
�[K    2% |▋                               | 10kB 53.6MB/s eta 0:00:01
�[K    4% |█▎                              | 20kB 41.3MB/s eta 0:00:01
�[K    6% |██                              | 30kB 42.5MB/s eta 0:00:01
�[K    8% |██▋                             | 40kB 41.3MB/s eta 0:00:01
�[K    10% |███▏                            | 51kB 42.6MB/s eta 0:00:01

This is the first time I see this problem, but it seems that until a new pip version with --progress-bar off option is released, we should use a workaround pip install package | cat && exit ${PIPESTATUS[0]} or set -o pipefail; pip install package | cat.

[rsepassi]

how about pip install -q to have the install be quiet?

[rsepassi]

And git clone -q

[rsepassi]

And thank you for spotting this.

[martinpopel]

According to pypa/pip#2756 if we provide the --quiet option, pip does not display exceptions, which is a bit unfortunate for Travis. That said, it would be still better than the current state.
And yes, git clone -q is a good idea.

Unfortunately, it seems Travis is now completely broken. See e.g. https://travis-ci.org/tensorflow/tensor2tensor/builds/329575693 (note that a given commit just changes README.md).
In #524 I have tried to silence the pip, but Travis again failed for other reasons.

[rsepassi]

I think it’s fine not to see the pip exceptions in Travis. The focus of the tests is not to test pip install. If a pip command fails, we'll see it and can retry locally to see the error.

…

On Thu, Jan 18, 2018 at 12:02 PM Martin Popel ***@***.***> wrote: According to pypa/pip#2756 <pypa/pip#2756> if we provide the --quiet option, pip does not display exceptions, which is a bit unfortunate for Travis. That said, it would be still better than the current state. And yes, git clone -q is a good idea. Unfortunately, it seems Travis is now completely broken. See e.g. https://travis-ci.org/tensorflow/tensor2tensor/builds/329575693 (note that a given commit just changes README.md). In #524 <#524> I have tried to silence the pip, but Travis again failed for other reasons. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#523 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABEGW7Dh5eapBCtAMw3hq-f6A_dogrSQks5tL6NzgaJpZM4Rgoy6> .

[martinpopel]

I changed | cat && exit ${PIPESTATUS[0]} into pip install -q in 61005b0
This commit (most probably disregarding its content) caused Travis rebuild and now it work and the raw log has just 82 KiB.
I was not able to find a place where to set git clone -q (git clone is done automatically by Travis).
So this issue can be closed once #524 is merged (sorry for combining two topics in one PR).