Closed
Bug 1221647
Opened 9 years ago
Closed 9 years ago
[gaia-taskcluster] Update to use newer version of mozilla-treeherder client >=0.6.0
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: garndt, Assigned: garndt)
References
Details
Attachments
(1 file)
New version of the mozilla-treeherder client has been released that makes use of hawk credentials instead of oauth. gaia-taskcluster should be updated to use these new credentials and client.
Comment 1•9 years ago
|
||
Ideally we'd have this using a different client id (eg "gaia-taskcluster") - if you request these on stage/prod I'll approve them for you:
Stage: https://treeherder.allizom.org/credentials
Prod: https://treeherder.mozilla.org/credentials
Thanks :-)
Comment 2•9 years ago
|
||
I wonder if this highlights that we need a nodejs client (or to get everyone to start using https://github.com/mozilla/treeherder-node), so we don't have to update in so many places?
Assignee | ||
Comment 3•9 years ago
|
||
gaia-taskcluster does use the nodejs client. However, gaia-taskcluster will need to have just a few changes made to be able to use the new client similar to the changes made in mozilla-taskcluster (use one set of creds instead of per project creds).
Comment 4•9 years ago
|
||
Ah I follow now, thank you :-)
Assignee | ||
Comment 5•9 years ago
|
||
No problem, it's a bit confusing with all of these components living all over doing very similar things. There is work to unify some of this. In particular taskcluster-github probably will take over generating graphs for all github projects, making part of gaia-taskcluster obsolete. That way at least for the generation of graphs, there will just be one true thing creating them for github repos.
Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8696969 -
Flags: review?(jopsen)
Updated•9 years ago
|
Assignee: nobody → garndt
Status: NEW → ASSIGNED
Updated•9 years ago
|
Attachment #8696969 -
Flags: review?(jopsen) → review+
Comment 7•9 years ago
|
||
@aus, looping you in here, as I suspect you own gaia-taskcluster, true/false?
(or has some interest in keeping it alive)
Flags: needinfo?(aus)
Assignee | ||
Comment 8•9 years ago
|
||
I definitely think the b2g team is interested in keeping this running, but I'm not sure who the right owner is. This kind of went into limbo once James left.
That being said, the changes have been pushed to heroku (required upgrading the platform type as well) and decision tasks are being created and reported to treeherder. Should be ok I think.
Reopen if there are issues.
Merged: https://github.com/taskcluster/gaia-taskcluster/commit/bb5baa2fc6502493ad8b65edaf9f5c0aac13eb8f
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 9•9 years ago
|
||
Thank you for landing this :-)
Comment 10•9 years ago
|
||
(In reply to Jonas Finnemann Jensen (:jonasfj) from comment #7)
> @aus, looping you in here, as I suspect you own gaia-taskcluster, true/false?
> (or has some interest in keeping it alive)
Yes! We're definitely interested in keeping that alive. I could potentially own this in the meantime although I have zero familiarity with it so it would be easier for me to co-own this with someone with more knowledge of it's inner workings.
Flags: needinfo?(aus)
Assignee | ||
Comment 11•9 years ago
|
||
Let's do it together, teamwork \o/ :)
Updated•6 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•