Skip to content

Add haproxy to benchmarking tool#52

Closed
jogme wants to merge 3 commits into
openssl:mainfrom
jogme:haproxy_bench
Closed

Add haproxy to benchmarking tool#52
jogme wants to merge 3 commits into
openssl:mainfrom
jogme:haproxy_bench

Conversation

@jogme

@jogme jogme commented Oct 15, 2025

Copy link
Copy Markdown
Contributor

Adding as a proxy - a middle actor.
Currently adding only for apache.

Support all the encryption modes for haproxy:

  • client to proxy
  • proxy to server
  • client to server
  • pure http without encryption

Currently adding it only for openssl-master crypto backend.

@jogme jogme closed this Oct 15, 2025
@jogme jogme reopened this Oct 15, 2025
@jogme

jogme commented Oct 15, 2025

Copy link
Copy Markdown
Contributor Author

Adding the benchmark results with 10s time for siege. The graph is unreadable, but I don't intend to fix that with this PR.
After boringssl: haproxy-both, haproxy-client, haproxy-no-ssl, haproxy-server, libressl-4.1.0

transactions transaction_rate throughput response_time data_transferred concurrency

@jogme jogme linked an issue Oct 15, 2025 that may be closed by this pull request
@jogme jogme requested a review from Sashan October 15, 2025 13:52
Comment thread bench-scripts/apache_bench.sh Outdated
Comment thread bench-scripts/apache_bench.sh Outdated
Comment thread bench-scripts/haproxy_bench.sh
Comment thread bench-scripts/apache_bench.sh
Comment thread bench-scripts/apache_bench.sh Outdated
jogme and others added 3 commits October 21, 2025 01:29
Adding as a proxy - a middle actor.
Currently adding only for apache.

Support all the encryption modes for haproxy:
- client to proxy
- proxy to server
- client to server
- pure http without encryption

Signed-off-by: Norbert Pocs <norbertp@openssl.org>
…obal state gets polluted. missing typeset in generate_download_files() prevents haproxy to build

@Sashan Sashan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

newly added versions for 3-0, 3-1, etc uncovered existing bugs, the forloop counter i also requires typeset. I've just added those.

I also added a make clean before we start building HA-proxy to cleanup files from previous build for different version.

@jogme

jogme commented Oct 21, 2025

Copy link
Copy Markdown
Contributor Author

Thanks!

@Sashan Sashan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

all looks good. thanks.

@github-project-automation github-project-automation Bot moved this from Waiting Review to Waiting Merge in Development Board Oct 21, 2025
@Sashan Sashan added the approval: done This pull request has the required number of approvals label Oct 21, 2025
Sashan pushed a commit that referenced this pull request Oct 21, 2025
Adding as a proxy - a middle actor.
Currently adding only for apache.

Support all the encryption modes for haproxy:
- client to proxy
- proxy to server
- client to server
- pure http without encryption

Signed-off-by: Norbert Pocs <norbertp@openssl.org>

Reviewed-by: Saša Nedvědický <sashan@openssl.org>
(Merged from #52)
@Sashan

Sashan commented Oct 21, 2025

Copy link
Copy Markdown
Contributor

changes are in. (via e17d5fb)

@Sashan Sashan closed this Oct 21, 2025
@github-project-automation github-project-automation Bot moved this from Waiting Merge to Done in Development Board Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add HAPROXY to tool set

4 participants