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

build(deps): bump tokio from 1.39.2 to 1.43.1 in the cargo group #2570

Merged
merged 2 commits into from
Apr 9, 2025

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 8, 2025

Bumps the cargo group with 1 update: tokio.

Updates tokio from 1.39.2 to 1.43.1

Release notes

Sourced from tokio's releases.

Tokio v1.43.0

1.43.0 (Jan 8th, 2025)

Added

  • net: add UdpSocket::peek methods (#7068)
  • net: add support for Haiku OS (#7042)
  • process: add Command::into_std() (#7014)
  • signal: add SignalKind::info on illumos (#6995)
  • signal: add support for realtime signals on illumos (#7029)

Fixed

  • io: don't call set_len before initializing vector in Blocking (#7054)
  • macros: suppress clippy::needless_return in #[tokio::main] (#6874)
  • runtime: fix thread parking on WebAssembly (#7041)

Changes

  • chore: use unsync loads for unsync_load (#7073)
  • io: use Buf::put_bytes in Repeat read impl (#7055)
  • task: drop the join waker of a task eagerly (#6986)

Changes to unstable APIs

  • metrics: improve flexibility of H2Histogram Configuration (#6963)
  • taskdump: add accessor methods for backtrace (#6975)

Documented

  • io: clarify ReadBuf::uninit allows initialized buffers as well (#7053)
  • net: fix ambiguity in TcpStream::try_write_vectored docs (#7067)
  • runtime: fix LocalRuntime doc links (#7074)
  • sync: extend documentation for watch::Receiver::wait_for (#7038)
  • sync: fix typos in OnceCell docs (#7047)

#6874: tokio-rs/tokio#6874 #6963: tokio-rs/tokio#6963 #6975: tokio-rs/tokio#6975 #6986: tokio-rs/tokio#6986 #6995: tokio-rs/tokio#6995 #7014: tokio-rs/tokio#7014 #7029: tokio-rs/tokio#7029 #7038: tokio-rs/tokio#7038 #7041: tokio-rs/tokio#7041 #7042: tokio-rs/tokio#7042 #7047: tokio-rs/tokio#7047 #7053: tokio-rs/tokio#7053 #7054: tokio-rs/tokio#7054 #7055: tokio-rs/tokio#7055

... (truncated)

Commits
  • a7b658c chore: prepare Tokio v1.43.1 release
  • c1c8d10 Merge remote-tracking branch 'origin/tokio-1.38.x' into forward-port-1.38.x
  • aa303bc chore: prepare Tokio v1.38.2 release
  • 7b6ccb5 chore: backport CI fixes
  • 4b174ce sync: fix cloning value when receiving from broadcast channel
  • 5f3296d chore: prepare Tokio v1.43.0 (#7079)
  • cc974a6 chore: prepare tokio-macros v2.5.0 (#7078)
  • 15495fd metrics: improve flexibility of H2Histogram Configuration (#6963)
  • ad41834 io: don't call set_len before initializing vector in Blocking (#7054)
  • bd3e857 runtime: move is_join_waker_set assertion in unset_waker (#7072)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions
    You can disable automated security fix PRs for this repo from the Security Alerts page.

@dependabot dependabot bot added dependencies Pull requests that update a dependency file rust Pull requests that update Rust code labels Apr 8, 2025
Copy link

github-actions bot commented Apr 8, 2025

Failed Interop Tests

QUIC Interop Runner, client vs. server, differences relative to c1679d8.

neqo-latest as client

neqo-latest as server

All results

Succeeded Interop Tests

QUIC Interop Runner, client vs. server

neqo-latest as client

neqo-latest as server

Unsupported Interop Tests

QUIC Interop Runner, client vs. server

neqo-latest as client

neqo-latest as server

@dependabot dependabot bot force-pushed the dependabot/cargo/cargo-8c74fa0cce branch from 98811c9 to c56cc93 Compare April 8, 2025 11:10
Copy link
Collaborator

@mxinden mxinden left a comment

Choose a reason for hiding this comment

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

Looking at the CI failures.

@mxinden mxinden enabled auto-merge April 8, 2025 13:05
@dependabot dependabot bot force-pushed the dependabot/cargo/cargo-8c74fa0cce branch from c56cc93 to a604605 Compare April 8, 2025 14:34
Bumps the cargo group with 1 update: [tokio](https://github.com/tokio-rs/tokio).


Updates `tokio` from 1.39.2 to 1.43.1
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](tokio-rs/tokio@tokio-1.39.2...tokio-1.43.1)

---
updated-dependencies:
- dependency-name: tokio
  dependency-version: 1.43.1
  dependency-type: direct:production
  dependency-group: cargo
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/cargo/cargo-8c74fa0cce branch from a604605 to cda0202 Compare April 8, 2025 17:21
@mxinden
Copy link
Collaborator

mxinden commented Apr 8, 2025

@larseggert do you have an intuition how codecov thinks that this pull request has a -2.6% change in test coverage compared to main? If not, I will debug this further.

image

https://app.codecov.io/gh/mozilla/neqo/pull/2570

@larseggert
Copy link
Collaborator

I don't. I wonder if we should revert the MC/DC stuff, it seems to lead to random noise like this.

@mxinden
Copy link
Collaborator

mxinden commented Apr 9, 2025

Thanks. Worth a try. For the record, I created #2575.

Copy link

codecov bot commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.54%. Comparing base (911ee9c) to head (63b9075).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2570   +/-   ##
=======================================
  Coverage   95.54%   95.54%           
=======================================
  Files         115      115           
  Lines       37275    37275           
  Branches    37275    37275           
=======================================
  Hits        35616    35616           
  Misses       1651     1651           
  Partials        8        8           
Components Coverage Δ
neqo-common 97.17% <ø> (ø)
neqo-crypto 90.45% <ø> (ø)
neqo-http3 94.50% <ø> (ø)
neqo-qpack 96.29% <ø> (ø)
neqo-transport 96.54% <ø> (ø)
neqo-udp 90.65% <ø> (ø)

@mxinden mxinden added this pull request to the merge queue Apr 9, 2025
Merged via the queue into main with commit 28c0ea7 Apr 9, 2025
81 checks passed
@mxinden mxinden deleted the dependabot/cargo/cargo-8c74fa0cce branch April 9, 2025 19:37
Copy link

github-actions bot commented Apr 9, 2025

Benchmark results

Performance differences relative to 911ee9c.

1-conn/1-100mb-resp/mtu-1504 (aka. Download)/client: 💚 Performance has improved.
       time:   [709.12 ms 713.13 ms 717.18 ms]
       thrpt:  [139.43 MiB/s 140.23 MiB/s 141.02 MiB/s]
change:
       time:   [-3.4262% -2.5569% -1.7289%] (p = 0.00 < 0.05)
       thrpt:  [+1.7593% +2.6240% +3.5478%]
1-conn/10_000-parallel-1b-resp/mtu-1504 (aka. RPS)/client: Change within noise threshold.
       time:   [350.57 ms 352.12 ms 353.67 ms]
       thrpt:  [28.275 Kelem/s 28.400 Kelem/s 28.525 Kelem/s]
change:
       time:   [+0.1946% +0.8111% +1.4121%] (p = 0.01 < 0.05)
       thrpt:  [-1.3924% -0.8046% -0.1942%]

Found 2 outliers among 100 measurements (2.00%)
1 (1.00%) low mild
1 (1.00%) high mild

1-conn/1-1b-resp/mtu-1504 (aka. HPS)/client: No change in performance detected.
       time:   [25.065 ms 25.223 ms 25.387 ms]
       thrpt:  [39.390  elem/s 39.646  elem/s 39.897  elem/s]
change:
       time:   [-0.7505% +0.1914% +1.1397%] (p = 0.69 > 0.05)
       thrpt:  [-1.1269% -0.1911% +0.7562%]
1-conn/1-100mb-req/mtu-1504 (aka. Upload)/client: 💔 Performance has regressed.
       time:   [1.8633 s 1.8798 s 1.8970 s]
       thrpt:  [52.714 MiB/s 53.196 MiB/s 53.669 MiB/s]
change:
       time:   [+1.9508% +3.6105% +5.3561%] (p = 0.00 < 0.05)
       thrpt:  [-5.0838% -3.4847% -1.9135%]

Found 2 outliers among 100 measurements (2.00%)
2 (2.00%) high mild

decode 4096 bytes, mask ff: Change within noise threshold.
       time:   [11.980 µs 11.988 µs 12.001 µs]
       change: [-1.6231% -1.0768% -0.3902%] (p = 0.00 < 0.05)

Found 7 outliers among 100 measurements (7.00%)
1 (1.00%) low severe
1 (1.00%) low mild
1 (1.00%) high mild
4 (4.00%) high severe

decode 1048576 bytes, mask ff: 💚 Performance has improved.
       time:   [2.9497 ms 2.9558 ms 2.9636 ms]
       change: [-6.3493% -5.9308% -5.5212%] (p = 0.00 < 0.05)

Found 5 outliers among 100 measurements (5.00%)
5 (5.00%) high severe

decode 4096 bytes, mask 7f: Change within noise threshold.
       time:   [20.011 µs 20.061 µs 20.117 µs]
       change: [-1.6841% -1.0549% -0.4583%] (p = 0.00 < 0.05)

Found 21 outliers among 100 measurements (21.00%)
5 (5.00%) low severe
1 (1.00%) low mild
1 (1.00%) high mild
14 (14.00%) high severe

decode 1048576 bytes, mask 7f: 💚 Performance has improved.
       time:   [4.7995 ms 4.8122 ms 4.8257 ms]
       change: [-8.7289% -8.4062% -8.0698%] (p = 0.00 < 0.05)

Found 15 outliers among 100 measurements (15.00%)
15 (15.00%) high severe

decode 4096 bytes, mask 3f: 💚 Performance has improved.
       time:   [6.3103 µs 6.3198 µs 6.3369 µs]
       change: [-10.189% -9.7872% -9.3153%] (p = 0.00 < 0.05)

Found 16 outliers among 100 measurements (16.00%)
2 (2.00%) low severe
3 (3.00%) low mild
5 (5.00%) high mild
6 (6.00%) high severe

decode 1048576 bytes, mask 3f: 💔 Performance has regressed.
       time:   [2.1512 ms 2.1583 ms 2.1667 ms]
       change: [+19.321% +19.952% +20.588%] (p = 0.00 < 0.05)

Found 10 outliers among 100 measurements (10.00%)
1 (1.00%) high mild
9 (9.00%) high severe

1 streams of 1 bytes/multistream: No change in performance detected.
       time:   [72.796 µs 73.015 µs 73.237 µs]
       change: [-0.4554% -0.0615% +0.3549%] (p = 0.77 > 0.05)

Found 1 outliers among 100 measurements (1.00%)
1 (1.00%) high mild

1000 streams of 1 bytes/multistream: 💔 Performance has regressed.
       time:   [25.286 ms 25.320 ms 25.354 ms]
       change: [+1.6601% +1.8688% +2.0730%] (p = 0.00 < 0.05)
10000 streams of 1 bytes/multistream: 💔 Performance has regressed.
       time:   [1.6999 s 1.7015 s 1.7031 s]
       change: [+1.1394% +1.2814% +1.4316%] (p = 0.00 < 0.05)

Found 13 outliers among 100 measurements (13.00%)
7 (7.00%) low mild
6 (6.00%) high mild

1 streams of 1000 bytes/multistream: No change in performance detected.
       time:   [74.471 µs 75.541 µs 77.043 µs]
       change: [-2.9160% -0.2351% +2.5749%] (p = 0.87 > 0.05)

Found 2 outliers among 100 measurements (2.00%)
2 (2.00%) high severe

100 streams of 1000 bytes/multistream: No change in performance detected.
       time:   [3.3779 ms 3.3842 ms 3.3910 ms]
       change: [-0.1023% +0.1855% +0.4792%] (p = 0.19 > 0.05)

Found 20 outliers among 100 measurements (20.00%)
20 (20.00%) high severe

1000 streams of 1000 bytes/multistream: Change within noise threshold.
       time:   [143.80 ms 143.87 ms 143.94 ms]
       change: [+0.5853% +0.6579% +0.7281%] (p = 0.00 < 0.05)
coalesce_acked_from_zero 1+1 entries: No change in performance detected.
       time:   [94.772 ns 95.141 ns 95.524 ns]
       change: [-1.1474% -0.3690% +0.3068%] (p = 0.35 > 0.05)

Found 14 outliers among 100 measurements (14.00%)
12 (12.00%) high mild
2 (2.00%) high severe

coalesce_acked_from_zero 3+1 entries: No change in performance detected.
       time:   [112.55 ns 112.91 ns 113.29 ns]
       change: [-0.9335% -0.4183% +0.0349%] (p = 0.11 > 0.05)

Found 12 outliers among 100 measurements (12.00%)
1 (1.00%) low severe
1 (1.00%) low mild
10 (10.00%) high severe

coalesce_acked_from_zero 10+1 entries: Change within noise threshold.
       time:   [111.84 ns 112.25 ns 112.74 ns]
       change: [-1.2865% -0.8794% -0.4895%] (p = 0.00 < 0.05)

Found 20 outliers among 100 measurements (20.00%)
6 (6.00%) low severe
6 (6.00%) low mild
2 (2.00%) high mild
6 (6.00%) high severe

coalesce_acked_from_zero 1000+1 entries: No change in performance detected.
       time:   [93.033 ns 93.463 ns 93.913 ns]
       change: [-1.9624% -0.9926% +0.0206%] (p = 0.06 > 0.05)

Found 4 outliers among 100 measurements (4.00%)
3 (3.00%) high mild
1 (1.00%) high severe

RxStreamOrderer::inbound_frame(): 💔 Performance has regressed.
       time:   [117.04 ms 117.09 ms 117.14 ms]
       change: [+1.9292% +2.0017% +2.0734%] (p = 0.00 < 0.05)

Found 9 outliers among 100 measurements (9.00%)
5 (5.00%) low mild
4 (4.00%) high mild

SentPackets::take_ranges: No change in performance detected.
       time:   [8.3156 µs 8.5543 µs 8.7701 µs]
       change: [-4.7915% -1.7543% +1.2616%] (p = 0.26 > 0.05)

Found 21 outliers among 100 measurements (21.00%)
9 (9.00%) low severe
9 (9.00%) low mild
3 (3.00%) high mild

transfer/pacing-false/varying-seeds: Change within noise threshold.
       time:   [35.721 ms 35.790 ms 35.858 ms]
       change: [-1.4005% -1.1234% -0.8582%] (p = 0.00 < 0.05)

Found 1 outliers among 100 measurements (1.00%)
1 (1.00%) low mild

transfer/pacing-true/varying-seeds: Change within noise threshold.
       time:   [36.483 ms 36.592 ms 36.700 ms]
       change: [-1.1385% -0.7317% -0.3019%] (p = 0.00 < 0.05)

Found 1 outliers among 100 measurements (1.00%)
1 (1.00%) high mild

transfer/pacing-false/same-seed: Change within noise threshold.
       time:   [35.352 ms 35.396 ms 35.441 ms]
       change: [-1.4171% -1.2522% -1.0778%] (p = 0.00 < 0.05)
transfer/pacing-true/same-seed: Change within noise threshold.
       time:   [36.806 ms 36.868 ms 36.930 ms]
       change: [-1.6423% -1.4000% -1.1604%] (p = 0.00 < 0.05)

Client/server transfer results

Performance differences relative to 911ee9c.

Transfer of 33554432 bytes over loopback, 30 runs. All unit-less numbers are in milliseconds.

Client Server CC Pacing Mean ± σ Min Max MiB/s ± σ Δ main Δ main
neqo neqo reno on 334.1 ± 43.1 304.7 536.8 95.8 ± 0.7 7.5 2.3%
neqo neqo reno 395.9 ± 193.0 302.8 1088.7 80.8 ± 0.2 33.8 9.3%
neqo neqo cubic on 346.5 ± 49.8 309.6 528.7 92.3 ± 0.6 15.2 4.6%
neqo neqo cubic 331.1 ± 24.0 302.0 416.0 96.7 ± 1.3 3.2 1.0%
google neqo reno on 766.5 ± 83.5 577.4 842.2 41.7 ± 0.4 1.9 0.2%
google neqo reno 771.5 ± 88.6 594.1 1008.4 41.5 ± 0.4 9.2 1.2%
google neqo cubic on 768.5 ± 81.8 583.6 862.2 41.6 ± 0.4 -2.6 -0.3%
google neqo cubic 768.4 ± 88.6 575.0 952.3 41.6 ± 0.4 1.0 0.1%
google google 572.8 ± 34.5 551.5 744.9 55.9 ± 0.9 0.1 0.0%
neqo msquic reno on 265.7 ± 17.3 245.2 326.8 120.4 ± 1.8 -15.0 -5.3%
neqo msquic reno 267.3 ± 24.1 245.3 361.9 119.7 ± 1.3 -27.0 -9.2%
neqo msquic cubic on 262.6 ± 18.8 240.0 342.0 121.8 ± 1.7 -2.7 -1.0%
neqo msquic cubic 266.6 ± 30.4 244.6 414.2 120.0 ± 1.1 1.9 0.7%
msquic msquic 178.4 ± 35.6 148.9 320.0 179.3 ± 0.9 5.8 3.4%

⬇️ Download logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants