1585344 - [meta] Get Talos responsiveness tests working with Fission enabled

Status: RESOLVED FIXED

(Testing :: Talos, task, P3)

Description

[Mike Conley (:mconley) (:⚙️)]

The front-end desktop perf team would like to:

1. Check to see if the Talos tests that measure browser responsiveness (as opposed to page load, graphics, or video performance) work with the Fission autostart pref enabled, and fix them if they're not.

2. Ensure that the Talos tests are exercising paths that are realistically going to be hit when Fission is enabled by default.

3. Get a Fission-enabled Talos variant running in CI so we can start collecting measurements and catching regressions.

Comment 1

[Chris Peterson [:cpeterson]]

Enabling Talos is a prerequisite for enabling Fission in Nightly (M6).

Comment 2

[Intermittent Failures Robot]

1 failures in 5444 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 1

Platform breakdown:

• windows7-32-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1585344&startday=2019-11-18&endday=2019-11-24&tree=all

Comment 3

[Chris Peterson [:cpeterson]]

Tentatively moving P1 Fission M6 bugs to M6a.

Comment 4

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Adjusting priority based on the priority of the remaining open dependencies.

Comment 5

[Randell Jesup [:jesup] (needinfo me)]

Dave - where does this stand? Any eta? We're moving this to m6c to not block fission in nightly, but it would be nice to monitor this

Comment 6

[Dave Hunt [:davehunt] [he/him] ⌚BST]

(In reply to Randell Jesup [:jesup] (needinfo me) from comment #5)

Dave - where does this stand? Any eta? We're moving this to m6c to not block fission in nightly, but it would be nice to monitor this

The only open dependency is bug 1454025, which is a DAMP meta bug with additional open dependencies. My team don't maintain the DAMP tests, so I can't comment on the status of those.

Comment 7

[Neha Kochar [:neha]]

Blocked on DAMP (bug 1454025) which is tracked for M7.

Comment 8

[Julian Descottes [:jdescottes]]

Based on the description in the summary, I think it makes more sense to block this bug on Bug 1673308 (fix DAMP with fission) rather than on Bug 1454025 (add more coverage for fission).

We already hit all the relevant codepaths with our current DAMP test suite, but we don't have tests with many iframes + devtools, which is the intent behind Bug 1454025. I think fixing the devtools DAMP test suite to fully work with Fission enabled should be sufficient here.

Comment 9

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Looks like all dependencies of this meta bug are resolved.