DM: don't redirect stderr to verbose.log

DM: don't redirect stderr to verbose.log

verbose.log is currently tricky to get to, so it's hurting more than it
helps to direct stderr there.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2178583002

Committed: https://skia.googlesource.com/skia/+/a1ce2163906eeb1dd5c543215c25cea856eda6cb

[mtklein_C, 2016-07-22 19:04:31]

Description was changed from

==========
DM: don't redirect stderr to verbose.log

verbose.log is currently tricky to get to, so it's hurting more than it
helps to direct stderr there.

BUG=skia:
==========

to

==========
DM: don't redirect stderr to verbose.log

verbose.log is currently tricky to get to, so it's hurting more than it
helps to direct stderr there.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2178583002
==========

[mtklein_C, 2016-07-22 19:04:45]

mtklein@chromium.org changed reviewers:
+ bungeman@google.com

[mtklein_C, 2016-07-22 19:04:45]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-07-22 19:04:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bungeman-skia, 2016-07-22 19:10:01]

lgtm

[commit-bot: I haz the power, 2016-07-22 19:23:51]

Description was changed from

==========
DM: don't redirect stderr to verbose.log

verbose.log is currently tricky to get to, so it's hurting more than it
helps to direct stderr there.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2178583002
==========

to

==========
DM: don't redirect stderr to verbose.log

verbose.log is currently tricky to get to, so it's hurting more than it
helps to direct stderr there.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2178583002

Committed:
https://skia.googlesource.com/skia/+/a1ce2163906eeb1dd5c543215c25cea856eda6cb
==========

[commit-bot: I haz the power, 2016-07-22 19:23:53]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/a1ce2163906eeb1dd5c543215c25cea856eda6cb

[jcgregorio, 2016-07-22 20:01:55]

On 2016/07/22 at 19:23:53, commit-bot wrote:
> Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/a1ce2163906eeb1dd5c543215c25cea856eda6cb

Actually verbose logs are easier to get to now that we've switched to swarming
since the outputs of the test are isolated.

Click on "shard #0 isolated output":

https://screenshot.googleplex.com/nGVt6yEVsZ1.png

That will bring you to a list of results files, which will include verbose.log:

https://screenshot.googleplex.com/qCL4QFT21R0.png

Click on the hash to see the file contents:

https://screenshot.googleplex.com/iUy87pVyYu5.png

[bungeman-skia, 2016-07-22 20:09:09]

On 2016/07/22 20:01:55, jcgregorio wrote:
> On 2016/07/22 at 19:23:53, commit-bot wrote:
> > Committed patchset #1 (id:1) as
> https://skia.googlesource.com/skia/+/a1ce2163906eeb1dd5c543215c25cea856eda6cb
> 
> Actually verbose logs are easier to get to now that we've switched to swarming
> since the outputs of the test are isolated.
> 
> Click on "shard #0 isolated output":
> 
> https://screenshot.googleplex.com/nGVt6yEVsZ1.png
> 
> That will bring you to a list of results files, which will include
verbose.log:
> 
> https://screenshot.googleplex.com/qCL4QFT21R0.png
> 
> Click on the hash to see the file contents:
> 
> https://screenshot.googleplex.com/iUy87pVyYu5.png

Part of this I believe is the additional issue that we don't seem to be
uploading the verbose.log when the step fails.

[mtklein, 2016-07-22 20:19:55]

> Part of this I believe is the additional issue that we don't seem to be
> uploading the verbose.log when the step fails.

Yep.  With that fixed, I'd be prepared to revert this if the verbose.log link
shows up on this page: https://screenshot.googleplex.com/nGVt6yEVsZ1.png