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

ci(nss): fix typo USE_SYSTEM USE_SYSTEM_NSS #2145

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

mxinden
Copy link
Collaborator

@mxinden mxinden commented Oct 2, 2024

No description provided.

Copy link

codecov bot commented Oct 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.35%. Comparing base (a4c4ccd) to head (f42a376).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2145      +/-   ##
==========================================
- Coverage   95.35%   95.35%   -0.01%     
==========================================
  Files         112      112              
  Lines       36345    36345              
==========================================
- Hits        34657    34656       -1     
- Misses       1688     1689       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martinthomson martinthomson added this pull request to the merge queue Oct 2, 2024
Copy link

github-actions bot commented Oct 2, 2024

Benchmark results

Performance differences relative to a4c4ccd.

coalesce_acked_from_zero 1+1 entries: No change in performance detected.
       time:   [98.652 ns 99.306 ns 100.17 ns]
       change: [-0.8204% -0.0623% +0.8609%] (p = 0.91 > 0.05)

Found 12 outliers among 100 measurements (12.00%)
7 (7.00%) high mild
5 (5.00%) high severe

coalesce_acked_from_zero 3+1 entries: No change in performance detected.
       time:   [116.59 ns 116.97 ns 117.36 ns]
       change: [-0.5644% -0.1733% +0.1871%] (p = 0.38 > 0.05)

Found 18 outliers among 100 measurements (18.00%)
2 (2.00%) low severe
2 (2.00%) low mild
3 (3.00%) high mild
11 (11.00%) high severe

coalesce_acked_from_zero 10+1 entries: Change within noise threshold.
       time:   [116.01 ns 116.35 ns 116.78 ns]
       change: [-2.0685% -0.9588% -0.0989%] (p = 0.05 < 0.05)

Found 15 outliers among 100 measurements (15.00%)
6 (6.00%) low severe
1 (1.00%) low mild
5 (5.00%) high mild
3 (3.00%) high severe

coalesce_acked_from_zero 1000+1 entries: Change within noise threshold.
       time:   [97.301 ns 97.474 ns 97.667 ns]
       change: [-2.2049% -1.0876% +0.0649%] (p = 0.04 < 0.05)

Found 14 outliers among 100 measurements (14.00%)
6 (6.00%) high mild
8 (8.00%) high severe

RxStreamOrderer::inbound_frame(): Change within noise threshold.
       time:   [110.88 ms 110.94 ms 111.00 ms]
       change: [-0.6022% -0.3958% -0.2550%] (p = 0.00 < 0.05)

Found 13 outliers among 100 measurements (13.00%)
9 (9.00%) low mild
4 (4.00%) high mild

transfer/pacing-false/varying-seeds: No change in performance detected.
       time:   [26.929 ms 27.971 ms 29.000 ms]
       change: [-4.3381% +1.3921% +7.3039%] (p = 0.64 > 0.05)

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

transfer/pacing-true/varying-seeds: No change in performance detected.
       time:   [35.441 ms 37.017 ms 38.598 ms]
       change: [-8.4597% -2.6072% +3.2677%] (p = 0.39 > 0.05)
transfer/pacing-false/same-seed: No change in performance detected.
       time:   [25.757 ms 26.590 ms 27.433 ms]
       change: [-7.8196% -3.5598% +0.4616%] (p = 0.10 > 0.05)
transfer/pacing-true/same-seed: No change in performance detected.
       time:   [39.599 ms 41.651 ms 43.797 ms]
       change: [-8.1043% -1.1319% +6.2190%] (p = 0.76 > 0.05)

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

1-conn/1-100mb-resp (aka. Download)/client: No change in performance detected.
       time:   [113.13 ms 113.60 ms 114.04 ms]
       thrpt:  [876.87 MiB/s 880.24 MiB/s 883.90 MiB/s]
change:
       time:   [-0.9537% -0.3193% +0.3480%] (p = 0.35 > 0.05)
       thrpt:  [-0.3468% +0.3204% +0.9629%]

Found 5 outliers among 100 measurements (5.00%)
2 (2.00%) low severe
3 (3.00%) low mild

1-conn/10_000-parallel-1b-resp (aka. RPS)/client: No change in performance detected.
       time:   [311.06 ms 314.86 ms 318.64 ms]
       thrpt:  [31.383 Kelem/s 31.761 Kelem/s 32.148 Kelem/s]
change:
       time:   [-2.7492% -1.3046% +0.2160%] (p = 0.11 > 0.05)
       thrpt:  [-0.2156% +1.3219% +2.8269%]
1-conn/1-1b-resp (aka. HPS)/client: No change in performance detected.
       time:   [34.151 ms 34.300 ms 34.465 ms]
       thrpt:  [29.015  elem/s 29.154  elem/s 29.281  elem/s]
change:
       time:   [-0.0999% +0.6011% +1.3414%] (p = 0.11 > 0.05)
       thrpt:  [-1.3236% -0.5975% +0.1000%]

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

Client/server transfer results

Transfer of 33554432 bytes over loopback.

Client Server CC Pacing Mean [ms] Min [ms] Max [ms] Relative
msquic msquic 126.1 ± 45.3 90.9 231.3 1.00
neqo msquic reno on 257.8 ± 66.2 222.7 464.2 1.00
neqo msquic reno 210.3 ± 4.4 204.0 221.0 1.00
neqo msquic cubic on 242.4 ± 63.3 206.0 436.5 1.00
neqo msquic cubic 227.4 ± 18.0 203.3 259.9 1.00
msquic neqo reno on 114.9 ± 63.9 81.3 349.9 1.00
msquic neqo reno 142.7 ± 82.9 85.9 352.8 1.00
msquic neqo cubic on 130.6 ± 80.1 83.2 401.3 1.00
msquic neqo cubic 118.2 ± 64.7 81.7 334.2 1.00
neqo neqo reno on 212.8 ± 139.7 129.1 586.4 1.00
neqo neqo reno 215.2 ± 109.0 130.3 409.5 1.00
neqo neqo cubic on 201.2 ± 90.2 128.3 423.2 1.00
neqo neqo cubic 192.0 ± 73.1 126.9 365.3 1.00

⬇️ Download logs

Copy link

github-actions bot commented Oct 2, 2024

Failed Interop Tests

QUIC Interop Runner, client vs. server

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

Merged via the queue into mozilla:main with commit bafc244 Oct 2, 2024
64 of 65 checks passed
Copy link

github-actions bot commented Oct 2, 2024

Firefox builds for this PR

The following builds are available for testing. Crossed-out builds did not succeed.

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

Successfully merging this pull request may close these issues.

2 participants