This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Deflake the Clang Modules CI job
ClosedPublic

Authored by ldionne on Aug 7 2023, 11:41 AM.

Details

Reviewers
Bigcheese
iana
ldionne
Group Reviewers
Restricted Project
Commits
rGd2a61db072e9: [libc++] Deflake the Clang Modules CI job
Summary

This re-introduces the workaround that had been introduced in d7ca140c0122
and then removed in 0c0628c92c0d, since it seems like it is needed after all.

Diff Detail

Event Timeline

ldionne created this revision.Aug 7 2023, 11:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 7 2023, 11:41 AM
ldionne requested review of this revision.Aug 7 2023, 11:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 7 2023, 11:41 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
iana accepted this revision.Aug 7 2023, 11:54 AM

params.py looks fine to me. I don't understand the pipeline changes, but if those are on purpose then I trust they're necessary.

params.py looks fine to me. I don't understand the pipeline changes, but if those are on purpose then I trust they're necessary.

I'll revert the pipeline changes, the goal is only to avoid running the full pipeline on this change several times.

ldionne accepted this revision.Aug 8 2023, 5:42 AM

I'm going to land this (without the pipeline changes) and monitor the libc++ CI pipeline to see whether that helps with flaky modular builds.

This revision is now accepted and ready to land.Aug 8 2023, 5:42 AM
This revision was automatically updated to reflect the committed changes.