Closed
Bug 1384938
Opened 7 years ago
Closed 7 years ago
funsize groups show "unknown" in tooltip
Categories
(Release Engineering :: Release Automation: Other, enhancement)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: bhearsum, Assigned: jfrench)
Details
Attachments
(1 file)
Eg: the "fs-g-1" group on https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=e5693cea1ec944ca077c7a46c5f127c828a90f1b. Hard to decode unless you already know what funsize is.
Assignee | ||
Comment 1•7 years ago
|
||
For what its worth it appears 'groupName' and 'tier' are missing from the TC Treeherder json in funsize jobs. https://tools.taskcluster.net/groups/Cr8c0VCdSTWcPh6F_ico5A/tasks/eaUaSOyYS8muPZ5A2_0x4Q/details
Assignee | ||
Comment 2•7 years ago
|
||
I've opened a PR in the Funsize repo for this. I assume this bug should be moved over there, but I've noted it here. https://github.com/mozilla-releng/funsize/pull/80
Assignee | ||
Comment 3•7 years ago
|
||
Assignee | ||
Comment 4•7 years ago
|
||
I took a flier on the Product/Component reassignment, from Treeherder to Releng based on other bugs I looked at. If I'm wrong, apologies :)
Component: Treeherder → Release Automation
Product: Tree Management → Release Engineering
QA Contact: catlee
Version: --- → unspecified
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•7 years ago
|
||
Comment on attachment 8893116 [details] [review] PR 80 Adding catlee for review at your leisure. Feel free to reassign if needed :)
Attachment #8893116 -
Flags: review?(catlee)
Comment 6•7 years ago
|
||
Comment on attachment 8893116 [details] [review] PR 80 LGTM, but I'll let the funsize expert weigh in.
Attachment #8893116 -
Flags: review?(catlee) → review?(sfraser)
Comment 7•7 years ago
|
||
Looks good to me. groupName for the signing and update tasks would complete the set.
Assignee | ||
Comment 8•7 years ago
|
||
Thanks Simon let me know if there's anything to be done on my end. :) It appears it can be merged based on those additions.
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•7 years ago
|
||
I've confirmed the task definitions are updating correctly with these new properties. :bearsum this fix should be present and visible in Treeherder after bug 1215587 is fixed and merged, which :camd mentioned should be in a week or two.
Assignee | ||
Comment 10•7 years ago
|
||
Verified fixed on production in central.
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
Attachment #8893116 -
Flags: review?(sfraser) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•