Skip to content

Add HAProxy #148

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
Mar 13, 2025
Merged

Add HAProxy #148

merged 4 commits into from
Mar 13, 2025

Conversation

Acconut
Copy link
Contributor

@Acconut Acconut commented Mar 12, 2025

This PR adds HAProxy to the list of tested proxies. I followed all steps in https://github.com/http-tests/cache-tests/tree/main/docker#docker-image-for-proxy-caches, but please let me know if additional modifications are necessary. ./test-docker.sh haproxy works and produces the included results.

The cache configuration for HAProxy has been taken from https://www.haproxy.com/documentation/haproxy-configuration-tutorials/network-performance/caching/ without additional modifications or optimization. So there might be additional settings that could make more tests pass, but I haven't look into those.

For reference, the HAProxy version installed in the Docker image is 3.0.8-1ubuntu1.

@mnot
Copy link
Collaborator

mnot commented Mar 13, 2025

Looks good - I think the only thing missing is an entry in results/index.mjs

@Acconut
Copy link
Contributor Author

Acconut commented Mar 13, 2025

Good call! I updated the file and it now renders when viewing the web page:

image

@mnot mnot merged commit ac21ca9 into http-tests:main Mar 13, 2025
@mnot
Copy link
Collaborator

mnot commented Mar 13, 2025

Thanks!

@Acconut Acconut deleted the haproxy branch March 13, 2025 07:29
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