Closed
Bug 1453478
Opened 6 years ago
Closed 2 years ago
Enable test-verify jobs for QR
Categories
(Core :: Graphics: WebRender, enhancement, P3)
Tracking
()
RESOLVED
WONTFIX
mozilla61
| Tracking | Status | |
|---|---|---|
| firefox61 | --- | fixed |
People
(Reporter: kats, Unassigned)
References
(Depends on 1 open bug)
Details
(Whiteboard: [gfx-noted])
Attachments
(1 file)
I keep forgetting to do this, but we should add the test-verify tasks to QR platforms so that doing a verify push will verify new/modified tests on QR as well.
| Comment hidden (mozreview-request) |
|
Reporter | |
Comment 2•6 years ago
|
||
(For the record, it seems to me that the verify-wpt job should be part of the wpt test group, not sure why it's in a separate group)
|
||
Updated•6 years ago
|
Blocks: stage-wr-trains
Priority: -- → P2
|
||
Comment 3•6 years ago
|
||
| mozreview-review | ||
Comment on attachment 8967152 [details] Bug 1453478 - Add test-verify jobs for QR platforms. https://reviewboard.mozilla.org/r/235802/#review241726 thanks!
Attachment #8967152 -
Flags: review?(jmaher) → review+
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/93c851a45ec7 Add test-verify jobs for QR platforms. r=jmaher
|
||
Comment 5•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/93c851a45ec7
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
|
|
Reporter | |
Comment 6•6 years ago
|
||
This is causing issues because the test-verify task will run tests in suites that we don't have enabled on the platform. This is causing many failures that are wasting people's time (see bug 1454531 and dupes). So I'm going to back this out. Bug 1455309 is tracking fixing the test-verify job.
|
|
Reporter | |
Comment 7•6 years ago
|
||
Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/1f7eb198c7e1590d1a4801906f93c0014a094e72
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
|
Reporter | |
Comment 8•6 years ago
|
||
Backout merged to m-c: https://hg.mozilla.org/mozilla-central/rev/1f7eb198c7e1
|
|
Reporter | |
Comment 9•6 years ago
|
||
This is waiting on dependencies, unassigning for now.
Assignee: bugmail → nobody
|
||
Updated•6 years ago
|
Priority: P2 → P3
|
||
Comment 10•6 years ago
|
||
Kats, how important is this? What priority do you think this should be?
Flags: needinfo?(kats)
|
|
Reporter | |
Comment 11•6 years ago
|
||
Not important, it's mostly a developer/sheriff convenience and doesn't affect end users.
Flags: needinfo?(kats)
|
|
||
Updated•6 years ago
|
|
||
Comment 12•2 years ago
|
||
Seems like this isn't necessary.
Status: REOPENED → RESOLVED
Closed: 6 years ago → 2 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•