1
0
Fork 0
llvm-premerge-checks/docs/development.md
Mikhail Goncharov 5610d8d02f review fixes 2
2020-08-27 15:30:03 +02:00

5.7 KiB

Overview

  • Buildkite orchestrates each build.

  • multiple Linux and windows agents connected to Buildkite. Agents are run at Google Cloud Platform.

  • small proxy service that takes build requests from reviews.llvm.org and converts them into Buildkite build request. Buildkite job sends build results directly to Phabricator.

  • every review creates a new branch in fork of llvm-project.

deployment diagram

Phabricator integration

Buildkite pipelines

Buildkite allows dynamically define pipelines as the output of a command. That gives us the flexibility to generate pipeline code using a script from a specific branch of pre-merge checks.

For example, "pre-merge" pipeline has a single fixed step, that checks out this repo and runs a python script to generate further steps:

export SRC=${BUILDKITE_BUILD_PATH}/llvm-premerge-checks
rm -rf ${SRC}
git clone --depth 1 --branch ${scripts_branch:-master} https://github.com/google/llvm-premerge-checks.git ${SRC}
export SCRIPT_DIR=${SRC}/scripts
${SCRIPT_DIR}/buildkite/build_branch_pipeline.py | tee /dev/tty | buildkite-agent pipeline upload

One typically edits corresponding script, not a pipeline definition in the Buildkite interface. How to test changes.

Life of a pre-merge check

When new diff arrives for review it triggers a Herald rule ("everyone" or "beta testers").

That in sends an HTTP POST request to phab-proxy that submits a new buildkite job diff-checks. All parameters from the original request are put in the build's environment with ph_ prefix (to avoid shadowing any Buildkite environment variable). "scripts_branch" parameter defines which branch of llvm-premerge-checks to use.

diff-checks pipeline (create_branch_pipeline.py) downloads a patch (or series of patches) and applies it to a fork of the llvm-project repository. Then it pushes a new state as a new branch (e.g. "phab-diff-288211") and triggers "premerge-checks" on it (all "ph_" env variables are passed to it). This new branch can now be used to reproduce the build or by another tooling. Periodical cleanup-branches pipeline deletes branches older than 30 days.

premerge-checks pipeline (build_branch_pipeline.py) builds and tests changes on Linux and Windows agents. Then it uploads a combined result to Phabricator.

Enabled projects and project detection

To reduce build times and mask unrelated problems, we're only building and testing the projects that were modified by a patch. choose_projects.py uses manually maintained config file to define inter-project dependencies and exclude projects:

  1. Get prefix (e.g. "llvm", "clang") of all paths modified by a patch.

  2. Add all dependant projects.

  3. Add all projects that this extended list depends on, completing the dependency subtree.

  4. Remove all disabled projects.

Agent machines

All build machines are running from Docker containers so that they can be debugged, updated, and scaled easily:

See playbooks how to manage and set up machines.

Compilation caching

Each build is performed on a clean copy of the git repository. To speed up the builds ccache is used on Linux and sccache on Windows.

Buildkite monitoring

VM instance buildkite-monitoring exposes Buildkite metrics to GCP. To set up a new instance:

  1. Create as small Linux VM with full access to Stackdriver Monitoring API.

  2. Follow instructions to install monitoring agent and enable statsd plugin.

  3. Download recent release of buildkite-agent-metrics.

  4. Run in SSH session:

chmod +x buildkite-agent-metrics-linux-amd64
nohup ./buildkite-agent-metrics-linux-amd64 -token XXXX -interval 30s -backend statsd &

TODO: update "Testing scripts locally" playbook on how to run Linux build locally with Docker.