thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions 3

Immediately Hammer if second kill is sent (#18823) (#18826)

Browse source 
Backport #18823

Currently Gitea will wait for HammerTime or nice shutdown if kill -1 or kill -2
is sent. We should just immediately hammer if there is a second kill.

Signed-off-by: Andrew Thornton <art27@cantab.net>
This commit is contained in:
zeripath 2022-02-19 17:37:52 +00:00 committed by GitHub
parent fa9be55018
commit bec566282e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
modules/graceful/manager.go
View file

@ -192,6 +192,7 @@ func (g *Manager) RunAtHammer(hammer func()) {
} }
func (g *Manager) doShutdown() { func (g *Manager) doShutdown() {
if !g.setStateTransition(stateRunning, stateShuttingDown) { if !g.setStateTransition(stateRunning, stateShuttingDown) {
g.DoImmediateHammer()
return return
} }
g.lock.Lock() g.lock.Lock()

8
modules/graceful/manager_unix.go
View file

@ -168,8 +168,12 @@ func (g *Manager) DoGracefulRestart() {
if setting.GracefulRestartable { if setting.GracefulRestartable {
log.Info("PID: %d. Forking...", os.Getpid()) log.Info("PID: %d. Forking...", os.Getpid())
err := g.doFork() err := g.doFork()
if err != nil && err.Error() != "another process already forked. Ignoring this one" { if err != nil {
log.Error("Error whilst forking from PID: %d : %v", os.Getpid(), err) if err.Error() == "another process already forked. Ignoring this one" {
g.DoImmediateHammer()
} else {
log.Error("Error whilst forking from PID: %d : %v", os.Getpid(), err)
}
} }
} else { } else {
log.Info("PID: %d. Not set restartable. Shutting down...", os.Getpid()) log.Info("PID: %d. Not set restartable. Shutting down...", os.Getpid())