Closed
Bug 1311158
Opened 8 years ago
Closed 8 years ago
Modify the `date` hook to use Level 2 rather than Level 1 scopes/credentials/decision
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Callek, Assigned: Callek)
Details
Attachments
(2 files)
1.72 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
1.72 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•8 years ago
|
||
As discussed at the taskcluster-nightly meeting today, we should swap date to use L2 stuff rather than L1 stuff. This could have impact for testing on try, and if so we should probably revert and/or figure out a way to keep this testing on try.
Assignee | ||
Comment 2•8 years ago
|
||
Made the diff locally with copy/paste and manual changing on my laptop.
Attachment #8802291 -
Flags: review?(dustin)
Assignee | ||
Comment 3•8 years ago
|
||
Similarly made locally.
Attachment #8802292 -
Flags: review?(dustin)
Comment 4•8 years ago
|
||
Comment on attachment 8802292 [details] [diff] [review] desktop diff I expect you will need to update the scopes for the hook's role as well.
Attachment #8802292 -
Flags: review?(dustin) → review+
Updated•8 years ago
|
Attachment #8802291 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 5•8 years ago
|
||
I forgot to update the cache level numbers, but with fixing that desktop decision task went fine, and I just triggered a new android decision task with the --level2 stuff. Lets see if these tasks all complete fine.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: Hooks → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•