-
Notifications
You must be signed in to change notification settings - Fork 1k
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
deps: update webrtc to 0.11.0 #5448
base: master
Are you sure you want to change the base?
Conversation
EDIT: Solved, see later comment. Hey @jxs, I saw this PR as I was debugging some weird However, I get the following error now and I can't quite figure it out:
It's basically from the
I confirmed that this is only happening when switching from
|
Sorry for the previous comment. Seems like it was a distraction not related to this PR. I also had to patch the
|
rcgen 0.11 depends on ring 0.16.20 and that ring version does not compile on windows uwp arm64, so we are not able to use quic transport on that platform, which is somewhat a problem to us. Is there anything to help on getting rcgen update to happen and is this the branch that would be best candidate for getting merged? |
@jxs I was trying to wrap my head around the details for getting the rcgen updated to latest and seeing what happens and applying by hand same changes from (#5591). It leads for needing to update webrtc, and the end result is what this PR already contains. So I'm bit confused what there would be to do apart just get this PR merged and just close #5591 ? |
Hi Lassi, yeah I noticed that while giving another go at this PR some weeks ago. Sorry for the confusion I have meanwhile closed #5591. |
Thanks @jxs, I will try to reproduce the smoke test failure locally to see if there is anything I could do. |
I did run failing test of this branch locally (on macOs), but I could not reproduce the failure, and I don't have rights to trigger the CI to re-run tests to see if it's just flaky test (seems test failure is a timeout: https://github.com/libp2p/rust-libp2p/actions/runs/12953210303/job/36132382210?pr=5448). |
Yeah Locally I could not reproduce the issue as well, only on CI, but it's not a flaky test it definitely reproduces deterministically. |
I can! Somehow wasn't thinking of that... And can indeed reproduce the problem in CI ubuntu runner. Let's see if I can get any sensible reason for the test failure. |
It seems to be something to do how libp2p-webrtc smoke test tries to make local nodes to connect to each other using STUN in Webrtc ICE module. I have enabled trace logging and below are examples of sequence of discovery logs that keeps on repeating until it times out. Problem has to be something to do with either x86 platform or the Github runner network conditions, I tried running the PR locally on arm64 using Ubuntu 24.04 in Docker, and smoke test works fine. I'm not very familiar on under the hood workings of ICE and STUN and where to look further, but maybe the following logs gives some clues to someone who has more knowledge on what should or shouldn't happen. This uses the 0.12 version of webrtc which is what the PR uses.
I also tried to use master version of webrtc which fails as well, but the ice part has been changed, so the logs reveal bit different information.
|
Description