Compress uploaded android logs

Compress uploaded android logs

gzip files so clients can download faster.

BUG=362787
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=267983

[luqui, 2014-05-02 18:21:42]

lgtm (I'm no owner though)

[Isaac (away), 2014-05-02 18:53:06]

The CQ bit was checked by ilevy@chromium.org

[commit-bot: I haz the power, 2014-05-02 18:53:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ilevy@chromium.org/268743002/20001

[commit-bot: I haz the power, 2014-05-02 18:53:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-02 18:53:42]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[iannucci, 2014-05-02 18:56:53]

https://codereview.chromium.org/268743002/diff/20001/build/android/buildbot/b...
File build/android/buildbot/bb_device_steps.py (right):

https://codereview.chromium.org/268743002/diff/20001/build/android/buildbot/b...
build/android/buildbot/bb_device_steps.py:576: RunCmd([bb_utils.GSUTIL_PATH,
'cp', '-z', 'txt', logcat_file,
isn't gsutil content type deprecated/detected with `file <thing to upload>`
nowadays?

[Isaac (away), 2014-05-02 18:59:16]

https://codereview.chromium.org/268743002/diff/20001/build/android/buildbot/b...
File build/android/buildbot/bb_device_steps.py (right):

https://codereview.chromium.org/268743002/diff/20001/build/android/buildbot/b...
build/android/buildbot/bb_device_steps.py:576: RunCmd([bb_utils.GSUTIL_PATH,
'cp', '-z', 'txt', logcat_file,
On 2014/05/02 18:56:53, iannucci wrote:
> isn't gsutil content type deprecated/detected with `file <thing to upload>`
> nowadays?

I'm not sure what you mean.  Adding '.txt' to file allows gsutil to
automatically set text/plain based on extension.  It's also required to use
their -z option, per
https://developers.google.com/storage/docs/gsutil/commands/cp#options

[iannucci, 2014-05-02 19:12:52]

okie smokes, lgtm :)

[Isaac (away), 2014-05-02 19:16:20]

The CQ bit was checked by ilevy@chromium.org

[commit-bot: I haz the power, 2014-05-02 19:16:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ilevy@chromium.org/268743002/20001

[commit-bot: I haz the power, 2014-05-03 00:07:32]

Change committed as 267983