Tree Rules/Integration: Difference between revisions

Tree Rules/Integration (view source)

127 bytes added , 28 May 2024

→‎Sheriff Duty: quick follow-up fixes might be accepted to prevent overhead of backout and reland

Confirmed users

571

edits

@@ Line 32: / Line 32: @@
 = Sheriff Duty =
 * Sheriffs will watch these trees (mozilla-inbound and autoland) and back out bustage/regressions as necessary to keep it green.
-** Bustage is backed out right away. There's no "we'll let you fix this tree while everybody stands by."
+** If code sheriffs notice a permanent task failure, they might reach out to the developer to check if the issue is simple mistake which can be fixed with a follow-up patch provided to them through Phabricator in the next 5 minutes.
 ** The backout will be noted in the bug. The onus isn't on the sheriff to contact the developer via IRC regarding the backout. Developers who do not read all bugmail are encouraged to set up appropriate mail filters on bugs to which they are assigned, to ensure they are aware of the backout.
 ** If it's not possible to identify the guilty changeset, the sheriffs may back out more changesets to minimize the overhead/time to fix the tree. Completely innocent pushes may be re-landed by the sheriff once the bustage is cleared or, in case of doubts, a Try server run will be requested in the bug, before the next landing.