Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crater shows many crates as "regressed" but there is no error #733

Open
RalfJung opened this issue Aug 24, 2024 · 5 comments
Open

Crater shows many crates as "regressed" but there is no error #733

RalfJung opened this issue Aug 24, 2024 · 5 comments

Comments

@RalfJung
Copy link
Member

@Skgland
Copy link
Contributor

Skgland commented Aug 24, 2024

3 out of the four listed examples appear to have an error in crater/rustwide itself

  • The second example has this error:

[INFO] running Command { std: "docker" "start" "-a" "4587da511f2e05fc5af9052e2a4c27f9b9c9a90262f127468c228df05bbf3564", kill_on_drop: false }
[INFO] [stderr] time="2024-08-22T06:05:15Z" level=error msg="error waiting for container: "
[INFO] [stderr] Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: unable to apply cgroup configuration: unable to start unit "docker-4587da511f2e05fc5af9052e2a4c27f9b9c9a90262f127468c228df05bbf3564.scope" (properties [{Name:Description Value:"libcontainer container 4587da511f2e05fc5af9052e2a4c27f9b9c9a90262f127468c228df05bbf3564"} {Name:Slice Value:"system.slice"} {Name:Delegate Value:true} {Name:PIDs Value:@au [763051]} {Name:MemoryAccounting Value:true} {Name:CPUAccounting Value:true} {Name:IOAccounting Value:true} {Name:TasksAccounting Value:true} {Name:DefaultDependencies Value:false}]): Message recipient disconnected from message bus without replying: unknown
[INFO] running Command { std: "docker" "inspect" "4587da511f2e05fc5af9052e2a4c27f9b9c9a90262f127468c228df05bbf3564", kill_on_drop: false }

  • The third example has a basically identical error.
  • The fourth example ends with the error

[INFO] [stderr] Error response from daemon: write /var/lib/docker/overlay2/aedbbd379c1a6eb2ba4545c5b28a51da14744c09bf720798329d19a1e8fffa42-init/.tmp-lower3624005366: no space left on device

@Skgland
Copy link
Contributor

Skgland commented Aug 24, 2024

The fourth one might be sorted properly with 397f3f0 from #729

@RalfJung
Copy link
Member Author

RalfJung commented Aug 24, 2024

Some more examples:

These have

[INFO] [stderr] error: could not compile `advent_of_code` (bin "04" test); 1 warning emitted

which is very strange -- it seems like rustc says "could not compile" but there has been no error emitted.

That's the same as what the first of the above has.

@RalfJung
Copy link
Member Author

RalfJung commented Aug 24, 2024

These errors all seem to be spurious, for the errors mentioned in the OP none of them reproduced in a rerun.

@Mark-Simulacrum
Copy link
Member

Interestingly, "could not compile" messaging actually comes from Cargo, not rustc (https://github.com/rust-lang/cargo/blob/3d2cf56d40041b4974bae74e7ad6b293f4f5daad/src/cargo/core/compiler/mod.rs#L426). That same code has a debug assert trying to cover this case. My suspicion is that we're seeing something in exec fail in an unexpected way -- but I'm not sure what it could be. It seems like this is probably a Cargo bug though.

bors added a commit to rust-lang/cargo that referenced this issue Aug 26, 2024
Log details of failure if no errors were seen

### What does this PR try to resolve?

Crater has started (some time ago, though exact point is not known) to see failures in crates without logging an error. We're suspecting that there's something wrong in Cargo or rustc, but it's also possible that Crater itself is at fault. In any case, this should provide more details.

cc rust-lang/crater#733

### How should we test and review this PR?

My hope is that this is reasonable to land without additional tests -- as-is, we expect it to never happen (there's even a debug assert later in the same code path) but it seems it *is* happening sometimes in the wild.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants