Conversation
|
I'll review and report back first thing tomorrow EU time. |
|
MR looks great, but with the following remarks:
|
|
LGTM! 👍 |
| USE_PCRE2=1 USE_PCRE2_JIT=1 \ | ||
| USE_PROMEX=1 \ | ||
| # https://github.com/haproxy/haproxy/blob/a203ce685464988afbd92c8a376fcafae9bab5b5/INSTALL#L368-L373 | ||
| USE_PTHREAD_EMULATION=1 \ |
There was a problem hiding this comment.
From that doc:
It should not be used by distro packagers as it is a bit less observable.
I'm guessing this concern doesn't really matter as much for consumers of this image?
(I admit I don't understand the option nor the effects, so advice is appreciated 😅)
There was a problem hiding this comment.
When this is enabled we replace pthread rwlocks (that OpenSSL uses) with our own lighter variant, but the eventual lock debugging would be more complicated (but that's rarely done by regular users, if ever). However the performance gain is quite visible for larger SMP systems and we in all our performance (CE and EE) packages enable USE_PTHREAD_EMULATION=1 by default. 🚀
I think this is safe to add -- and in fact I've reached to rest of our engineering team early on this morning and double checked if we all agree on enabling this for Docker images by default.
There was a problem hiding this comment.
Ah nice, thank you for the added detail!
LGTM 👍
Changes: - docker-library/haproxy@29f17a0: Merge pull request docker-library/haproxy#257 from infosiftr/quic - docker-library/haproxy@68b7118: Add pthread emulation to speed up openssl - docker-library/haproxy@54f063d: Add QUIC to 3.2+
Changes: - docker-library/haproxy@29f17a0: Merge pull request docker-library/haproxy#257 from infosiftr/quic - docker-library/haproxy@68b7118: Add pthread emulation to speed up openssl - docker-library/haproxy@54f063d: Add QUIC to 3.2+ Co-authored-by: Docker Library Bot <doi+docker-library-bot@docker.com>
With OpenSSL 3.5 in both Debian Trixie and Alpine 3.22+, it's finally here. Releases have aligned and we can now compile
haproxywith QUIC support enabled! 🎉 Thanks to all those in OpenSSL, Haproxy, and Debian that moved it all along. ❤️I don't think there are risks to adding this because users have to add it to their config to use it.
The image is just slightly bigger on disk and in the registry.
*-quictags are my local builds; there will not be separate tags for QUIC images since it will just be enabled in the current images (3.2 and above).Fixes #182
cc @dkorunic, ref #243; If you can give this a review, that would be helpful.