linuxserver / docker-smokeping

GNU General Public License v3.0
346 stars 95 forks source link

Revert to apache #141

Closed thespad closed 1 year ago

thespad commented 1 year ago

linuxserver.io



Description:

Latency with nginx/fcgiwrap is unacceptably high and so far I haven't found a way to improve it.

Benefits of this PR and context:

How Has This Been Tested?

Source / References:

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/smokeping/2.7.3-r9-pkg-117e2b0a-pr-141/index.html https://ci-tests.linuxserver.io/lspipepr/smokeping/2.7.3-r9-pkg-117e2b0a-pr-141/shellcheck-result.xml

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/smokeping/2.7.3-r9-pkg-117e2b0a-pr-141/index.html https://ci-tests.linuxserver.io/lspipepr/smokeping/2.7.3-r9-pkg-117e2b0a-pr-141/shellcheck-result.xml

nemchik commented 1 year ago

Just for fun, I believe this is a comparison of this PR against the repo prior to #136 https://github.com/linuxserver/docker-smokeping/compare/9d5800eb4e7db5551d1c6d6d52ecbf7548928d66...94d2c1250280cead3805f4b1049b73db307b2cb6

nemchik commented 1 year ago

Arm (32 and 64) CI report looks broken.

thespad commented 1 year ago

Arm (32 and 64) CI report looks broken.

Looks like a qemu issue, aarch64 runs fine natively.