1286652 - remove scopes for caches in try, too

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Dustin J. Mitchell [:dustin] (he/him)]

We currently remove caches in try, but do not remove the corresponding scopes.

It turns out, there are several jobs for which the scopes are incorrect, too -- apparently never run outside of try!

Comment 1

[Dustin J. Mitchell [:dustin] (he/him)]

Attachment: Bug 1286652: also remove cache scopes on try and fix scopes;

Review commit: https://reviewboard.mozilla.org/r/64082/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/64082/

Comment 2

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 8770698 [details]
Bug 1286652: also remove cache scopes on try and fix scopes;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/64082/diff/1-2/

Comment 3

[Justin Wood (:Callek)]

Comment on attachment 8770698 [details]
Bug 1286652: also remove cache scopes on try and fix scopes;

https://reviewboard.mozilla.org/r/64082/#review61092

For some reason I thought try had a workspace cache usable (but not shared with m-c) -- code makes me think I was wrong. Either way r+ for this!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/484a3b7cb17b
also remove cache scopes on try and fix scopes; r=Callek

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/484a3b7cb17b