Closed Bug 1607620 Opened 6 years ago Closed 6 years ago

Permafailing REFTEST TEST-UNEXPECTED-FAIL | layout/reftests/mathml/mfenced-12.html == layout/reftests/mathml/mfenced-12-ref.html | image comparison, max difference: 255, number of differing pixels: 164

Categories

(Core :: MathML, defect, P5)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: intermittent-bug-filer, Unassigned)

References

(Regression)

Details

(Keywords: intermittent-failure, regression, Whiteboard: [stockwell disable-recommended])

Filed by: ncsoregi [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=283915091&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Niig-TILRReigpF9TvZIwQ/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Niig-TILRReigpF9TvZIwQ/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1


[task 2020-01-07T23:26:18.183Z] 23:26:18 INFO - REFTEST TEST-START | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12.html == file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12-ref.html
[task 2020-01-07T23:26:18.184Z] 23:26:18 INFO - REFTEST INFO | RESTORE PREFERENCE pref(mathml.mfenced_element.disabled,true)
[task 2020-01-07T23:26:18.185Z] 23:26:18 INFO - REFTEST INFO | SET PREFERENCE pref(mathml.mfenced_element.disabled,false)
[task 2020-01-07T23:26:18.186Z] 23:26:18 INFO - REFTEST TEST-LOAD | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12.html | 51 / 384 (13%)
[task 2020-01-07T23:26:18.527Z] 23:26:18 INFO - REFTEST INFO | RESTORE PREFERENCE pref(mathml.mfenced_element.disabled,true)
[task 2020-01-07T23:26:18.528Z] 23:26:18 INFO - REFTEST INFO | SET PREFERENCE pref(mathml.mfenced_element.disabled,false)
[task 2020-01-07T23:26:18.529Z] 23:26:18 INFO - REFTEST TEST-LOAD | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12-ref.html | 51 / 384 (13%)
[task 2020-01-07T23:26:18.926Z] 23:26:18 INFO - REFTEST TEST-UNEXPECTED-PASS | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12.html == file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12-ref.html | image comparison, max difference: 0, number of differing pixels: 0

Regressed by: 1607496
Flags: needinfo?(dmajor)

mathml/mfenced-12.html is already unexpected pass on the push prior to mine, but the test was in R5 https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=39c4e3bab99a7dc933cf97593157204b4a280acd&selectedJob=283957486

Edwin, I see you have a patch in bug 1599654 that can resolve this. Perhaps you know what's going on here?

Flags: needinfo?(aethanyc) → needinfo?(egao)

The needinfo should go to egao

Flags: needinfo?(dmajor)

Acknowledging ni.

This is expected outcome from the migration from ubuntu1604 to ubuntu1804.

TYLin, I have a planned migration as you noted - I do plan on landing this today, which would resolve this unexpected pass.

Flags: needinfo?(egao)

The patch to migrate reftests to run on ubuntu1804, and to update the expected outcome of the mfenced-12.html test result has landed on autoland:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=9b598ca58e8d10c73c72956ce2e1b5a1bc7945b0

Summary: Permafailing tier 2 TEST-UNEXPECTED-PASS | mathml/mfenced-12.html == mathml/mfenced-12-ref.html | image comparison, max difference: 0, number of differing pixels: 0 → Permafailing TEST-UNEXPECTED-PASS | mathml/mfenced-12.html == mathml/mfenced-12-ref.html | image comparison, max difference: 0, number of differing pixels: 0

My patch will address the issues with the normal reftest. I'm trying to green up the reftest-no-accel variant at the moment.

The issue is not because of new failure, rather it has to do with my migration work to run these tests under ubuntu1804 where this test does not fail. Reftest manifests don't allow enough flexibility to mark individual platforms sadly.

I'm simply going to disable this platform linux1804-64-asan-qr for the time being as it was not meant to be enabled at the moment.

We will not lose coverage since the linux64-asan-qr variant still runs and it is identical except for that it uses the legacy ubuntu1604 image.

Hi Edwin, are you referring to the patch in bug 1428705 ? if so, that reached central and not this is failing as

REFTEST TEST-UNEXPECTED-FAIL | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12.html == file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12-ref.html | image comparison, max difference: 255, number of differing pixels: 164

Can you please take a look?

Treeherder link: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&resultStatus=testfailed%2Cbusted%2Cexception&fromchange=c0a6eb95b65cce34497c3f6aca502934ed0191cc&selectedJob=284139166

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=284139166&repo=mozilla-central&lineNumber=10006

Flags: needinfo?(egao)
Summary: Permafailing TEST-UNEXPECTED-PASS | mathml/mfenced-12.html == mathml/mfenced-12-ref.html | image comparison, max difference: 0, number of differing pixels: 0 → Permafailing REFTEST TEST-UNEXPECTED-FAIL | layout/reftests/mathml/mfenced-12.html == layout/reftests/mathml/mfenced-12-ref.html | image comparison, max difference: 255, number of differing pixels: 164

(In reply to Andreea Pavel [:apavel] from comment #12)

Hi Edwin, are you referring to the patch in bug 1428705 ? if so, that reached central and not this is failing as

REFTEST TEST-UNEXPECTED-FAIL | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12.html == file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/mathml/mfenced-12-ref.html | image comparison, max difference: 255, number of differing pixels: 164

Can you please take a look?

Treeherder link: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&resultStatus=testfailed%2Cbusted%2Cexception&fromchange=c0a6eb95b65cce34497c3f6aca502934ed0191cc&selectedJob=284139166

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=284139166&repo=mozilla-central&lineNumber=10006

:apavel - that is not the patch I was referring to. This is the patch that should silence failures on test-linux1804-asan-qr and test-linux1804-ccov: https://phabricator.services.mozilla.com/D59214

Flags: needinfo?(egao)

Looks like that patch landed and the failures have been silenced. I will close this bug for the time being.

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED

I noticed that this is the bug that reports unexpected failure, I think I mixed it up with a very similar bug 1607916 - my mistake.

I'll wait for the patch to land successfully on mozilla-central first.

Has Regression Range: --- → yes
Keywords: regression

Seems to have stopped. Last failure was ~22 hours ago.

Yes, the patch to disable the test made its way to mozilla-central. Now I need to work carefully to re-enable the test is appropriate.

Closing as the work to re-enable this test is being handled elsewhere.

Status: REOPENED → RESOLVED
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.