Closed
Bug 1308371
Opened 9 years ago
Closed 9 years ago
Fix lint job so it can be unhidden
Categories
(Firefox for Android Graveyard :: Testing, defect)
Firefox for Android Graveyard
Testing
Tracking
(firefox52 fixed)
RESOLVED
FIXED
Firefox 52
Tracking | Status | |
---|---|---|
firefox52 | --- | fixed |
People
(Reporter: philor, Assigned: kats)
References
Details
Sure wish someone had actually given a shred of thought to how having tier-2 jobs would actually work.
lint is hidden on all trunk trees for https://treeherder.mozilla.org/logviewer.html#?job_id=4608421&repo=autoland / https://queue.taskcluster.net/v1/task/KSRF1-5tSaObHSWq-4GcJw/runs/0/artifacts/public/android/lint/lint-results-automationDebug.html caused by bug 1235513. Needinfo me to unhide it when it's fixed, since that's apparently my sole and personal responsibility too.
Comment 1•9 years ago
|
||
(In reply to Phil Ringnalda (:philor) from comment #0)
> Sure wish someone had actually given a shred of thought to how having tier-2
> jobs would actually work.
>
> lint is hidden on all trunk trees for
> https://treeherder.mozilla.org/logviewer.html#?job_id=4608421&repo=autoland
> /
> https://queue.taskcluster.net/v1/task/KSRF1-5tSaObHSWq-4GcJw/runs/0/
> artifacts/public/android/lint/lint-results-automationDebug.html caused by
> bug 1235513. Needinfo me to unhide it when it's fixed, since that's
> apparently my sole and personal responsibility too.
I'm confused. I made the lint job "work", in that it correctly displays a (somewhat) helpful error:
13:49:43 INFO - TEST-UNEXPECTED-FAIL | android-lint | Lint found errors in the project; aborting build. See the report at: https://queue.taskcluster.net/v1/task/KSRF1-5tSaObHSWq-4GcJw/runs/0/artifacts/public/android/lint/lint-results-automationDebug.html
Tier-2 jobs are explicitly such that they don't require offending patches to get backed out, so this error isn't "criticial".
philor: is your frustration that Tier-2 busting jobs aren't getting bugs filed by anybody other than you? That the bugs aren't getting addressed? Is my (our?) understanding of Tier-2 jobs incorrect?
Flags: needinfo?(philringnalda)
Reporter | ||
Comment 2•9 years ago
|
||
The theory behind the creation of tier-2 jobs was that someone would do some magical something somehow to treeherder so that permaorange would somehow in some unimaginable way become tolerable despite being visible.
There is no magic. A tier-2 job that's permaorange is still in the N/P keyboard cycle, still included in the unstarred count, still causes changes in the unstarred count in the tab title causing a color change saying there's something in a pinned tab which must be dealt with, still has the same visible appearance as a real failure. That makes having it visible during the periods of permaorange which the tree rules absolutely require for it unacceptable, so instead we're going to have this system where it gets ignored for hours (I know some, perhaps all, of the paid sheriffs just hide all tier-2, either part or all of the time), and then I file the bug about it, pissed off as always, and then I hide it.
Flags: needinfo?(philringnalda)
Assignee | ||
Comment 4•9 years ago
|
||
Removing the two xml files it's complaining about fixed the lint job for me locally. sebastian gave r+ on IRC, so I'll just land it.
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/847a2c97b414
Remove unused files. rs=sebastian
Comment 6•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Reporter | ||
Comment 7•9 years ago
|
||
Unhidden.
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•