queue: load-test and fix performance issues

RESOLVED FIXED

Status

Taskcluster
Queue
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jonasfj, Assigned: jonasfj)

Tracking

Details

Attachments

(2 attachments)

(Assignee)

Description

3 years ago
Created attachment 8590640 [details] [review]
Github PR for queue

Done...

PR attached... I want to roll this out tomorrow.
@jhford, if you take a quick sanity check and see if I missing some crazy debug statement or left in a "return" that would be great.

There is an other PR for the changes in TC-base which will boost performance for other components too.
Attachment #8590640 - Flags: review?(jhford)
(Assignee)

Comment 1

3 years ago
Created attachment 8590641 [details] [review]
Github PR for taskcluster-base

Please take a look at the this.. too... I'll fix the reference in taskcluster-queue when I publish new taskcluster-base.
Assignee: nobody → jopsen
Status: NEW → ASSIGNED
Attachment #8590641 - Flags: review?(jhford)
Comment on attachment 8590641 [details] [review]
Github PR for taskcluster-base

Looks good, that sure is a nice improvement in the requests per second.  Does babel-node no longer need -r as of version 5?
Attachment #8590641 - Flags: review?(jhford) → review+
Comment on attachment 8590640 [details] [review]
Github PR for queue

looks good to me
Attachment #8590640 - Flags: review?(jhford) → review+
(Assignee)

Comment 4

3 years ago
@jhford, yeah, babel-node doesn't take -r anymore... Instead we put a .babelrc file in the root and write the configuration in there as long as it is in cwd.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Blocks: 1155450
Component: TaskCluster → Queue
Product: Testing → Taskcluster
You need to log in before you can comment on or make changes to this bug.