Closed Bug 1759666 Opened 2 years ago Closed 2 years ago

Add type-specific console.log content process DAMP test

Categories

(DevTools :: Console, task)

task

Tracking

(firefox100 fixed)

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

References

Details

(Whiteboard: dt-console-perf-2022)

Attachments

(1 file)

This will help us track performance of ObjectActor/previewers for specific types

Attachment #9267808 - Attachment description: WIP: Bug 1759666 - [devtools] Add type-specific console.log content process damp test → Bug 1759666 - [devtools] Add type-specific console.log content process damp test. r=ochameau.
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ec2f5253456e
[devtools] Add type-specific console.log content process damp test. r=perftest-reviewers,ochameau,AlexandruIonescu.

Backed out 3 changesets (bug 1759666, bug 1760541, bug 1760540) for causing talos damp failures.

Backout link: https://hg.mozilla.org/integration/autoland/rev/270fdfb511cfab2286f4d52fdfde34bc762b7938

Push with failures

Failure log

 Start "console.log-in-loop-content-process-infinity"
[task 2022-03-22T20:50:29.552Z] 20:50:29     INFO -  PID 19227 | Garbage collect
[taskcluster:error] Aborting task...
[taskcluster:warn 2022-03-22T20:50:31.878Z] no such process
Flags: needinfo?(nchevobbe)

Waiting on Bug 1749928 to land so the new test won't make the suite time out.

Depends on: 1749928
Flags: needinfo?(nchevobbe)
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8f76638b034b
[devtools] Add type-specific console.log content process damp test. r=perftest-reviewers,ochameau,AlexandruIonescu.
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch

It seems like this test makes the damp-webconsole job > 20mn longer.
It used to take 25mn, now it's 45mn or more. damp-inspector and damp-other only take 10-15mn, so we might want to address this?

We can move this test to its own suite, or maybe reduce the number of reloads/reruns if it's not a very noisy test?

Flags: needinfo?(nchevobbe)

Let's move to a followup: Bug 1762922

Flags: needinfo?(nchevobbe)
No longer regressions: 1779847
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: