Phil Ringnalda (:philor) Reporter
	Description • 8 years ago

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.

	Nick Alexander :nalexander [he/him] (PTO until Montreallhands August 21, 2023)
	Comment 1 • 8 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?

	Phil Ringnalda (:philor) Reporter
	Comment 2 • 8 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.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 3 • 8 years ago

Imma fix this, philor. Just for you!

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 4 • 8 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.

	Pulsebot
	Comment 5 • 8 years ago

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/847a2c97b414
Remove unused files. rs=sebastian

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/847a2c97b414

	Phil Ringnalda (:philor) Reporter
	Comment 7 • 8 years ago

Unhidden.