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)

defect
Not set
normal

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.
mozilla-taskcluster does use the node treeherder client...

https://github.com/taskcluster/mozilla-taskcluster/blob/master/package.json#L40
(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? :-)
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)
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)
I don't know any either.
Flags: needinfo?(pmoore)
Flags: needinfo?(jopsen)
(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
Component: Integration → Platform and Services
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Component: Platform and Services → Services
You need to log in before you can comment on or make changes to this bug.