Skip to content

Adjusted database stop code to give more time to smart shutdown #1670

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

craigcomstock
Copy link
Contributor

@craigcomstock craigcomstock commented Apr 10, 2025

On a slow raspberry pi 4 I found that the current smart and then immediate stop (immediate) did not work well.
The system needed more time.

Ticket: ENT-12750
Changelog: title

Build Status

On a slow raspberry pi 4 I found that the current smart and then immediate stop (immediate) did not work well.
The system needed more time.

Ticket: ENT-12750
Changelog: title
@craigcomstock
Copy link
Contributor Author

Looking good. It takes about 4 minutes to shutdown on my rpi4 and I have given the timeout to -m smart as 300 seconds aka 5 minutes. Just enough to squeak by for another year? ;)

@craigcomstock craigcomstock requested review from nickanderson, aleksandrychev and olehermanse and removed request for nickanderson April 11, 2025 17:48
@craigcomstock
Copy link
Contributor Author

@cf-bottom jenkins please

@craigcomstock craigcomstock marked this pull request as ready for review April 11, 2025 17:49
@cf-bottom
Copy link

Copy link
Member

@olehermanse olehermanse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but makes sense to wait for Jenkins

@craigcomstock craigcomstock marked this pull request as draft April 17, 2025 13:19
@craigcomstock craigcomstock added the after-release don't merge this until after the current set of releases label Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
after-release don't merge this until after the current set of releases
Development

Successfully merging this pull request may close these issues.

4 participants