This is an archive of the discontinued LLVM Phabricator instance.

[clang] Move the Clang CI jobs off of the libc++ builders
ClosedPublic

Authored by ldionne on Aug 7 2023, 1:25 PM.

Details

Summary

The libc++ builders do not have the capacity to serve both libc++ and
Clang CI needs. We initially used the libc++ builders when adding
pre-commit CI for Clang as an experiment, however this setup has been
problematic for libc++ for a long time. It makes libc++ CI turnaround
way too slow, which has been a frequent source of frustration.

We didn't want to make this change before the release to avoid rocking
the boat too much in such an important time, however now that the
release has been cut, we need to make this change. This will shift
the traffic for Clang pre-commit CI from the libc++ builders to the
builders that have been used for Clang pre-commit CI defined outside
of Clang's BuildKite pipeline.

Diff Detail

Event Timeline

ldionne created this revision.Aug 7 2023, 1:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 7 2023, 1:25 PM
ldionne requested review of this revision.Aug 7 2023, 1:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 7 2023, 1:25 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

@goncharov Now that we have a buildkite-pipeline.yml for Clang, would it make sense to stop running the "builtin" jobs like https://buildkite.com/llvm-project/premerge-checks/builds/169498#0189d1af-3c81-4ab5-a87a-ac8db177ccf0 and instead run these tests inside the Clang CI pipeline by defining them inside buildkite-pipeline.yml? This would centralize where all the jobs for Clang CI are defined.

ldionne added a subscriber: Restricted Project.Aug 7 2023, 1:30 PM
ldionne updated this revision to Diff 547950.Aug 7 2023, 2:35 PM

Try to fix build on queue=linux builders

ldionne added a reviewer: Restricted Project.Aug 8 2023, 4:44 AM

I'm not qualified to give a review of the functional changes, but the intent for the changes SGTM!

philnik accepted this revision.Aug 8 2023, 9:09 AM
philnik added a subscriber: philnik.

Thanks for working on this! I think it would make a lot of sense to unify this build and the Debian build, since currently we are building Clang twice without a good reason.

This revision is now accepted and ready to land.Aug 8 2023, 9:09 AM
Mordante accepted this revision.Aug 9 2023, 9:40 AM
Mordante added a subscriber: Mordante.

Nice to see the Clang CI new builder! Thanks for working on this, LGTM!

This revision was automatically updated to reflect the committed changes.