8000 Static site for rustls.dev, including home for performance reports by ctz · Pull Request #2201 · rustls/rustls · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Static site for rustls.dev, including home for performance reports #2201

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

Merged
merged 4 commits into from
Nov 12, 2024

Conversation

ctz
Copy link
Member
@ctz ctz commented Nov 8, 2024

This is a more permanent and visible home for https://github.com/rustls/rustls/wiki/Performance-measurements-(2024%E2%80%9010%E2%80%9018) -- with more to come (for arm64, and multithreaded cases).

At the same time, I started a landing site for rustls.dev. Feedback on what we say there gratefully accepted!

To see what this looks like, schedule https://github.com/rustls/rustls/actions/workflows/docs.yml for the branch jbp-landing-page and then look at https://rustls.dev/ . Each time main changes (and nightly) this will be lost and you'll see a github 404.

Copy link
rustls-benchmarking bot commented Nov 8, 2024

Benchmark results

Instruction counts

Significant differences

There are no significant instruction count differences

Other differences

Click to expand
Scenario Baseline Candidate Diff Threshold
handshake_no_resume_ring_1.3_ecdsap256_aes_server 1612605 1616969 4364 (0.27%) 1.06%
handshake_no_resume_aws_lc_rs_1.3_ecdsap384_aes_client 8606451 8583257 -23194 (-0.27%) 0.98%
handshake_no_resume_aws_lc_rs_1.3_rsa_chacha_server 10734033 10717238 -16795 (-0.16%) 1.16%
handshake_no_resume_ring_1.3_ecdsap256_aes_client 3623290 3627707 4417 (0.12%) 0.45%
handshake_no_resume_aws_lc_rs_1.3_ecdsap384_chacha_client 8593831 8583393 -10438 (-0.12%) 0.76%
handshake_no_resume_aws_lc_rs_1.3_rsa_aes_server 10698553 10707255 8702 (0.08%) 1.00%
handshake_no_resume_aws_lc_rs_1.3_ecdsap256_chacha_client 3086532 3087162 630 (0.02%) 0.27%
handshake_no_resume_aws_lc_rs_1.3_ecdsap384_aes_server 2076544 2076730 186 (0.01%) 0.20%
handshake_no_resume_aws_lc_rs_1.2_rsa_aes_server 10467216 10467774 558 (0.01%) 1.39%
handshake_no_resume_ring_1.3_ecdsap256_chacha_client 3624678 3624792 114 (0.00%) 0.48%
handshake_no_resume_aws_lc_rs_1.3_rsa_chacha_client 1932569 1932591 22 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap256_aes_server 28965667 28965966 299 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_rsa_chacha_server 28883950 28883707 -243 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap256_chacha_client 28190230 28190028 -202 (-0.00%) 0.20%
handshake_no_resume_aws_lc_rs_1.3_ecdsap256_aes_server 1179494 1179487 -7 (-0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_rsa_aes_client 27860596 27860755 159 (0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_rsa_chacha_client 28194107 28193948 -159 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_rsa_aes_server 30410751 30410915 164 (0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_rsa_aes_client 28224581 28224732 151 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap256_aes_client 27856253 27856399 146 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_rsa_chacha_client 27800402 27800266 -136 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap384_chacha_server 30370660 30370804 144 (0.00%) 0.20%
handshake_no_resume_aws_lc_rs_1.2_rsa_aes_client 1717479 1717472 -7 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap384_chacha_client 28187169 28187058 -111 (-0.00%) 0.20%
handshake_no_resume_aws_lc_rs_1.3_rsa_aes_client 1925848 1925855 7 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_rsa_aes_server 28962967 28963041 74 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap256_chacha_server 28886497 28886428 -69 (-0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap384_chacha_server 28886535 28886600 65 (0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap256_aes_server 30413727 30413794 67 (0.00%) 0.20%
handshake_no_resume_aws_lc_rs_1.3_ecdsap256_aes_client 3084560 3084566 6 (0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap256_chacha_server 30370242 30370299 57 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap256_chacha_client 27796005 27796055 50 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap384_chacha_client 27793450 27793402 -48 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_rsa_chacha_server 30367265 30367213 -52 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap384_aes_server 30414068 30414103 35 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap384_aes_client 27853766 27853737 -29 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap384_aes_client 28217748 28217727 -21 (-0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.3_ecdsap256_aes_client 28220708 28220695 -13 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_rsa_aes_server 46480088 46480100 12 (0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap384_aes_server 46460134 46460144 10 (0.00%) 0.20%
transfer_no_resume_ring_1.3_rsa_aes_client 58352513 58352502 -11 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap256_aes_client 58251825 58251815 -10 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap256_aes_server 46467121 46467127 6 (0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap256_chacha_server 80648886 80648876 -10 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap256_chacha_client 92673771 92673762 -9 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_rsa_chacha_client 92684671 92684662 -9 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_rsa_chacha_client 92718851 92718860 9 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.3_ecdsap384_aes_server 28965828 28965826 -2 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap256_aes_client 58341603 58341607 4 (0.00%) 0.20%
transfer_no_resume_ring_1.2_rsa_aes_server 46389591 46389588 -3 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap256_aes_server 46467808 46467811 3 (0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap384_chacha_client 92682835 92682830 -5 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap256_chacha_server 80535647 80535643 -4 (-0.00%) 0.20%
transfer_no_resume_ring_1.2_rsa_aes_client 58236456 58236454 -2 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap384_aes_client 58254054 58254052 -2 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_rsa_chacha_server 80554934 80554936 2 (0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.2_rsa_aes_server 46434710 46434709 -1 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap384_aes_server 46470690 46470689 -1 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_rsa_aes_server 46487100 46487101 1 (0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_rsa_aes_client 58256132 58256131 -1 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.2_rsa_aes_client 68683802 68683801 -1 (-0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap384_chacha_server 80540443 80540444 1 (0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_rsa_chacha_server 80661853 80661852 -1 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap256_chacha_client 92712626 92712625 -1 (-0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap384_chacha_client 92714857 92714857 0 (0.00%) 0.20%
transfer_no_resume_ring_1.3_ecdsap384_aes_client 58348757 58348757 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap384_aes_server 41601939 41601939 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_rsa_aes_client 40292413 40292413 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap384_chacha_server 41483709 41483709 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_rsa_chacha_client 40515800 40515800 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap256_chacha_server 41964748 41964748 0 (0.00%) 0.20%
handshake_no_resume_aws_lc_rs_1.3_ecdsap256_chacha_server 1181756 1181756 0 (0.00%) 0.20%
handshake_no_resume_ring_1.2_rsa_aes_server 11292386 11292386 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap384_chacha_server 41964968 41964968 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap256_aes_client 40586050 40586050 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_rsa_aes_server 41599268 41599268 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_rsa_chacha_client 2662046 2662046 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap256_chacha_server 41483631 41483631 0 (0.00%) 0.20%
handshake_tickets_ring_1.2_rsa_aes_server 4696093 4696093 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap384_aes_client 40582783 40582783 0 (0.00%) 0.20%
handshake_no_resume_ring_1.2_rsa_aes_client 2563562 2563562 0 (0.00%) 0.20%
transfer_no_resume_aws_lc_rs_1.3_ecdsap384_chacha_server 80641258 80641258 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap256_aes_client 40288362 40288362 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap256_chacha_client 40193982 40193982 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_rsa_chacha_client 40198033 40198033 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_rsa_aes_client 2656181 2656181 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap256_chacha_client 40511800 40511800 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_rsa_aes_server 11425365 11425365 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_ecdsap256_chacha_server 1614006 1614006 0 (0.00%) 1.06%
handshake_no_resume_ring_1.3_ecdsap384_chacha_client 35185016 35185016 0 (0.00%) 0.20%
handshake_session_id_ring_1.2_rsa_aes_server 4253569 4253569 0 (0.00%) 0.20%
handshake_tickets_ring_1.2_rsa_aes_client 4498203 4498203 0 (0.00%) 0.20%
handshake_session_id_ring_1.2_rsa_aes_client 4237209 4237209 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap256_aes_server 41601861 41601861 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_ecdsap384_aes_server 7567070 7567070 0 (0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.2_rsa_aes_client 4216429 4216429 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_rsa_chacha_server 41481038 41481038 0 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.2_rsa_aes_server 3893661 3893661 0 (0.00%) 0.20%
handshake_tickets_aws_lc_rs_1.2_rsa_aes_server 5015140 5015140 0 (0.00%) 0.20%
handshake_no_resume_aws_lc_rs_1.3_ecdsap384_chacha_server 2079569 2079569 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_ecdsap384_aes_client 35183086 35183086 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_ecdsap384_chacha_server 7569248 7569248 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_rsa_aes_client 40590050 40590050 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap384_chacha_client 40508525 40508525 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap384_aes_server 42063278 42063278 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap384_aes_client 40285413 40285413 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_rsa_chacha_server 41961848 41961848 0 (0.00%) 0.20%
handshake_session_id_ring_1.3_ecdsap384_chacha_client 40191025 40191025 0 (0.00%) 0.20%
handshake_no_resume_ring_1.3_rsa_chacha_server 11431345 11431345 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_ecdsap256_aes_server 42063058 42063058 0 (0.00%) 0.20%
handshake_session_id_aws_lc_rs_1.2_rsa_aes_client 3873865 3873865 0 (0.00%) 0.20%
handshake_tickets_ring_1.3_rsa_aes_server 42060158 42060158 0 (0.00%) 0.20%

Wall-time

Significant differences

There are no significant wall-time differences

Other differences

Click to expand
Scenario Baseline Candidate Diff Threshold
handshake_tickets_aws_lc_rs_1.3_ecdsap256_aes 4.51 ms 4.47 ms -0.04 ms (-0.86%) 1.54%
handshake_tickets_aws_lc_rs_1.3_ecdsap384_aes 5.23 ms 5.19 ms -0.04 ms (-0.79%) 1.31%
handshake_session_id_aws_lc_rs_1.3_ecdsap256_aes 4.31 ms 4.28 ms -0.03 ms (-0.75%) 2.13%
handshake_tickets_aws_lc_rs_1.3_rsa_aes 5.15 ms 5.11 ms -0.04 ms (-0.75%) 1.53%
handshake_tickets_aws_lc_rs_1.3_ecdsap256_chacha 4.48 ms 4.44 ms -0.03 ms (-0.74%) 1.90%
handshake_session_id_aws_lc_rs_1.3_ecdsap256_chacha 4.28 ms 4.25 ms -0.03 ms (-0.74%) 1.53%
handshake_tickets_aws_lc_rs_1.3_rsa_chacha 5.13 ms 5.10 ms -0.04 ms (-0.70%) 1.49%
handshake_session_id_aws_lc_rs_1.3_rsa_aes 4.96 ms 4.92 ms -0.03 ms (-0.67%) 1.61%
handshake_session_id_aws_lc_rs_1.3_ecdsap384_aes 5.03 ms 5.00 ms -0.03 ms (-0.62%) 1.57%
handshake_tickets_aws_lc_rs_1.3_ecdsap384_chacha 5.19 ms 5.16 ms -0.03 ms (-0.60%) 1.55%
handshake_session_id_aws_lc_rs_1.3_ecdsap384_chacha 4.99 ms 4.96 ms -0.03 ms (-0.55%) 1.58%
handshake_session_id_aws_lc_rs_1.3_rsa_chacha 4.93 ms 4.90 ms -0.03 ms (-0.55%) 2.06%
transfer_no_resume_ring_1.3_ecdsap256_aes 6.33 ms 6.34 ms 0.01 ms (0.23%) 3.49%
handshake_tickets_aws_lc_rs_1.2_rsa_aes 1.77 ms 1.77 ms -0.00 ms (-0.20%) 3.02%
handshake_tickets_ring_1.2_rsa_aes 1.61 ms 1.60 ms -0.00 ms (-0.20%) 1.07%
handshake_no_resume_aws_lc_rs_1.3_rsa_aes 1.09 ms 1.09 ms -0.00 ms (-0.19%) 2.33%
handshake_session_id_ring_1.2_rsa_aes 1.52 ms 1.52 ms -0.00 ms (-0.18%) 1.00%
handshake_tickets_ring_1.3_ecdsap256_aes 6.45 ms 6.44 ms -0.01 ms (-0.18%) 1.00%
handshake_no_resume_aws_lc_rs_1.3_rsa_chacha 1.08 ms 1.08 ms -0.00 ms (-0.17%) 6.60%
handshake_session_id_ring_1.3_ecdsap256_chacha 6.33 ms 6.32 ms -0.01 ms (-0.17%) 1.00%
handshake_tickets_ring_1.3_ecdsap256_chacha 6.40 ms 6.39 ms -0.01 ms (-0.16%) 1.00%
transfer_no_resume_aws_lc_rs_1.3_ecdsap384_chacha 13.67 ms 13.69 ms 0.02 ms (0.16%) 1.63%
transfer_no_resume_aws_lc_rs_1.3_ecdsap384_aes 5.21 ms 5.22 ms 0.01 ms (0.15%) 3.84%
handshake_no_resume_aws_lc_rs_1.3_ecdsap256_aes 456.48 µs 455.90 µs -0.58 µs (-0.13%) 3.01%
transfer_no_resume_aws_lc_rs_1.3_ecdsap256_aes 4.47 ms 4.48 ms 0.01 ms (0.12%) 4.43%
handshake_session_id_ring_1.3_ecdsap256_aes 6.38 ms 6.37 ms -0.01 ms (-0.12%) 1.00%
handshake_tickets_ring_1.3_ecdsap384_aes 9.54 ms 9.53 ms -0.01 ms (-0.11%) 1.00%
transfer_no_resume_ring_1.3_ecdsap384_aes 9.47 ms 9.48 ms 0.01 ms (0.10%) 2.11%
handshake_tickets_ring_1.3_rsa_chacha 6.89 ms 6.89 ms -0.01 ms (-0.09%) 1.00%
handshake_tickets_ring_1.3_rsa_aes 6.95 ms 6.94 ms -0.01 ms (-0.09%) 1.00%
handshake_tickets_ring_1.3_ecdsap384_chacha 9.49 ms 9.48 ms -0.01 ms (-0.09%) 1.00%
transfer_no_resume_ring_1.3_ecdsap256_chacha 12.97 ms 12.98 ms 0.01 ms (0.08%) 1.57%
handshake_session_id_aws_lc_rs_1.2_rsa_aes 1.60 ms 1.60 ms -0.00 ms (-0.08%) 2.12%
transfer_no_resume_ring_1.3_ecdsap384_chacha 16.07 ms 16.08 ms 0.01 ms (0.07%) 1.28%
transfer_no_resume_ring_1.3_rsa_chacha 13.46 ms 13.47 ms 0.01 ms (0.07%) 1.31%
transfer_no_resume_ring_1.2_rsa_aes 6.79 ms 6.79 ms 0.00 ms (0.07%) 2.84%
transfer_no_resume_aws_lc_rs_1.3_ecdsap256_chacha 12.94 ms 12.95 ms 0.01 ms (0.07%) 1.75%
handshake_no_resume_aws_lc_rs_1.2_rsa_aes 1.06 ms 1.06 ms -0.00 ms (-0.06%) 1.65%
transfer_no_resume_aws_lc_rs_1.3_rsa_chacha 13.59 ms 13.60 ms 0.01 ms (0.06%) 1.57%
transfer_no_resume_ring_1.3_rsa_aes 6.82 ms 6.83 ms 0.00 ms (0.06%) 3.15%
handshake_session_id_ring_1.3_rsa_aes 6.87 ms 6.87 ms -0.00 ms (-0.05%) 1.00%
handshake_no_resume_ring_1.3_rsa_chacha 994.02 µs 993.53 µs -0.49 µs (-0.05%) 1.00%
handshake_session_id_ring_1.3_ecdsap384_aes 9.46 ms 9.45 ms -0.00 ms (-0.05%) 1.00%
transfer_no_resume_aws_lc_rs_1.3_rsa_aes 5.12 ms 5.12 ms -0.00 ms (-0.05%) 3.68%
handshake_no_resume_ring_1.3_ecdsap384_aes 3.60 ms 3.60 ms 0.00 ms (0.05%) 1.00%
handshake_no_resume_ring_1.3_ecdsap256_chacha 499.54 µs 499.76 µs 0.22 µs (0.04%) 2.57%
handshake_no_resume_ring_1.3_ecdsap256_aes 502.86 µs 503.07 µs 0.21 µs (0.04%) 2.45%
handshake_no_resume_ring_1.2_rsa_aes 991.57 µs 991.88 µs 0.31 µs (0.03%) 1.00%
transfer_no_resume_aws_lc_rs_1.2_rsa_aes 5.13 ms 5.13 ms 0.00 ms (0.03%) 3.79%
handshake_session_id_ring_1.3_rsa_chacha 6.82 ms 6.82 ms -0.00 ms (-0.03%) 1.00%
handshake_session_id_ring_1.3_ecdsap384_chacha 9.41 ms 9.41 ms -0.00 ms (-0.03%) 1.00%
handshake_no_resume_aws_lc_rs_1.3_ecdsap384_chacha 1.18 ms 1.18 ms -0.00 ms (-0.02%) 1.00%
handshake_no_resume_aws_lc_rs_1.3_ecdsap384_aes 1.18 ms 1.18 ms -0.00 ms (-0.02%) 1.13%
handshake_no_resume_aws_lc_rs_1.3_ecdsap256_chacha 454.96 µs 455.03 µs 0.07 µs (0.01%) 3.13%
handshake_no_resume_ring_1.3_rsa_aes 995.03 µs 995.13 µs 0.10 µs (0.01%) 1.03%
handshake_no_resume_ring_1.3_ecdsap384_chacha 3.59 ms 3.59 ms -0.00 ms (-0.01%) 1.00%

Additional information

Historical results

Checkout details:

Copy link
codecov bot commented Nov 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.63%. Comparing base (d1c33f8) to head (3eaa70b).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2201   +/-   ##
=======================================
  Coverage   94.63%   94.63%           
=======================================
  Files         102      102           
  Lines       23928    23928           
=======================================
  Hits        22645    22645           
  Misses       1283     1283           

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

@djc
Copy link
Member
djc commented Nov 8, 2024

Did you consider picking up something like oranda? Seems like it might be a good fit.

@ctz
Copy link
Member Author
ctz commented Nov 8, 2024

I haven't heard of that, will look into it.

@ctz
Copy link
Member Author
ctz commented Nov 11, 2024

OK, I spent an hour or so with oranda. I like the way it can present github release notes, but unfortunately it only really supports a handful of built-in themes (and limited CSS customisation on top of that). I filed an issue about that, but in the meantime I think that possibility need not block this?

@djc
Copy link
Member
djc commented Nov 11, 2024

The Makefile and custom pulldown-cmark stuff seems a little ugly, I'd prefer doing something with Zola or maybe another Rust script that uses Askama and the pulldown-cmark library to do the same stuff.

Copy link
Member
@cpu cpu left a comment

Choose a reason for hiding this comment

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

Zola seems pretty heavy weight for this case and I think I'd prefer the Makefile personally. A Rust script seems OK.

@ctz ctz force-pushed the jbp-landing-page branch 3 times, most recently from e632875 to 1e5e4aa Compare November 11, 2024 17:00
@ctz ctz force-pushed the jbp-landing-page branch from 1e5e4aa to 3eaa70b Compare November 11, 2024 17:04
@ctz
Copy link
Member Author
ctz commented Nov 11, 2024

Zola seems to work pretty well, stays out the way, and has good docs. I'm minded to write the absolute minimum of code for this.

@cpu
Copy link
Member
cpu commented Nov 11, 2024

Zola seems to work pretty well, stays out the way, and has good docs. I'm minded to write the absolute minimum of code for this.

My enthusiasm for static site generators was reduced by the experience of Hugo and Jekyll. Maybe Zola will be better, but if it's like those two then there's a non-zero amount of headache related to keeping up to date with the tooling and associated breaking changes.

This branch is also hardcoding a Zola version in CI in a way that I think excludes Rennovate from helping us keep it in-sync with upstream releases. Is there a way to avoid that?

I don't feel strongly enough to block this getting merged but given there's only one page with no significant site structure to contend with it feels like dragging in a whole site generation framework is overkill.

@cpu
Copy link
Member
cpu commented Nov 11, 2024

This branch is also hardcoding a Zola version in CI in a way that I think excludes Rennovate from helping us keep it in-sync with upstream releases. Is there a way to avoid that?

I noticed Curl has some special markup in some of its CI YAML that looks like it configures Renovate to update versions, e.g.: https://github.com/curl/curl/blob/774844ab5db84e2368389e6cd9f0b8f81852b738/.github/workflows/linux.yml#L38-L57

That might be an option if we stick with Zola.

@ctz
Copy link
Member Author
ctz commented Nov 11, 2024

My enthusiasm for static site generators was reduced by the experience of Hugo and Jekyll. Maybe Zola will be better, but if it's like those two then there's a non-zero amount of headache related to keeping up to date with the tooling and associated breaking changes.

This branch is also hardcoding a Zola version in CI in a way that I think excludes Rennovate from helping us keep it in-sync with upstream releases. Is there a way to avoid that?

I think these two points kinda cancel each other out a bit? For the purposes of static site generation, I'm happy to fix on a working version until we have a reason to change (for my own blog, I basically did that with jekyll, only upgrading it 3 times in 11 years).

@cpu
Copy link
Member
cpu commented Nov 11, 2024

For the purposes of static site generation, I'm happy to fix on a working version until we have a reason to change

I feel like that just shifts the pain, collecting it all up at the point where you have to jump many versions at once. Put another way: if we adopt Zora we're on the hook for future update pain. We can either pay it in small increments to keep up with the tooling as it's released (generally easier IME but still annoying), or we can pay it in large lump sums when we're forced to for feature-needs or because the tooling stopped working on our systems.

We're using so few of the features of Zora that it seems excessive weighed against a small makefile directly invoking the one thing we do want, markdown -> HTML for ~a single file but if I'm in the minority here I'm happy to go along with the crowd.

@djc
Copy link
Member
djc commented Nov 11, 2024

I feel like that just shifts the pain, collecting it all up at the point where you have to jump many versions at once. Put another way: if we adopt Zora we're on the hook for future update pain.

I'm sympathetic and don't mind either route.

That said, I'm optimistic about the Zola route just based on the notion that the Rust culture around backwards compatibility is pretty good.

I think Joe will most likely end up being on the hook for maintenance so ultimately his call?

@ctz
Copy link
Member Author
ctz commented Nov 12, 2024

I think one of two things will happen:

  • either we use more of Zola's features, and therefore it was worth it, or
  • we don't, and fall-back to something simpler is pretty trivial if that becomes needed

@ctz ctz added this pull request to the merge queue Nov 12, 2024
Merged via the queue into main with commit b66efdd Nov 12, 2024
60 checks passed
@ctz ctz deleted the jbp-landing-page branch November 12, 2024 08:33
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.

3 participants
0