[PATCH] D112776: [docs] Update new builder instructions to default to building each change

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 4 09:37:11 PDT 2021


This revision was automatically updated to reflect the committed changes.
Closed by commit rGab4375e99961: [docs] Update new builder instructions to default to building each change (authored by reames).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112776/new/

https://reviews.llvm.org/D112776

Files:
  llvm/docs/HowToAddABuilder.rst


Index: llvm/docs/HowToAddABuilder.rst
===================================================================
--- llvm/docs/HowToAddABuilder.rst
+++ llvm/docs/HowToAddABuilder.rst
@@ -121,6 +121,14 @@
    Please make sure your builder name and its builddir are unique through the
    file.
 
+   All new builders should default to using the "'collapseRequests': False"
+   configuration.  This causes the builder to build each commit individually
+   and not merge build requests.  To maximize quality of feedback to developers,
+   we *strongly prefer* builders to be configured not to collapse requests.
+   This flag should be removed only after all reasonable efforts have been
+   exhausted to improve build times such that the builder can keep up with
+   commit flow.
+
    It is possible to allow email addresses to unconditionally receive
    notifications on build failure; for this you'll need to add an
    ``InformativeMailNotifier`` to ``buildbot/osuosl/master/config/status.py``.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112776.384783.patch
Type: text/x-patch
Size: 991 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211104/a3f2dd40/attachment.bin>


More information about the llvm-commits mailing list