reduce the android test pages which are redundant

RESOLVED FIXED

Status

Testing
Talos
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jmaher, Assigned: jmaher)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox40 affected)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
removing a lot of tp4m pages, 3 tsvgx pages (calling it tsvgm now) and reducing the cycles of tsvgm from 10 (which is defined in the talos.json) to 7.
(Assignee)

Comment 1

3 years ago
Created attachment 8601702 [details] [diff] [review]
reduce android pages (1.0)
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8601702 - Flags: review?(gbrown)
Comment on attachment 8601702 [details] [diff] [review]
reduce android pages (1.0)

Review of attachment 8601702 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks Joel. How much time does this save? How did you determine redundancy?
Attachment #8601702 - Flags: review?(gbrown) → review+
(Assignee)

Comment 4

3 years ago
previously we had around 4.5 minutes to run the test, with the reduced pages we have around 3.5 minutes.

I need to figure out the svg stuff, that will be smaller as we just trimmed a couple of pages which are fast to load.

While this isn't much overall savings, it does make a difference when we scale out.  Sadly our tp4m jobs take ~60 minutes so the <5 minutes of testing doesn't account for much.
(Assignee)

Comment 5

3 years ago
gbrown, do you have additional thoughts?
Flags: needinfo?(gbrown)
As long as we are not losing any testing value, a 1 minute time reduction is a good thing -- let's go ahead with this, then see what can be done about the other 55 minutes.
Flags: needinfo?(gbrown)
(Assignee)

Updated

3 years ago
Blocks: 1162753
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.