Closed
Bug 1112303
Opened 10 years ago
Closed 10 years ago
Add taskcluster auth parameters to BuildSlaves.py
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mshal, Assigned: mshal)
References
Details
Attachments
(3 files)
(deleted),
patch
|
rail
:
review+
mshal
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
rail
:
review+
mshal
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
rail
:
review+
mshal
:
checked-in+
|
Details | Diff | Splinter Review |
We need credentials for mozharness to be able to use TC to upload files to s3. I've already created a buildbot user in https://tools.taskcluster.net/auth/ and added 'taskcluster_clientId' and 'taskcluster_accessToken' into hiera. This bug will track adding them into BuildSlaves.py so that mozharness can use them from oauth.txt
Assignee | ||
Updated•10 years ago
|
Comment 1•10 years ago
|
||
Self note (for seamonkey)
Well need to add these secrets to our puppet even if unused to avoid breaking startup
Assignee | ||
Comment 2•10 years ago
|
||
template in buildbot-configs
Attachment #8537451 -
Flags: review?(rail)
Assignee | ||
Comment 3•10 years ago
|
||
Actual puppet changes to pull the credentials from hiera. Please double-check hiera to make sure I didn't hose it up :)
Attachment #8537452 -
Flags: review?(rail)
Updated•10 years ago
|
Attachment #8537452 -
Flags: review?(rail) → review+
Updated•10 years ago
|
Attachment #8537451 -
Flags: review?(rail) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8537451 [details] [diff] [review]
bb-configs-taskcluster-creds
https://hg.mozilla.org/build/buildbot-configs/rev/2bc473e710b9
Attachment #8537451 -
Flags: checked-in+
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8537452 [details] [diff] [review]
puppet-taskcluster-creds
default:
remote: https://hg.mozilla.org/build/puppet/rev/29f074e457d6
production:
remote: https://hg.mozilla.org/build/puppet/rev/875351727cdc
Attachment #8537452 -
Flags: checked-in+
Comment 6•10 years ago
|
||
Assignee | ||
Comment 7•10 years ago
|
||
We need separate credentials for try builds. I've created a new taskcluster auth and put the keys in hiera as try_taskcluster_clientId / try_taskcluster_accessToken
Attachment #8544802 -
Flags: review?(rail)
Updated•10 years ago
|
Attachment #8544802 -
Flags: review?(rail) → review+
Assignee | ||
Updated•10 years ago
|
Attachment #8544802 -
Flags: checked-in+
Assignee | ||
Comment 8•10 years ago
|
||
I think we are set here - the credentials still need to be copied in for try builds, but that's bug 1119500.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•