Skip to content

ci: Try cargo-careful #2607

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

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Conversation

larseggert
Copy link
Collaborator

Now that RalfJung/cargo-careful#35 got fixed.

@larseggert
Copy link
Collaborator Author

Except that fix isn't in a release yet. v0.4.3 is too old.

Copy link

github-actions bot commented May 2, 2025

Failed Interop Tests

QUIC Interop Runner, client vs. server, differences relative to 37c3aee.

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

Copy link

github-actions bot commented May 2, 2025

Benchmark results

1-conn/1-100mb-resp/mtu-1504 (aka. Download)/client:
       time:   [694.67 ms 698.59 ms 702.50 ms]
       thrpt:  [142.35 MiB/s 143.15 MiB/s 143.95 MiB/s]
1-conn/10_000-parallel-1b-resp/mtu-1504 (aka. RPS)/client:
       time:   [351.58 ms 353.08 ms 354.59 ms]
       thrpt:  [28.202 Kelem/s 28.322 Kelem/s 28.443 Kelem/s]
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) low mild
1-conn/1-1b-resp/mtu-1504 (aka. HPS)/client:
       time:   [25.566 ms 25.735 ms 25.916 ms]
       thrpt:  [38.587  elem/s 38.858  elem/s 39.114  elem/s]
Found 4 outliers among 100 measurements (4.00%)
  4 (4.00%) high mild
1-conn/1-100mb-req/mtu-1504 (aka. Upload)/client:
       time:   [1.8442 s 1.8652 s 1.8874 s]
       thrpt:  [52.984 MiB/s 53.613 MiB/s 54.224 MiB/s]
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild
decode 4096 bytes, mask ff:
       time:   [12.090 µs 12.129 µs 12.175 µs]
Found 19 outliers among 100 measurements (19.00%)
  3 (3.00%) low severe
  3 (3.00%) low mild
  2 (2.00%) high mild
  11 (11.00%) high severe
decode 1048576 bytes, mask ff:
       time:   [3.1246 ms 3.1305 ms 3.1382 ms]
Found 5 outliers among 100 measurements (5.00%)
  1 (1.00%) low mild
  4 (4.00%) high severe
decode 4096 bytes, mask 7f:
       time:   [20.154 µs 20.205 µs 20.262 µs]
Found 20 outliers among 100 measurements (20.00%)
  1 (1.00%) low severe
  4 (4.00%) low mild
  2 (2.00%) high mild
  13 (13.00%) high severe
decode 1048576 bytes, mask 7f:
       time:   [5.2474 ms 5.2589 ms 5.2720 ms]
Found 14 outliers among 100 measurements (14.00%)
  14 (14.00%) high severe
decode 4096 bytes, mask 3f:
       time:   [7.0034 µs 7.0227 µs 7.0509 µs]
Found 14 outliers among 100 measurements (14.00%)
  4 (4.00%) low severe
  4 (4.00%) low mild
  2 (2.00%) high mild
  4 (4.00%) high severe
decode 1048576 bytes, mask 3f:
       time:   [1.7921 ms 1.7978 ms 1.8048 ms]
Found 8 outliers among 100 measurements (8.00%)
  1 (1.00%) low mild
  1 (1.00%) high mild
  6 (6.00%) high severe
1000 streams of 1 bytes/multistream:
       time:   [25.541 ms 25.566 ms 25.592 ms]
Found 52 outliers among 500 measurements (10.40%)
  51 (10.20%) high mild
  1 (0.20%) high severe
1000 streams of 1000 bytes/multistream:
       time:   [143.92 ms 143.95 ms 143.99 ms]
Found 3 outliers among 500 measurements (0.60%)
  3 (0.60%) high mild
coalesce_acked_from_zero 1+1 entries:
       time:   [94.504 ns 94.791 ns 95.075 ns]
Found 8 outliers among 100 measurements (8.00%)
  2 (2.00%) low mild
  6 (6.00%) high severe
