Gsutil ls misses should not be treated as failures by bisect.

Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output. The gsutil
module calls a wrapper that contains retry logic to avoid
cases of flake.

Actual failures of gsutil remain a possibility, and this
change would make it a little tricky to diagnose since we
won't have purple steps, but in this case all that will
happen is that builds will either time out or the download
and unzip step would fail.

Google Storage going down has not as of yet been a problem
for bisect.

R=sullivan,dtu@chromium.org
BUG=634045
Committed: https://chromium.googlesource.com/chromium/tools/build/+/e5316b18800771931a523593991755d5a978f840

[sullivan, 2016-08-03 18:53:11]

This makes sense to me, but can you explain at a high level if this failure is
ever a real problem, and if so, how that would get surfaced?

[RobertoCN, 2016-08-10 18:11:30]

Description was changed from

==========
Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output.

R=sullivan,dtu@chromium.org
BUG=634045
==========

to

==========
Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output. The gsutil module calls a
wrapper that contains retry logic to avoid cases of flake.

Actual failures of gsutil remain a possibility, and this change would make it a
little tricky to diagnose since we won't have purple steps, but in this case all
that will happen is that builds will either time out or the download and unzip
step would fail.

Google Storage going down has not as of yet been a problem for bisect.

R=sullivan,dtu@chromium.org
BUG=634045
==========

[RobertoCN, 2016-08-10 18:12:24]

Description was changed from

==========
Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output. The gsutil module calls a
wrapper that contains retry logic to avoid cases of flake.

Actual failures of gsutil remain a possibility, and this change would make it a
little tricky to diagnose since we won't have purple steps, but in this case all
that will happen is that builds will either time out or the download and unzip
step would fail.

Google Storage going down has not as of yet been a problem for bisect.

R=sullivan,dtu@chromium.org
BUG=634045
==========

to

==========
Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output. The gsutil
module calls a wrapper that contains retry logic to avoid
cases of flake.

Actual failures of gsutil remain a possibility, and this
change would make it a little tricky to diagnose since we
won't have purple steps, but in this case all that will
happen is that builds will either time out or the download
and unzip step would fail.

Google Storage going down has not as of yet been a problem
for bisect.

R=sullivan,dtu@chromium.org
BUG=634045
==========

[RobertoCN, 2016-08-10 18:12:45]

Updated CL description PTAL

[sullivan, 2016-08-10 18:31:23]

lgtm

[RobertoCN, 2016-08-10 19:14:08]

The CQ bit was checked by robertocn@chromium.org

[commit-bot: I haz the power, 2016-08-10 19:14:11]

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

[commit-bot: I haz the power, 2016-08-10 19:17:51]

Description was changed from

==========
Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output. The gsutil
module calls a wrapper that contains retry logic to avoid
cases of flake.

Actual failures of gsutil remain a possibility, and this
change would make it a little tricky to diagnose since we
won't have purple steps, but in this case all that will
happen is that builds will either time out or the download
and unzip step would fail.

Google Storage going down has not as of yet been a problem
for bisect.

R=sullivan,dtu@chromium.org
BUG=634045
==========

to

==========
Gsutil ls misses should not be treated as failures by bisect.

The purpose of this is to declutter bisect output. The gsutil
module calls a wrapper that contains retry logic to avoid
cases of flake.

Actual failures of gsutil remain a possibility, and this
change would make it a little tricky to diagnose since we
won't have purple steps, but in this case all that will
happen is that builds will either time out or the download
and unzip step would fail.

Google Storage going down has not as of yet been a problem
for bisect.

R=sullivan,dtu@chromium.org
BUG=634045

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/e5316b18800771931a52...
==========

[commit-bot: I haz the power, 2016-08-10 19:17:52]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/e5316b18800771931a52...