Closed Bug 1149748 Opened 10 years ago Closed 10 years ago

[docker-worker] Update to use new taskcluster-client

Categories

(Taskcluster :: Workers, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: garndt, Assigned: garndt)

Details

Attachments

(1 file)

52 bytes, text/x-github-pull-request
Details | Review
Need to update the worker to use the new taskcluster-client which includes the method to cancel a task that can be used within the cancel test. Also, there is an issue within the task listener where the cancel method doesn't work (fix that also).
Attached file GH PR 76
Updated to use new client and also fix faulty code in cancelling a task. Looks like the only place getTask was used was in the task listener: $ grep -nr --exclude-dir=node_modules "getTask(" ./ .//lib/task_listener.js:239: var task = await this.runtime.queue.getTask(claim.status.taskId);
Attachment #8586349 - Flags: review?(jlal)
Attachment #8586349 - Flags: review?(jlal)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: TaskCluster → Docker-Worker
Product: Testing → Taskcluster
Component: Docker-Worker → Workers
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: