Roll CLs cause too much spam for roll sheriffs |
||
Issue descriptionWe should run first try as a CQ dry run to avoid spamming developers on first error. In the last 4 months, there were only 17 CLs that landed from 3rd try, so if we dry run the first try, we'll only be getting ~4.25 spam emails per month. We could increase the number of dry runs to 2, but then we'll be increasing time from the moment CL fails to the point when sheriff is notified, which we'd like to avoid.
,
Nov 6 2017
,
Nov 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/923fe77de3eaadf496b9120675c5276576d41b6b commit 923fe77de3eaadf496b9120675c5276576d41b6b Author: Sergiy Byelozyorov <sergiyb@chromium.org> Date: Tue Nov 07 14:55:13 2017 V8: Try a dry run before full CQ run on roll CLs R=machenbach@chromium.org Bug: chromium:781855 Change-Id: Id50e92132784d6b122bddc11cd4e79e7fed15c38 Reviewed-on: https://chromium-review.googlesource.com/754935 Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> [modify] https://crrev.com/923fe77de3eaadf496b9120675c5276576d41b6b/scripts/slave/recipes/v8/auto_roll_deps.expected/active_roll.json [modify] https://crrev.com/923fe77de3eaadf496b9120675c5276576d41b6b/scripts/slave/recipes/v8/auto_roll_deps.expected/stale_roll.json [modify] https://crrev.com/923fe77de3eaadf496b9120675c5276576d41b6b/scripts/slave/recipes/v8/auto_roll_deps.py
,
Nov 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/ac48a9bdbddde5ba94ba6f0038f8acc6f29faf12 commit ac48a9bdbddde5ba94ba6f0038f8acc6f29faf12 Author: Sergiy Byelozyorov <sergiyb@chromium.org> Date: Tue Nov 07 15:42:00 2017 [tools] Trigger dry run on upload for auto-roller CLs R=machenbach@chromium.org Bug: chromium:781855 Change-Id: I919bd1340fa075819777ed685dbce88a0008d245 Reviewed-on: https://chromium-review.googlesource.com/756837 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#49197} [modify] https://crrev.com/ac48a9bdbddde5ba94ba6f0038f8acc6f29faf12/tools/release/auto_roll.py [modify] https://crrev.com/ac48a9bdbddde5ba94ba6f0038f8acc6f29faf12/tools/release/git_recipes.py [modify] https://crrev.com/ac48a9bdbddde5ba94ba6f0038f8acc6f29faf12/tools/release/test_scripts.py
,
Nov 7 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by serg...@chromium.org
, Nov 6 2017