coalesce_acked_from_zero 3+1 entries:
       time:   [112.41 ns 112.62 ns 112.85 ns]
Found 7 outliers among 100 measurements (7.00%)
  7 (7.00%) high severe
coalesce_acked_from_zero 10+1 entries:
       time:   [112.08 ns 112.50 ns 113.00 ns]
Found 25 outliers among 100 measurements (25.00%)
  5 (5.00%) low severe
  6 (6.00%) low mild
  4 (4.00%) high mild
  10 (10.00%) high severe
coalesce_acked_from_zero 1000+1 entries:
       time:   [92.563 ns 92.982 ns 93.433 ns]
Found 6 outliers among 100 measurements (6.00%)
  4 (4.00%) high mild
  2 (2.00%) high severe
RxStreamOrderer::inbound_frame():
       time:   [117.02 ms 117.07 ms 117.13 ms]
Found 16 outliers among 100 measurements (16.00%)
  2 (2.00%) low severe
  4 (4.00%) low mild
  10 (10.00%) high mild
SentPackets::take_ranges:
       time:   [8.3415 µs 8.5607 µs 8.7710 µs]
Found 19 outliers among 100 measurements (19.00%)
  3 (3.00%) low severe
  12 (12.00%) low mild
  3 (3.00%) high mild
  1 (1.00%) high severe
transfer/pacing-false/varying-seeds:
       time:   [35.714 ms 35.785 ms 35.853 ms]
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) low mild
transfer/pacing-true/varying-seeds:
       time:   [36.733 ms 36.822 ms 36.912 ms]
transfer/pacing-false/same-seed:
       time:   [35.648 ms 35.696 ms 35.744 ms]
transfer/pacing-true/same-seed:
       time:   [37.115 ms 37.174 ms 37.233 ms]

Client/server transfer results

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 347.6 ± 93.1 298.3 776.1
neqo neqo reno 359.4 ± 110.2 291.8 831.0
neqo neqo cubic on 347.2 ± 93.4 288.8 767.3
neqo neqo cubic 315.9 ± 13.7 298.1 361.0
google neqo reno on 764.2 ± 90.7 566.0 956.1
google neqo reno 760.6 ± 86.7 564.4 895.5
google neqo cubic on 765.7 ± 83.4 566.2 980.2
google neqo cubic 754.6 ± 84.8 564.6 856.9
google google 571.2 ± 18.2 547.5 636.4
neqo msquic reno on 271.6 ± 34.1 245.0 427.5
neqo msquic reno 267.0 ± 28.1 244.6 390.1
neqo msquic cubic on 270.0 ± 36.3 243.1 448.8
neqo msquic cubic 259.1 ± 10.9 244.4 284.7
msquic msquic 185.7 ± 31.4 162.7 336.8

⬇️ Download logs

@mxinden
Copy link
Member

mxinden commented May 5, 2025

Wasn't aware of this. Cool. In favor once fix is released.

Copy link

codecov bot commented Jul 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.91%. Comparing base (d01857c) to head (463402c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2607      +/-   ##
==========================================
+ Coverage   94.88%   94.91%   +0.02%     
==========================================
  Files         115      115              
  Lines       34256    34256              
  Branches    34256    34256              
==========================================
+ Hits        32505    32513       +8     
+ Misses       1744     1734      -10     
- Partials        7        9       +2     
Components Coverage Δ
neqo-common 97.08% <ø> (ø)
neqo-crypto 89.63% <ø> (ø)
neqo-http3 93.71% <ø> (ø)
neqo-qpack 95.45% <ø> (ø)
neqo-transport 96.00% <ø> (+0.04%) ⬆️
neqo-udp 89.37% <ø> (ø)

@larseggert larseggert added the blocked Blocked on something else label Jul 3, 2025
@larseggert
Copy link
Collaborator Author

Blocked by RalfJung/cargo-careful#39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked on something else
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants