Closed Bug 1579168 Opened 3 months ago Closed 3 months ago

Metadata update sometimes still overwrites intermittents

Categories

(Testing :: web-platform-tests, defect)

Version 3
defect
Not set

Tracking

(firefox71 fixed)

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: jgraham, Assigned: jgraham)

Details

Attachments

(1 file)

Looks like we aren't correctly passing the known intermittent information down to the update code

We were assuming that in the case that the status matched expected we'd have
no expected entry in the log data. But actually it was being set to None, so
handle both those cases.

Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/autoland/rev/4fe3084ae3e4
Fix updating metadata for results that are expected but have known intermittents, r=maja_zf
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Flags: needinfo?(wptsync)
Failed to create upstream wpt PR due to merge conflicts. This requires fixup from a wpt sync admin.
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/19318 for changes under testing/web-platform/tests
Can't merge web-platform-tests PR due to failing upstream checks:
Github PR https://github.com/web-platform-tests/wpt/pull/19318
* Taskcluster (pull_request) (https://tools.taskcluster.net/task-group-inspector/#/ShKKU5uQRW2j5DlIO67koQ)
Flags: needinfo?(wptsync) → needinfo?(james)
Can't merge web-platform-tests PR due to failing upstream checks:
Github PR https://github.com/web-platform-tests/wpt/pull/19318
* Taskcluster (pull_request) (https://tools.taskcluster.net/task-group-inspector/#/Ye9ixbF1TFimi2czAMAhlw)
Flags: needinfo?(james)
Upstream PR merged by moz-wptsync-bot
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ad77820040a4
[wpt PR 19318] - [Gecko Bug 1579168] Fix updating metadata for results that are expected but have known intermittents, a=testonly
You need to log in before you can comment on or make changes to this bug.