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

Container does not terminate on host reboot/shutdown #10

Closed
logopk opened this issue Mar 26, 2022 · 2 comments · Fixed by #25
Closed

Container does not terminate on host reboot/shutdown #10

logopk opened this issue Mar 26, 2022 · 2 comments · Fixed by #25

Comments

@logopk
Copy link

logopk commented Mar 26, 2022

Apparently clamav-prometheus-exporter does not react properly to kill -SIGTERM

So far with old containerd < v1.5 this did not matter, as containerd did not react properly on reboot when used in docker v20.10.x with live-restore. (containerd/containerd#5502)

Now that this is fixed, clamav-prometheus-exporter is the last container hanging.

When I enter the container and send kill -SIGTERM nothing happens.

Can you try and clarify, if that is something I can fix on my side?

Thank You!

@logopk
Copy link
Author

logopk commented Mar 27, 2022

I have to say that I run this with amd64 emulation via qemu on an mac m1 pro

▶ docker exec -ti clamav-exporter sh
/bin $ ps -aef
PID   USER     TIME  COMMAND
    1 promethe  0:00 {clamav-promethe} /usr/bin/qemu-x86_64 /bin/clamav-prometheus-exporter --clamav-address=clamav
   16 promethe  0:00 {sh} /usr/bin/qemu-x86_64 /bin/sh
   23 promethe  0:00 /bin/ps -aef

@logopk
Copy link
Author

logopk commented Mar 27, 2022

The host on the console logs

systemd-shutdown[1]: Waiting for process: clamav-promethe

r-tamura-omniment added a commit to Omniment-public/RECore-Fusion_sys that referenced this issue May 8, 2022
[fix] wlanのchannnel検索が正常に動作しないのを修正
[fix] 終了、再起動がハングするバグを踏んだので問題を修正 参照:r3kzi/clamav-prometheus-exporter#10
@r3kzi r3kzi closed this as completed in #25 Jan 27, 2025
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 a pull request may close this issue.

1 participant