Closed
Bug 1321953
Opened 8 years ago
Closed 6 years ago
MozReview should not count failed excluded jobs
Categories
(Conduit Graveyard :: Transplant, defect)
Conduit Graveyard
Transplant
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1261469
People
(Reporter: emk, Unassigned)
Details
STR: 1. Open https://reviewboard.mozilla.org/r/97302/diff/2#index_header AR: MozReview claims > 413 jobs success,19 jobs testfailed But I do not see any failed jobs when I click "See it in Treeherder". If I click "Excluded Jobs" on treeherder, it will show the failed jobs. But excluded jobs are excluded because they are expected to fail. So it should not obstruct landing.
Reporter | ||
Comment 1•8 years ago
|
||
This bug will obstruct landing my patch again (in bug 1322646). I reported three bugs to the MozReview product (this bug, bug 1316227, and bug 1321699). But all of them had no activities at all since reported. I seriously consider to stop using MozReview. Really frustrating.
Reporter | ||
Comment 2•8 years ago
|
||
Just in case, have you ever seen this bug? If not, what I should have done to file MozReview bugs?
Flags: needinfo?(gps)
Comment 3•8 years ago
|
||
You are filing bugs in the correct component. From my casual following of that bug product, usually there is a response to bugs within a day or two. I'm not sure why you are having bad luck :/ I'm not really doing MozReview development any more. I would encourage you to escalate MozReview complaints to mcote. I've needinfo'd him here.
Flags: needinfo?(gps) → needinfo?(mcote)
Comment 4•8 years ago
|
||
Sorry, with the All Hands last week we have been slow to reply to issues, and a lot of us are on PTO this week. The message from Try is informational only. It does not prevent you from autolanding. That said, I agree that the UI should not include, or should clarify, excluded results. We are doing some major work on autoland in Q1 and will be addressing a lot of UI issues. We'll be sure to include this.
Component: General → Autoland
Flags: needinfo?(mcote)
Updated•6 years ago
|
Product: MozReview → Conduit
Comment 5•6 years ago
|
||
This was arguably a treeherder bug, though we'll want to make sure information we expose about test runs in Lando has a good UX. This case is already tracked by Bug 1261469 though so duplicating.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Updated•1 month ago
|
Product: Conduit → Conduit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•