Closed Bug 806100 Opened 12 years ago Closed 12 years ago

datazilla ts results have more data than they should

Categories

(Testing :: Talos, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: k0scist, Assigned: jmaher)

References

Details

Attachments

(2 files)

From: talos -n -d -a ts --develop -e `which firefox` --datazilla-url foo.json --cycles 5 I get """ "results": { "ts": [ 494.0, 494.0, 420.0, 494.0, 420.0, 438.0, 494.0, 420.0, 438.0, 422.0, 494.0, 420.0, 438.0, 422.0, 441.0 ] """ This should be just 494.0, 420.0, 438.0, 422.0, 441.0 . We are wrongly extending vs appending somewhere.
Attached patch fix datazilla ts tests (1.0) (deleted) — Splinter Review
fix datazilla ts test output. We might want to extend this to more stuff as well. Also found a bug in terminate process and I needed to fix this in order to test this.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #675858 - Flags: review?(jhammel)
Comment on attachment 675858 [details] [diff] [review] fix datazilla ts tests (1.0) ABICT, this is a misindentation in the original file. I'll post a possible fix
Attachment #675858 - Flags: review?(jhammel) → review-
Attached patch fix indentation level (deleted) — Splinter Review
Attachment #675866 - Flags: review?(jmaher)
Comment on attachment 675866 [details] [diff] [review] fix indentation level Review of attachment 675866 [details] [diff] [review]: ----------------------------------------------------------------- ahh!
Attachment #675866 - Flags: review?(jmaher) → review+
There was an Android ts improvement of about 8% around Oct 18, which seemed to correspond to a talos.zip update -- might this bug account for that?
yeah, there is a good chance.
ABICT, this comes from this changeset: http://hg.mozilla.org/build/talos/rev/e3ab367914e6 . Of course this doesn't tell when a talos.zip was deployed, but Oct 18 seems a bit out.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Depends on: 811361
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: