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.

Sign up for GitHub

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

Jump to bottom

Docker Container says unhealthy but logs show nothing #89

Closed
dudefoxlive opened this issue May 4, 2023 · 4 comments · Fixed by #90
Closed

Docker Container says unhealthy but logs show nothing #89

dudefoxlive opened this issue May 4, 2023 · 4 comments · Fixed by #90
Assignees
@simonmicro
Labels
bug Something isn't working confirmed Okay, we can reproduce it!

Comments

@dudefoxlive
Copy link

Recently I have noticed that in portainer I see that my Py-KMS is showing as unhealthy. I look in the logs and see that its blank. Any reason as to why this is the case?
image
@didotb
Copy link

didotb commented May 5, 2023

restart the container, and while it's starting check the console outputs those could have something as well

@simonmicro
Copy link

@dudefoxlive Would you be able to also share your used environment variables? Maybe you've enabled a feature with a "feature"...

@simonmicro
Copy link

Also setting LOGLEVEL to e.g. DEBUG could help.

@simonmicro
Copy link

Nah, got it! With the new dual-stack default of :: the healthcheck for native Docker was broken...

simonmicro added a commit that referenced this issue May 6, 2023
@simonmicro
Fixed healthcheck for classic (non Kubernetes) Docker (broken with de…
474c5fe 
…fault-ipv6 "::" listen ip) #89

Added multi-ip healthcheck support
Cleanup of logging inside container-scripts
Removed "nc" from containers
@simonmicro simonmicro mentioned this issue May 6, 2023
Merged
@simonmicro simonmicro self-assigned this May 6, 2023
@simonmicro simonmicro added bug Something isn't working confirmed Okay, we can reproduce it! labels May 6, 2023
simonmicro added a commit that referenced this issue May 7, 2023
@simonmicro
Fixed healthcheck for classic (non Kubernetes) Docker (broken with de…
bc98019 
…fault-ipv6 "::" listen ip) #89

Added multi-ip healthcheck support
Cleanup of logging inside container-scripts
Removed "nc" from containers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simonmicro simonmicro

Labels
bug Something isn't working confirmed Okay, we can reproduce it!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Next Release 🎉 Py-KMS-Organization/py-kms
3 participants
@didotb @simonmicro @dudefoxlive