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

Suprious network errors when downloading source code #513

Closed
thomaseizinger opened this issue Aug 7, 2023 · 2 comments
Closed

Suprious network errors when downloading source code #513

thomaseizinger opened this issue Aug 7, 2023 · 2 comments
Labels
C-bug Category: doesn't meet expectations

Comments

@thomaseizinger
Copy link
Contributor

Steps to reproduce the bug with the above code

We've recently hit a fairly high number of spurious network errors in the use of cargo semver-checks. Not sure how much we can do about them but here are some links:

Actual Behaviour

Expected Behaviour

To run cargo semver-checks in CI without spurious errors.

Generated System Information

Hard to obtain for CI but we are running version 0.22.1: https://github.com/libp2p/rust-libp2p/blob/master/.github/actions/cargo-semver-checks/action.yml#L10

Build Configuration

No response

Additional Context

No response

@thomaseizinger thomaseizinger added the C-bug Category: doesn't meet expectations label Aug 7, 2023
@obi1kenobi
Copy link
Owner

It seems that GitHub was also having some sort of network or system outage very close to the same time — possibly related.
https://twitter.com/boshen_c/status/1688431701888999424

Interestingly, this also seems to have uncovered an underlying bug in cargo involving some incorrect string slicing, presumably in some error case where the server responds in an unexpected way: https://github.com/rust-lang/cargo/blob/0.72.1/src/cargo/ops/registry.rs#L715

I can report the cargo bug upstream. That might end up resolving the problem — I'd be surprised if cargo doesn't already have built-in retries for spurious network blips. Since this error happens during our cargo metadata call, the better place to trigger a retry is within cargo itself.

@thomaseizinger
Copy link
Contributor Author

Thanks for reporting it upstream, I think we can close this then :)

@thomaseizinger thomaseizinger closed this as not planned Won't fix, can't repro, duplicate, stale Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: doesn't meet expectations
Projects
None yet
Development

No branches or pull requests

2 participants