Closed
Bug 1374213
Opened 7 years ago
Closed 7 years ago
ESR52 tasks currently require taskcluster-worker OS X workers, and we don't have any
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pmoore, Assigned: wcosta)
References
Details
Attachments
(1 file)
880 bytes,
patch
|
pmoore
:
review+
|
Details | Diff | Splinter Review |
Context: http://logs.glob.uno/?c=taskcluster#c200665 07:56 KWierso|afk taskcluster jobs on esr52 seem to be unhappy: treeherder.mozilla.org/#/jobs?repo…=10.%20tc&group_state=expanded 07:56 KWierso|afk failing like queue.taskcluster.net/v1/task/N7N0…facts/public/logs/live_backing.log 07:56 KWierso|afk do we need to uplift something to esr52 or un-move things from taskcluster on that tree? 07:56 KWierso|afk back to buildbot/bbb? 08:11 * pmoore looks 08:16 pmoore looks like e.g. task tools.taskcluster.net/task-inspector/#N7N0H-xaRX-1zHvwqRS4Ig never got claimed - let me check the provisionerId / workerType is correct .... 08:20 pmoore wcosta: do you know what tc-worker-provisioner is? the only reference i can find is in bugzilla.mozilla.org/show_bug.cgi?id=1274980#c107 08:20 firebot Bug 1274980 — FIXED, wcosta%mozilla.com — Configure test tasks for Mac OSX builds 08:24 pmoore looks like it is coming from mozilla-esr52/annotate/796616713ef…ests/make_task_description.py#l393 08:25 pmoore KWierso|afk: i suspect this needs renaming - indeed it might be an uplift, i think the provisioner was renamed on other trees 08:26 pmoore i'll see if i can find what needs uplifting 08:26 KWierso|afk pmoore: okay. 1:30am is my cutoff time for looking at things. will follow up in the morning :) 08:27 pmoore KWierso|afk: sure no worries, i'll work with Tomcat|sheriffduty on this one. have a good night, and thanks for the heads up! 08:32 pmoore hmmmm, this looks like a taskcluster-worker task, our pool is entirely generic-worker these days :/ 08:33 pmoore i guess the options are keeping a very small taskcluster-worker pool for these ESR jobs, or patching ESR to use generic-worker 08:33 pmoore i'll create a bug
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(wcosta)
Reporter | ||
Comment 1•7 years ago
|
||
Full link from chat above is: https://hg.mozilla.org/releases/mozilla-esr52/annotate/796616713ef2887f42c361329602bbd04cbbf773/taskcluster/taskgraph/transforms/tests/make_task_description.py#l393
Comment 2•7 years ago
|
||
No jobs were migrated on the esr52 branch and these failures were present before the migration that started on June 13th. It looks like esr52 maybe was created at a time where things were in flux and never received our newest automation scheduling code after that. Those jobs indeed use a worker type that do not exist and the jobs will fail with deadline-exceeded. Perhaps a patch specific to esr52 coudl be created that just disabled these from even trying to run?
Assignee | ||
Comment 3•7 years ago
|
||
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8879130 [details] [diff] [review] esr52 tests should never be scheduled by tc. r=pmoore Surprisingly how this bug was hidden all this time, since we shutdown taskcluster-worker machines a long time ago ¯\_(ツ)_/¯
Flags: needinfo?(wcosta)
Attachment #8879130 -
Flags: review?(pmoore)
Assignee | ||
Comment 5•7 years ago
|
||
Uplifting this to esr52 should fix the issue.
Reporter | ||
Comment 6•7 years ago
|
||
Comment on attachment 8879130 [details] [diff] [review] esr52 tests should never be scheduled by tc. r=pmoore Review of attachment 8879130 [details] [diff] [review]: ----------------------------------------------------------------- lgtm, thanks Wander!
Attachment #8879130 -
Flags: review?(pmoore) → review+
This should be able to be uplifted a=test-only.
Whiteboard: checkin-needed-esr52
Updated•7 years ago
|
Assignee: nobody → wcosta
Comment 8•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr52/rev/d1f81cbeaae3
Whiteboard: checkin-needed-esr52
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•