Skip to content

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

Merged
craigcomstock merged 1 commit intocfengine:masterfrom
craigcomstock:ENT-12750/master
Nov 4, 2025
Merged

Adjusted database stop code to give more time to smart shutdown#1670
craigcomstock merged 1 commit intocfengine:masterfrom
craigcomstock:ENT-12750/master

Conversation

@craigcomstock
Copy link
Copy Markdown
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
Copy Markdown
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 aleksandrychev, nickanderson and olehermanse and removed request for nickanderson April 11, 2025 17:48
@craigcomstock
Copy link
Copy Markdown
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 Markdown

Copy link
Copy Markdown
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
@craigcomstock
Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins please

@cf-bottom
Copy link
Copy Markdown

@craigcomstock craigcomstock marked this pull request as ready for review November 4, 2025 13:51
@craigcomstock craigcomstock merged commit 13e7055 into cfengine:master Nov 4, 2025
29 checks passed
@craigcomstock craigcomstock deleted the ENT-12750/master branch November 4, 2025 13:54
@craigcomstock craigcomstock added cherry-pick? after-release don't merge this until after the current set of releases and removed after-release don't merge this until after the current set of releases labels Nov 4, 2025
@craigcomstock craigcomstock restored the ENT-12750/master branch November 4, 2025 14:14
@craigcomstock
Copy link
Copy Markdown
Contributor Author

cherry picks: #2194 and #2193

@craigcomstock craigcomstock deleted the ENT-12750/master branch April 22, 2026 15:31
@craigcomstock craigcomstock removed cherry-pick? after-release don't merge this until after the current set of releases labels Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants