Closed
Bug 1168360
Opened 9 years ago
Closed 9 years ago
add symbols for osx e10s jobs to treeherder
Categories
(Tree Management :: Treeherder: Data Ingestion, defect)
Tree Management
Treeherder: Data Ingestion
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Assigned: jmaher)
References
Details
Attachments
(2 files)
we have talos jobs definitions for many things, this weekend we turned on talos e10s for osx 10.10, we have these jobs which would live in the T-e10s() group:
g1 = g1-osx-e10s
g2 = g2-osx-e10s
c = chromez-osx-e10s
s = svgr-osx-e10s
we also need to fix the talos bits to pull e10s jobs out of the T() group and put them in a T-e10s() group:
o = other-osx-e10s
tp = tp5o-osx-e10s
Comment 1•9 years ago
|
||
Please could we use generic names so we don't need to continually update these as new OSes are added? The OS string in the job name already makes it unique.
Component: Treeherder → Treeherder: Data Ingestion
Assignee | ||
Comment 2•9 years ago
|
||
these are different builders and run frequencies- osx doesn't have pgo, so we end up with a need for nonpgo, but then we have unique stuff that only runs on certain platforms and other tests which perma fail on some platforms.
I am open to suggestions, but as it stands we sort of need granularity which we are just hacking around as we have a need for it.
Updated•9 years ago
|
Component: Treeherder: Data Ingestion → General
Product: Tree Management → Firefox
Version: --- → unspecified
Updated•9 years ago
|
tracking-e10s:
--- → +
Updated•9 years ago
|
Component: General → Treeherder: Data Ingestion
Product: Firefox → Tree Management
Version: unspecified → ---
Assignee | ||
Comment 3•9 years ago
|
||
whilst this isn't ideal, it does some cleanup of older stuff and solves our immediate problem.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
tracking-e10s:
+ → ---
Attachment #8637814 -
Flags: review?(emorley)
Comment 4•9 years ago
|
||
Comment on attachment 8637814 [details]
https://github.com/mozilla/treeherder/pull/802
There's a failing test; have added a comment to the PR :-)
Attachment #8637814 -
Flags: review?(emorley)
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 8637814 [details]
https://github.com/mozilla/treeherder/pull/802
ok, verified all tests pass, it is looking good now!
Attachment #8637814 -
Flags: review?(emorley)
Comment 6•9 years ago
|
||
Comment on attachment 8637814 [details]
https://github.com/mozilla/treeherder/pull/802
Looks good, thank you :-)
(I'm presuming none of the old jobs are running on old release repos; had a quick glance couldn't see any)
Attachment #8637814 -
Flags: review?(emorley) → review+
Assignee | ||
Comment 7•9 years ago
|
||
talos only runs on mozilla-beta, mozilla-aurora, mozilla-central, integration/*, and project branches.
no b2g*, mozilla-release, or ESR.
Updated•9 years ago
|
tracking-e10s:
--- → +
Comment 8•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/f6d14287f91c78be7ab18c3e4f80f7eace528834
Bug 1168360 - add symbols for osx e10s jobs to treeherder, remove old test definitions
https://github.com/mozilla/treeherder/commit/0be9fc2a6b31f49f5dad7d6dc19dbb8e5378f85c
Merge pull request #802 from jmaher/osxe10s
Bug 1168360 - add symbols for osx e10s jobs to treeherder, remove old…
tracking-e10s:
+ → ---
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•9 years ago
|
||
Attachment #8638542 -
Flags: review?(wlachance)
Assignee | ||
Updated•9 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 11•9 years ago
|
||
Comment on attachment 8638542 [details]
fix up osx tp on treeherder ui
thanks!
Attachment #8638542 -
Flags: review?(wlachance) → review+
Comment 12•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/83fb41b132af1f0a5811c50c188ea2120051baf9
Bug 1168360 - fix osx talos tp/tp-e10s
https://github.com/mozilla/treeherder/commit/6bf44c89e1674e9272d25913ab4dbe13d581100d
Merge pull request #808 from jmaher/osxe10s
Bug 1168360 - fix osx talos tp/tp-e10s
Assignee | ||
Updated•9 years ago
|
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•