Closed
Bug 1152004
Opened 10 years ago
Closed 10 years ago
Authenticate clobberer API requests
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mrrrgn, Assigned: mrrrgn)
Details
Attachments
(1 file)
|
60 bytes,
text/x-github-pull-request
|
Callek
:
review+
mrrrgn
:
checked-in+
|
Details | Review |
An easy way to achieve this will be to create a token and use puppet to distribute it to our slaves.
|
Assignee | |
Comment 1•10 years ago
|
||
Because: http://www.xclusivetouch.co.uk/wp-content/uploads/2014/07/because-we-smart-gif.gif s/we/me/
Attachment #8589657 -
Flags: review?(bugspam.Callek)
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8589657 [details] [review]
https://github.com/mozilla/build-relengapi-clobberer/pull/34
currently failing travis, re-request when thats fixed.
Attachment #8589657 -
Flags: review?(bugspam.Callek) → review-
|
|
Assignee | |
Comment 3•10 years ago
|
||
Oy, yes, I forgot about the tests needing auth now.
|
|
Assignee | |
Updated•10 years ago
|
Assignee: nobody → winter2718
|
|
Assignee | |
Updated•10 years ago
|
Attachment #8589657 -
Flags: review- → review?(bugspam.Callek)
|
|
||
Comment 4•10 years ago
|
||
Comment on attachment 8589657 [details] [review]
https://github.com/mozilla/build-relengapi-clobberer/pull/34
as said in PR, this was r+, but since you did say you were going to do some changes, if they are non-trivial might warrant another pass.
Attachment #8589657 -
Flags: review?(bugspam.Callek) → review+
|
|
Assignee | |
Updated•10 years ago
|
Attachment #8589657 -
Flags: checked-in+
|
|
Assignee | |
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•