All users were logged out of Bugzilla on October 13th, 2018

make test-perf: boot stats test

RESOLVED WONTFIX

Status

P1
normal
RESOLVED WONTFIX
4 years ago
4 years ago

People

(Reporter: jbailey, Assigned: jon.bailey45+bugmail)

Tracking

({perf})

unspecified
2.1 S3 (29aug)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [c=boot p= s= u=])

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

4 years ago
As per Bug 1047118 and Bug 1045938 : Performance test changes to add a pseudo app-target "boot", which can be invoked to collect boot timing information.
(Reporter)

Comment 1

4 years ago
Created attachment 8480973 [details] [diff] [review]
60001-Bug-1058843-Implement-marionette-test-code-required-.patch

Marionette testing code. Part of its functionality depends on landing Bug 1058810. If the other instrumentation pieces are missing, it would not adversely affect the execution of this script, just the usefulness of its results.

Updated

4 years ago
Whiteboard: [c=boot p= s= u=]
(Reporter)

Comment 2

4 years ago
Created attachment 8481710 [details] [diff] [review]
60001-Bug-1058843-Implement-marionette-test-code-required-.patch
Attachment #8480973 - Attachment is obsolete: true
(Reporter)

Comment 3

4 years ago
Created attachment 8481712 [details] [review]
Github PR
Attachment #8481712 - Flags: review?(hub)
(Reporter)

Comment 4

4 years ago
Since the original patch, test changed to no longer depend (or even use) the lower level instrumentation pieces. Now it is only for collecting data in system app and verticalhome.

Updated

4 years ago
Attachment #8481710 - Attachment is obsolete: true

Updated

4 years ago
Component: Performance → Gaia::PerformanceTest
I have a few question on the PR.

(can't NI the reporter)

Updated

4 years ago
Assignee: jbailey → jon.bailey45+bugmail

Updated

4 years ago
Flags: needinfo?(jon.bailey45+bugmail)
(Assignee)

Comment 6

4 years ago
Changes made according to PR comments, and as well the commits for the instrumentation portions have been included in the PR.
Flags: needinfo?(jon.bailey45+bugmail)
Comment on attachment 8481712 [details] [review]
Github PR

There are a few patches for modules I'm not peer. Please get these in first into gaia.
Attachment #8481712 - Flags: review?(hub) → review-

Comment 8

4 years ago
Triage: wontfix due to upcoming test-perf deprecation. Functionality adapted into Raptor to gather boot metrics.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.