[llvm] 7f9717b - added section on CI system
Christian Kühnel via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 21 00:59:50 PDT 2021
Author: Christian Kühnel
Date: 2021-04-21T09:59:41+02:00
New Revision: 7f9717b922d421c30f889034488563c67076aca1
URL: https://github.com/llvm/llvm-project/commit/7f9717b922d421c30f889034488563c67076aca1
DIFF: https://github.com/llvm/llvm-project/commit/7f9717b922d421c30f889034488563c67076aca1.diff
LOG: added section on CI system
Add documentation for working with the CI systems.
This is based on the discussion in the Infrastructure Working Group:
https://github.com/ChristianKuehnel/iwg-workspace/issues/37
Differential Revision: https://reviews.llvm.org/D97389
Added:
llvm/docs/buildbot_worker_contact.png
Modified:
llvm/docs/DeveloperPolicy.rst
Removed:
################################################################################
diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index c2171eb588e2..e4d763790bac 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -643,6 +643,51 @@ Here's a `sample RFC
<http://lists.llvm.org/pipermail/llvm-dev/2019-January/129452.html>`_ and the
`corresponding change <https://reviews.llvm.org/D57264>`_.
+.. _ci-usage:
+
+Working with the CI system
+--------------------------
+
+The main continuous integration (CI) tool for the LLVM project is the
+`LLVM Buildbot <https://lab.llvm.org/buildbot/>`_. It uses
diff erent *builders*
+to cover a wide variety of sub-projects and configurations. The builds are
+executed on
diff erent *workers*. Builders and workers are configured and
+provided by community members.
+
+The Buildbot tracks the commits on the main branch and the release branches.
+This means that patches are built and tested after they are merged to the these
+branches (aka. post-merge testing). This also means it's okay to break the build
+occasionally, as it's unreasonable to expect contributors to build and test
+their patch with every possible configuration.
+
+*If your commit broke the build:*
+
+* Fix the build as soon as possible as this might block other contributors or
+ downstream users.
+* If you need more time analyze and fix the bug, please revert your change to
+ unblock others.
+
+*If someone else broke the build and this blocks your work*
+
+* Comment on the code review in `Phabricator <https://reviews.llvm.org/>`_
+ (if available) or email the author, explain the problem and how this impacts
+ you. Add a link to the broken build and the error message so folks can
+ understand the problem.
+* Revert the commit if this blocks your work, see revert_policy_ .
+
+*If a build/worker is permanently broken*
+
+* 1st step: contact the owner of the worker. You can find the name and contact
+ information for the *Admin* of worker on the page of the build in the
+ *Worker* tab:
+
+ .. image:: buildbot_worker_contact.png
+
+* 2nd step: If the owner does not respond or fix the worker, please escalate
+ to Galina Kostanova, the maintainer of the BuildBot master.
+* 3rd step: If Galina could not help you, please escalate to the
+ Infrastructure Working Group `mailto:iwg at llvm.org`.
+
.. _new-llvm-components:
Introducing New Components into LLVM
diff --git a/llvm/docs/buildbot_worker_contact.png b/llvm/docs/buildbot_worker_contact.png
new file mode 100644
index 000000000000..1032b434deef
Binary files /dev/null and b/llvm/docs/buildbot_worker_contact.png
diff er
More information about the llvm-commits
mailing list