1861042 - 'duration' can be `null` in errorsummary file

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

This task has a crashtest errorsummary file without a line which mentions the duration. It should be provided, Treeherder assumes it is a required property.

I will adjust Treeherder to set duration to 0 (like done for other fallbacks) in a different bug.

Comment 1

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Correction: The task is this one and it contains a duration of null.

{"group": "", "status": "ERROR", "duration": null, "action": "group_result", "line": 13403}

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

I see 2 things:

1. treeherder fix for duration=null to be treated as 0
2. fix process-crash output to have the group name (especially since we have the testname.

I am not sure of the root cause, the group="" is a problem, and that indicates that we are logging stuff outside of the normal channels (i.e. not using mozlog.crash()) and need to fix that.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

I think I found the reason for the missing data in errorsummary. so #2 should have a patch for review before too long.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: Bug 1861042 - sanitize test name to match group mapping. r=marco!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0bf5a15f1e03
sanitize test name to match group mapping. r=marco

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/0bf5a15f1e03

Comment 7

[BugBot [:suhaib / :marco]]

The patch landed in nightly and beta is affected.
:jmaher, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox120 to wontfix.

For more information, please visit BugBot documentation.