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

No graceful shutdown when connected to website #1028

Closed
5 tasks done
lambdareader opened this issue Dec 27, 2022 · 3 comments
Closed
5 tasks done

No graceful shutdown when connected to website #1028

lambdareader opened this issue Dec 27, 2022 · 3 comments
Labels
bug Something isn't working released

Comments

@lambdareader
Copy link

lambdareader commented Dec 27, 2022

Steps to reproduce

1: log-in on website
2: shut down server via SIGTERM or SIGINT

Expected behavior

graceful shutdown even with active requests perhaps?

Actual behavior

graceful shutdown aborted

Logs

2022-12-27 16:52:56.921  INFO 25786 --- [SpringApplicationShutdownHook] o.s.b.w.e.tomcat.GracefulShutdown        : Commencing graceful shutdown. Waiting for active requests to complete
2022-12-27 16:53:26.926  INFO 25786 --- [SpringApplicationShutdownHook] o.s.c.support.DefaultLifecycleProcessor  : Failed to shut down 1 bean with phase value 2147483647 within timeout of 30000ms: [webServerGracefulShutdown]
2022-12-27 16:53:26.969  INFO 25786 --- [tomcat-shutdown] o.s.b.w.e.tomcat.GracefulShutdown        : Graceful shutdown aborted with one or more requests still active
2022-12-27 16:53:28.949  INFO 25786 --- [SpringApplicationShutdownHook] io.hawt.web.auth.AuthenticationFilter    : Destroying hawtio authentication filter
2022-12-27 16:53:28.951  INFO 25786 --- [SpringApplicationShutdownHook] io.hawt.HawtioContextListener            : Destroying hawtio services
2022-12-27 16:53:29.321  INFO 25786 --- [SpringApplicationShutdownHook] org.apache.activemq.artemis.core.server  : AMQ221002: Apache ActiveMQ Artemis Message Broker version 2.19.1 [e181f9df-756d-11ed-84c0-dca63217cef3] stopped, uptime 2 minutes
2022-12-27 16:53:29.330  INFO 25786 --- [SpringApplicationShutdownHook] com.zaxxer.hikari.HikariDataSource       : SqliteUdfPool - Shutdown initiated...
2022-12-27 16:53:29.338  INFO 25786 --- [SpringApplicationShutdownHook] com.zaxxer.hikari.HikariDataSource       : SqliteUdfPool - Shutdown completed.

Komga version

v0.157.5

Operating system

Linux

Other details

I am not too sure if it is a problem at all. Most likely it isn't just wanted to know if it is intended behavior or if it causes any kind of issues.

Additionally could a SIGKILL / forceful server shutdown / power outage mid scan corrupt the database? Should I keep versioned backups?

Acknowledgements

  • I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
  • I have written a short but informative title.
  • I have checked the FAQ.
  • I have updated the app to the latest version.
  • I will fill out all of the requested information in this form.
@gotson
Copy link
Owner

gotson commented Jan 2, 2023

I don't understand what's your issue? The server stopped according to the logs.

@gotson gotson added waiting for feedback Waiting for a feedback from the issue creator bug Something isn't working and removed waiting for feedback Waiting for a feedback from the issue creator triage labels Jan 3, 2023
@gotson
Copy link
Owner

gotson commented Jan 4, 2023

I checked this, and SEE connections are not getting closed during the graceful shutdown phase. Other connections would still terminate gracefully, but if a browser session is open then the graceful shutdown would always timeout after the default 30s.

@gotson gotson closed this as completed in 3f773d1 Jan 4, 2023
github-actions bot pushed a commit that referenced this issue Jan 12, 2023
# [0.158.0](v0.157.5...v0.158.0) (2023-01-12)

### Bug Fixes

* close SSE connections during shutdown ([3f773d1](3f773d1)), closes [#1028](#1028)
* translated using Weblate  ([f06c680](f06c680))

### Features

* library option to append ComicInfo Volume to Series title ([63b3c83](63b3c83))
* **webui:** navigate back to collection from series ([db56a38](db56a38)), closes [#1042](#1042)
* **webui:** order collections/readlists by most recently modified in the Add To dialog ([592f87c](592f87c)), closes [#862](#862)
@github-actions
Copy link
Contributor

🎉 This issue has been resolved in version 0.158.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working released
Projects
None yet
Development

No branches or pull requests

2 participants