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

force kill wsgidav #6615

Closed
wants to merge 1 commit into from
Closed

force kill wsgidav #6615

wants to merge 1 commit into from

Conversation

imwhatiam
Copy link
Member

No description provided.

@laur89
Copy link
Contributor

laur89 commented Oct 11, 2024

Why do we even need to kill wsgidav from seafile.sh? The script already sends sigterm to controller:

pkill -SIGTERM -f "seafile-controller -c ${default_ccnet_conf_dir}"

which in turn causes controller to send sigkill to webdav.

If anything, we should fix how controller terminates webdav.

@freeplant
Copy link
Member

Sometimes the wsgidav cannot be shutdown correctly. Anyway, we will move the control part of wsgidav to monitor.sh instead of seafile-controller in version 12. We will recheck the issue later.

@freeplant freeplant closed this Oct 12, 2024
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 this pull request may close these issues.

3 participants