Recipe roller has spammy bugdroid messages |
||||||||
Issue descriptionIf you make a recipe engine change, you'll end up with a bugdroid message like " The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/bed5ced56d75a4a63d5b4e4241e4e9aa51e2de31 commit bed5ced56d75a4a63d5b4e4241e4e9aa51e2de31 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jun 15 10:46:14 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/ed82e67abdd60ce13f7bdae56d648bd62776f0e3 annotated_run: Use builder directory for temp. (dnj@google.com) https://crrev.com/3ff29095a3ded4744b3dc5c7367956471a9010ba libyuv: Set android_full_debug=1 in GYP_DEFINES for Android x86. (kjellander@chromium.org) https://crrev.com/98122d8810af966ebd7da62629c688c1ab6c8c9a remote_run: use /b as base directory for temporary files when available (phajdan.jr@chromium.org) R=kjellander@chromium.org,dnj@google.com,fbarchard@chromium.org,phajdan.jr@chromium.org BUG= 593999 , libyuv:602 , chromium:620158 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2068043003 [modify] https://crrev.com/bed5ced56d75a4a63d5b4e4241e4e9aa51e2de31/infra/config/recipes.cfg " Which is pretty spammy for a simple change. It'd be nice if we could make this smaller, or maybe not post on every change? I think it would be confusing if the recipe roller didn't post for every roll. For example, if I make a change in bot_update, it doesn't actually take effect until the CL rolls into build, which might take a bit, depending on if the roll is stuck. Thoughts?
,
Jun 16 2016
I've started a document talking about the emails recipe developers receive as part of landing a CL. https://docs.google.com/document/d/1cme6lMGO7mMYK7LxO1SFwTKhwK7OrhfWUntKCq6wj04/edit (feel free to ask for access to it if you don't have it)
,
Jun 29 2016
See doc; moved into two separate bugs.
,
Jun 29 2016
laving this open to track two child bugs.
,
Jun 29 2016
,
Jun 29 2016
,
Sep 27 2016
Ben: can you please assign this to owner of recipe roller? This is really spammy.
,
Sep 27 2016
" The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/bed5ced56d75a4a63d5b4e4241e4e9aa51e2de31 commit bed5ced56d75a4a63d5b4e4241e4e9aa51e2de31 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jun 15 10:46:14 2016 Roll recipes into "infra" (trivial). "build": https://chromium.googlesource.com/chromium/tools/build/+log/98122d8810af966ebd7da62629c688c1ab6c8c9a..ed82e67abdd60ce13f7bdae56d648bd62776f0e3 Confused? More info is at https://goo.gl/zkKdpD. BUG= 593999 ,libyuv:602,chromium:620158 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2068043003 [modify] https://crrev.com/bed5ced56d75a4a63d5b4e4241e4e9aa51e2de31/infra/config/recipes.cfg " This is about the smallest we can make the commit message without losing important information. I'd prefer to keep all the separate changes being rolled in here, so it's easy to look at the roll and decide if something potentially dangerous gets rolled, but I can be persuaded to just have a link to a gitiles log if it's important to save lines in the message.
,
Oct 14 2016
maybe we'ere not talking about the same thing. I don't care what the commit message is. Just when I land a change in infra, I don't want the bug that it referenced to be updated so many times as it's rolled to different repos. Can we make the rolling changes not list bugs so that this doesn't happen? Or have them not update the bug system?
,
Oct 19 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/6b546bafec41a73de183bb99fd206a8c97807f46 commit 6b546bafec41a73de183bb99fd206a8c97807f46 Author: Stephen Martinis <martiniss@chromium.org> Date: Mon Oct 17 22:46:07 2016 Recipe autoroller: Don't post to bug of rolled CL BUG= 620511 Change-Id: I1d07524577e5038b837adfbb416f5267d7020838 Reviewed-on: https://chromium-review.googlesource.com/400019 Reviewed-by: Erik Staab <estaab@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> [modify] https://crrev.com/6b546bafec41a73de183bb99fd206a8c97807f46/recipes/recipe_modules/recipe_autoroller/api.py [modify] https://crrev.com/6b546bafec41a73de183bb99fd206a8c97807f46/recipes/recipes/recipe_autoroller.expected/basic.json [modify] https://crrev.com/6b546bafec41a73de183bb99fd206a8c97807f46/recipes/recipes/recipe_autoroller.expected/failed_upload.json [modify] https://crrev.com/6b546bafec41a73de183bb99fd206a8c97807f46/recipes/recipes/recipe_autoroller.expected/nontrivial.json [modify] https://crrev.com/6b546bafec41a73de183bb99fd206a8c97807f46/recipes/recipes/recipe_autoroller.expected/repo_data_trivial_closed.json [modify] https://crrev.com/6b546bafec41a73de183bb99fd206a8c97807f46/recipes/recipes/recipe_autoroller.expected/with_auth.json
,
Oct 20 2016
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by estaab@chromium.org
, Jun 16 2016