Closed
Bug 1188398
Opened 9 years ago
Closed 8 years ago
Taskcluster should use the nodejs treeherder client to access Treeherder's API
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: emorley, Unassigned)
References
Details
One of the ways we could avoid problems like bug 1187395, is by having taskcluster use the nodejs client everywhere and having the Treeherder team taking ownership of it. Bug 1188396 is filed for the latter, this bug is to get taskcluster to use the client in all places, rather than directly accessing the API. This bug may need to wait for bug 1188396 to be finished first, though I imagine there may need to be some back and forth between that bug and this one, if there are missing features that we need to add to the nodejs client, in order for mozilla-taskcluster to make the switch.
Comment 1•9 years ago
|
||
mozilla-taskcluster does use the node treeherder client... https://github.com/taskcluster/mozilla-taskcluster/blob/master/package.json#L40
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Greg Arndt [:garndt] from comment #1) > mozilla-taskcluster does use the node treeherder client... Is there anywhere in TC that does not? :-)
Comment 3•9 years ago
|
||
So these are the projects I know about so far that might post something to treeherder, all of them really are not part of taskcluster itself but components that integrate with taskcluster and other systems... Autolander (uses the client) https://github.com/mozilla/autolander/blob/master/package.json#L38 gaia-taskcluster (uses the client) https://github.com/lightsofapollo/gaia-taskcluster/blob/master/package.json#L55 mozilla-taskcluster (users the client) https://github.com/taskcluster/mozilla-taskcluster/blob/master/package.json#L40 Jonas, John, Pete....do you know of any other systems using nodejs and might be posting to treeherder that integrated with taskcluster? We want to ensure they are all using the nodejs client.
Flags: needinfo?(pmoore)
Flags: needinfo?(jopsen)
Flags: needinfo?(jhford)
Comment 4•9 years ago
|
||
i dont know of others. can the client set a custom user agent string and then monitor the api enpoints for requests that don't have that user agent string?
Flags: needinfo?(jhford)
Updated•9 years ago
|
Flags: needinfo?(jopsen)
Reporter | ||
Comment 6•9 years ago
|
||
(In reply to John Ford [:jhford] -- please use 'needinfo?' instead of a CC from comment #4) > i dont know of others. can the client set a custom user agent string and > then monitor the api enpoints for requests that don't have that user agent > string? Yeah that would be helpful; bug 1186918 is filed for this.
Depends on: 1186918
Updated•8 years ago
|
Component: Integration → Platform and Services
Reporter | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•5 years ago
|
Component: Platform and Services → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•