Skip to content

Conversation

BetsyMcPhail
Copy link
Contributor

@BetsyMcPhail BetsyMcPhail commented Jul 30, 2025

Dear @rpoyner-tri ,

The on-call build cop, @BetsyMcPhail, believes that your PR #22951 may have
broken one or more of Drake's continuous integration builds [1]. It is
possible to break a build even if your PR passed continuous integration
pre-merge because additional platforms are tested post-merge.

The specific build failures under investigation are:
https://drake-jenkins.csail.mit.edu/job/linux-noble-gcc-bazel-experimental-debug/413
https://drake-jenkins.csail.mit.edu/job/linux-noble-gcc-bazel-experimental-debug/416
https://drake-jenkins.csail.mit.edu/job/linux-noble-gcc-bazel-experimental-debug/418

Therefore, the build cop has created this revert PR and started a complete
post-merge build to determine whether your PR was in fact the cause of the
problem. If that build passes, this revert PR will be merged 60 minutes from
now. You can then fix the problem at your leisure, and send a new PR to
reinstate your change.

If you believe your original PR did not actually break the build, please
explain on this thread.

If you believe you can fix the break promptly in lieu of a revert, please
explain on this thread, and send a PR to the build cop for review ASAP.

If you believe your original PR definitely did break the build and should be
reverted, please review and LGTM this PR. This allows the build cop to merge
without waiting for CI results.

For advice on how to handle a build cop revert, see [2].

Thanks!
Your Friendly On-call Build Cop

[1] CI Production Dashboard: https://drake-jenkins.csail.mit.edu/view/Production/
[2] https://drake.mit.edu/buildcop.html#workflow-for-handling-a-build-cop-revert


This change is Reviewable

@BetsyMcPhail
Copy link
Contributor Author

The error is either reporting a full disk at the end of the job (e.g. 413) or a failing job with warnings in the log such as No space left on device

@BetsyMcPhail BetsyMcPhail added status: single reviewer ok https://drake.mit.edu/reviewable.html release notes: none This pull request should not be mentioned in the release notes labels Jul 30, 2025
Copy link
Contributor Author

@BetsyMcPhail BetsyMcPhail left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+@sherm1 for both reviews please

Reviewable status: LGTM missing from assignee sherm1(platform)

Copy link
Member

@sherm1 sherm1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rpoyner-tri do you want to look at this before merging the revert?

Reviewable status: LGTM missing from assignees rpoyner-tri(platform),sherm1(platform)

Copy link
Member

@sherm1 sherm1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: x 2

Reviewable status: LGTM missing from assignee rpoyner-tri(platform)

@sherm1 sherm1 merged commit 218b815 into master Jul 30, 2025
11 checks passed
@jwnimmer-tri
Copy link
Collaborator

Note that all 3 of the failures were observed on a single pull request. So there is the possibility that the bug lies with that pull request, not with the mold switch.

rpoyner-tri added a commit to rpoyner-tri/drake that referenced this pull request Aug 5, 2025
rpoyner-tri added a commit to rpoyner-tri/drake that referenced this pull request Aug 6, 2025
@jwnimmer-tri jwnimmer-tri deleted the revert-22951-mold-integration branch August 7, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: none This pull request should not be mentioned in the release notes status: single reviewer ok https://drake.mit.edu/reviewable.html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants