-
Notifications
You must be signed in to change notification settings - Fork 94
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
Tested for concurrency? #11
Comments
Testing the returned content in PDF is a PITA. Any ideas on how a test with concurrency could be done ? |
I wrote a perl script (call it '
Comparison done through this pipleline:
Then I wrote another perl script (call it ' |
@dtoso-skymesh 👍 thank you |
We are going to use this on enterprise scale which will perform 100 req/s on each server. Is the issue still exists? Does any alternative solution available? |
@alicpr not sure if @sharoonthomas has fixed this issue, but I worked around it by running many docker containers each running If you've got a fast enough machine(s) you could just launch ( On our hardware that wasn't fast enough so I came up with an HTTP-proxy based solution.
I've found the limiting factor to be the server hardware. |
In the past running multiple copies of
wkhtmltopdf
concurrently had issues; there were threading problems and some named-pipes were at the same fs path across multiplewkhtmltopdf
processes. (I found out after an ugly incident involving customers getting other's PDFs in a parallelized batch run).wkhtmltopdf
seems to have had many version bumps since then, but nothing I've read from the commits screams out that this issue has been fixed.In
openlabs/docker-wkhtmltopdf-aas
thegunicorn
WSGI daemon seems to fork on request, so if the concurrency issue still exists inwkhtmltopdf
then this service exports the problem to the service's users.In my use case, I needed to substitute the version of
wkhtmltopdf
shipped withopenlabs/docker-wkhtmltopdf-aas
with a staticly linked copy ofwkhtmltopdf 0.10.0rc2
because the PDF output from identical HTML had changed over the years due to webkit html rendering fixes. (I have legacy HTML that would be a massive PITA to change).As I know at least my version of wkhtmltopdf (0.10.0rc2) has concurrency issues, I'm treating docker as an isolation mechanism rather than simply a deployment helper. I have 20 identical containers running with a home-made HTTP load-balancing proxy sitting in front of them. It hands off (unmodifed) requests to available containers and makes subsequent requests wait until workers become available (by simply blocking on the HTTP response).
The text was updated successfully, but these errors were encountered: