Skip to content
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

healthcheck high load #43

Closed
visibilityspots opened this issue Jul 4, 2020 · 11 comments
Closed

healthcheck high load #43

visibilityspots opened this issue Jul 4, 2020 · 11 comments
Assignees

Comments

@visibilityspots
Copy link
Contributor

I had to force my nomad based piaware docker setup down to the image with label 3.8.0 due to the health check in version 3.8.1. The node it was running on got a really high load making it unstable.

I'll dig into it once I find some time to debug the issue but thought I already give a headsup.

@mikenye
Copy link
Member

mikenye commented Jul 5, 2020

Hi Jan,

Sorry this is causing you problems.

You can just disable the health check, this might be a better option for you.

With docker run add --no-healthcheck.

With docker-compose add the following to the piaware service:

healthcheck:
      disable: true

Keep me posted!

@mikenye mikenye self-assigned this Jul 5, 2020
@mikenye
Copy link
Member

mikenye commented Jul 8, 2020

Hi @visibilityspots,

Any updates?

@visibilityspots
Copy link
Contributor Author

Finally found some time to further investigate the issue. It's definitely the healthcheck which is causing the high load every 30s as you can see in the image:
healthcheck
The peaks are topping over 600Mhz during the healthcheck..

Unfortunately nomad doesn't support the disabling of the HEALTHCHECK from a docker image.. So or I have to stick with 3.8.0 or I have to rebuild a custom image based on yours without the healthcheck :(

Besides the fact nomad can't disable health checks I'm not sure if a healthcheck which causes such a higher load compared to the actual service is advisable?

@mikenye
Copy link
Member

mikenye commented Jul 23, 2020

Yes, this is not ideal. I will rewrite the healthcheck.

In the meantime, I've created you mikenye/piaware:latest_no_healthcheck and mikenye/piaware:v3.8.1_no_healthcheck.

Feel free to use these. Once I've fixed the healthcheck I'll get you to test and revert to the normal stream if that's OK.

Thanks.

@mikenye
Copy link
Member

mikenye commented Jul 23, 2020

Hi @visibilityspots, can you pull and try out mikenye/piaware:development? This contains a reworked healthcheck using bash instead of python.

If this solves your problem, I'll merge into master.

Thanks.

@visibilityspots
Copy link
Contributor Author

Hi,

I tried the development tagged image but the issue still pops up now only every 300s :)
flightaware-peaks

@mikenye
Copy link
Member

mikenye commented Jul 27, 2020

Annoying. I hoped that moving the healthcheck from python to bash would alleviate the problem.

Looking at the healthcheck.sh script, I can't figure out what could be contributing to the high load. My best guess is running piaware-status.

Are you able to run piaware-status manually to see whether this causes a CPU spike?

@mikenye
Copy link
Member

mikenye commented Jul 28, 2020

Would you like me to maintain an additional "latest_nohealthcheck" version of the image?

@visibilityspots
Copy link
Contributor Author

if that's possible that would be great to keep the noheathcheck version alive. I double checked with the piaware-status but that particular command didn't really spiked the CPU...

@mikenye
Copy link
Member

mikenye commented Aug 3, 2020

Hi @visibilityspots,

It would be my pleasure. I'll update my nightly build scripts to build the nohealthcheck version nightly.

mikenye added a commit that referenced this issue Aug 3, 2020
…tagged images as-per issue #43 (#50)

* Rewrite healthcheck
* Implement "_nohealthcheck" tags
@mikenye
Copy link
Member

mikenye commented Aug 3, 2020

Hi @visibilityspots,

This has been completed. I changed the name of the tag to _nohealthcheck, so you should be able to pull latest_nohealthcheck, and this will be built nightly from master branch.

Cheers!

@mikenye mikenye closed this as completed Aug 3, 2020
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

No branches or pull requests

2 participants