Closed
Bug 1189897
Opened 10 years ago
Closed 10 years ago
Android Talos not producing talosdata blob, so no data in perfherder
Categories
(Testing :: Talos, defect)
Testing
Talos
Tracking
(firefox42 fixed)
RESOLVED
FIXED
mozilla42
| Tracking | Status | |
|---|---|---|
| firefox42 | --- | fixed |
People
(Reporter: wlach, Assigned: wlach)
Details
Attachments
(1 file)
|
2.90 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
Unclear why, but it looks like Android talos isn't producing the TALOSDATA blob, so data isn't being ingested by treeherder/perfherder.
You can see the abrupt stop in new data here:
https://treeherder.mozilla.org/perf.html#/graphs?timerange=2592000&series=[mozilla-central,c233ba1133abbd544002dfbc29d9e63ced42a20e,1]
Just look at any recent talos log to notice lack of TALOSDATA.
Android talos uses a different mozharness path so bug 1187082 doesn't help us here. This harness is going away soon but we should probably keep perfherder submission going as long as it's easy.
android talos was updated May 22, odd that it stopped on july 22 unless this is a mozharness change. I suspect we need to add some cli params for android only there.
| Assignee | ||
Comment 2•10 years ago
|
||
I think the mozharness changes in bug 1172918 made the android harness stop outputting the TALOSDATA blob.
| Assignee | ||
Comment 3•10 years ago
|
||
I think this should fix things. Putting patch through try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=5813a9ee9b6e
Assignee: nobody → wlachance
Attachment #8641899 -
Flags: review?(jmaher)
Comment on attachment 8641899 [details] [diff] [review]
Revert android changes from bug 1172918
Review of attachment 8641899 [details] [diff] [review]:
-----------------------------------------------------------------
this looks right- motivation to port to autophone!
Attachment #8641899 -
Flags: review?(jmaher) → review+
| Assignee | ||
Comment 5•10 years ago
|
||
(In reply to William Lachance (:wlach) from comment #3)
> Created attachment 8641899 [details] [diff] [review]
> I think this should fix things. Putting patch through try:
>
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=5813a9ee9b6e
Hmm, that seems not to have run the android talos tests. :jmaher, do you know why?
Here was the commit message: `try: -b o -p android-api-9,android-api-11 -u none -t all `
I experimented with the trigger all talos jobs option, maybe it'll work...
Flags: needinfo?(jmaher)
not sure why all the other platforms are built. For the record we don't do all the android stuff in the all talos scripts, but triggerbyfilter.py from mozci would do it :)
As for why the original commit message failed, I have no idea :(
Flags: needinfo?(jmaher)
| Assignee | ||
Comment 7•10 years ago
|
||
Ok, after manually retriggering using mozci, I got one talos job that succeeded and produced TALOSDATA:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=5813a9ee9b6e
I think this can go in.
Keywords: checkin-needed
Keywords: checkin-needed
we should uplift this to aurora also.
Comment 10•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in
before you can comment on or make changes to this bug.
Description
•