Revert of Ensure that builders that use goma strip absolute paths by default.

Revert of Ensure that builders that use goma strip absolute paths by default. (patchset #1 id:1 of https://codereview.chromium.org/2873513003/ )

Reason for revert:
Speculatively reverting in case this is causing goma crashes (see crbug.com/720074)

Original issue's description:
> Reland of Ensure that builders that use goma strip absolute paths by default. (patchset #1 id:1 of https://codereview.chromium.org/2843403004/ )
> 
> Reason for revert:
> Relanding, since this was not a cause of the goma outage.
> 
> Original issue's description:
> > Revert of Ensure that builders that use goma strip absolute paths by default. (patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )
> > 
> > Reason for revert:
> > Reverting in case this is related to the goma errors in https://crbug.com/716089
> > 
> > Original issue's description:
> > > Ensure that builders that use goma strip absolute paths by default.
> > > 
> > > This CL tweaks the MB configs so that any builder that is using goma
> > > also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> > > This helps ensure that we get the best goma cache hit rate possible.
> > > We don't just make the default equal to use_goma=true because doing
> > > so may make it harder for devs to debug binaries, and we want devs
> > > to be able to both debug binaries and use goma by default; the
> > > tradeoff is that their builds are slightly slower.
> > > 
> > > R=thakis@chromium.org, jochen@chromium.org
> > > BUG=603286, 712790
> > > 
> > > Review-Url: https://codereview.chromium.org/2837863005
> > > Cr-Commit-Position: refs/heads/master@{#467574}
> > > Committed: https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4e79642b9edc3
> > 
> > TBR=jochen@chromium.org,thakis@chromium.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=603286, 712790
> > 
> > Review-Url: https://codereview.chromium.org/2843403004
> > Cr-Commit-Position: refs/heads/master@{#467769}
> > Committed: https://chromium.googlesource.com/chromium/src/+/999471957dc5cf51048b0b344c58925a2954d452
> 
> TBR=jochen@chromium.org,thakis@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2873513003
> Cr-Commit-Position: refs/heads/master@{#470369}
> Committed: https://chromium.googlesource.com/chromium/src/+/c7b0cf640eee349516d275dad1a00bd393ac8644

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
Review-Url: https://codereview.chromium.org/2874613002
Cr-Commit-Position: refs/heads/master@{#470444}
Committed: https://chromium.googlesource.com/chromium/src/+/67970ee192706eed7c080811732bab47383d636c

[Dirk Pranke, 2017-05-10 00:35:16]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2017-05-10 00:35:16]

Created Revert of Ensure that builders that use goma strip absolute paths by
default.

[commit-bot: I haz the power, 2017-05-10 00:36:07]

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, 2017-05-10 00:36:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494376516433800, "parent_rev":
"0c48176f8e9bc36ef2ef84e7dc8c96e9e52f183b", "commit_rev":
"67970ee192706eed7c080811732bab47383d636c"}

[commit-bot: I haz the power, 2017-05-10 00:36:55]

Description was changed from

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2873513003/ )

Reason for revert:
Speculatively reverting in case this is causing goma crashes (see
crbug.com/720074)

Original issue's description:
> Reland of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2843403004/ )
> 
> Reason for revert:
> Relanding, since this was not a cause of the goma outage.
> 
> Original issue's description:
> > Revert of Ensure that builders that use goma strip absolute paths by
default. (patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )
> > 
> > Reason for revert:
> > Reverting in case this is related to the goma errors in
https://crbug.com/716089
> > 
> > Original issue's description:
> > > Ensure that builders that use goma strip absolute paths by default.
> > > 
> > > This CL tweaks the MB configs so that any builder that is using goma
> > > also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> > > This helps ensure that we get the best goma cache hit rate possible.
> > > We don't just make the default equal to use_goma=true because doing
> > > so may make it harder for devs to debug binaries, and we want devs
> > > to be able to both debug binaries and use goma by default; the
> > > tradeoff is that their builds are slightly slower.
> > > 
> > > R=thakis@chromium.org, jochen@chromium.org
> > > BUG=603286, 712790
> > > 
> > > Review-Url: https://codereview.chromium.org/2837863005
> > > Cr-Commit-Position: refs/heads/master@{#467574}
> > > Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...
> > 
> > TBR=jochen@chromium.org,thakis@chromium.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=603286, 712790
> > 
> > Review-Url: https://codereview.chromium.org/2843403004
> > Cr-Commit-Position: refs/heads/master@{#467769}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/999471957dc5cf51048b0b344c58...
> 
> TBR=jochen@chromium.org,thakis@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2873513003
> Cr-Commit-Position: refs/heads/master@{#470369}
> Committed:
https://chromium.googlesource.com/chromium/src/+/c7b0cf640eee349516d275dad1a0...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790
==========

to

==========
Revert of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2873513003/ )

Reason for revert:
Speculatively reverting in case this is causing goma crashes (see
crbug.com/720074)

Original issue's description:
> Reland of Ensure that builders that use goma strip absolute paths by default.
(patchset #1 id:1 of https://codereview.chromium.org/2843403004/ )
> 
> Reason for revert:
> Relanding, since this was not a cause of the goma outage.
> 
> Original issue's description:
> > Revert of Ensure that builders that use goma strip absolute paths by
default. (patchset #1 id:1 of https://codereview.chromium.org/2837863005/ )
> > 
> > Reason for revert:
> > Reverting in case this is related to the goma errors in
https://crbug.com/716089
> > 
> > Original issue's description:
> > > Ensure that builders that use goma strip absolute paths by default.
> > > 
> > > This CL tweaks the MB configs so that any builder that is using goma
> > > also sets strip_absolute_paths_from_debug_symbols_by_default=true.
> > > This helps ensure that we get the best goma cache hit rate possible.
> > > We don't just make the default equal to use_goma=true because doing
> > > so may make it harder for devs to debug binaries, and we want devs
> > > to be able to both debug binaries and use goma by default; the
> > > tradeoff is that their builds are slightly slower.
> > > 
> > > R=thakis@chromium.org, jochen@chromium.org
> > > BUG=603286, 712790
> > > 
> > > Review-Url: https://codereview.chromium.org/2837863005
> > > Cr-Commit-Position: refs/heads/master@{#467574}
> > > Committed:
https://chromium.googlesource.com/chromium/src/+/050dbe550d14e99ad4e83093d4f4...
> > 
> > TBR=jochen@chromium.org,thakis@chromium.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=603286, 712790
> > 
> > Review-Url: https://codereview.chromium.org/2843403004
> > Cr-Commit-Position: refs/heads/master@{#467769}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/999471957dc5cf51048b0b344c58...
> 
> TBR=jochen@chromium.org,thakis@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=603286, 712790
> 
> Review-Url: https://codereview.chromium.org/2873513003
> Cr-Commit-Position: refs/heads/master@{#470369}
> Committed:
https://chromium.googlesource.com/chromium/src/+/c7b0cf640eee349516d275dad1a0...

TBR=jochen@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=603286, 712790

Review-Url: https://codereview.chromium.org/2874613002
Cr-Commit-Position: refs/heads/master@{#470444}
Committed:
https://chromium.googlesource.com/chromium/src/+/67970ee192706eed7c080811732b...
==========

[commit-bot: I haz the power, 2017-05-10 00:36:56]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/67970ee192706eed7c080811732b...

[Nico, 2017-05-23 18:37:08]

Can this be relanded?

[Dirk Pranke, 2017-05-23 19:12:04]

On 2017/05/23 18:37:08, Nico wrote:
> Can this be relanded?

Yes, I think so. I'll try to land it later today or tomorrow. It keeps getting
mixed up with other outages ...

[Dirk Pranke, 2017-05-25 19:13:20]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2909523002/ by dpranke@chromium.org.

The reason for reverting is: Reland #3 ....