SIGKILL used instead of SIGTERM when force-killing backup process #5

New Issue

2025-11-15T03:44:09Z

snegov commented

2025-11-15 03:44:09 +00:00

SIGKILL used instead of SIGTERM when force-killing backup process

Priority: High
Component: backup.py
Type: Bug

Description

When the --force flag is used and a previous backup is still running, the code sends SIGKILL to terminate it. This doesn't allow the previous process to clean up resources, release locks, or remove incomplete backups.

Location

curateipsum/backup.py:128

Current Code

os.kill(pid, signal.SIGKILL)

Problem

SIGKILL (signal 9) cannot be caught or ignored by the process, preventing graceful cleanup. This could leave:

Incomplete backup directories
Unclosed file descriptors
Corrupted backup metadata

Proposed Solution

Send SIGTERM first, wait briefly, then escalate to SIGKILL if needed:

_lg.warning("Sending SIGTERM to process %d", pid)
os.kill(pid, signal.SIGTERM)
time.sleep(5)
if _pid_exists(pid):
    _lg.warning("Process %d didn't terminate, sending SIGKILL", pid)
    os.kill(pid, signal.SIGKILL)

Impact

High - Current behavior can leave backup directory in inconsistent state.

# SIGKILL used instead of SIGTERM when force-killing backup process **Priority:** High **Component:** backup.py **Type:** Bug ## Description When the `--force` flag is used and a previous backup is still running, the code sends `SIGKILL` to terminate it. This doesn't allow the previous process to clean up resources, release locks, or remove incomplete backups. ## Location `curateipsum/backup.py:128` ## Current Code ```python os.kill(pid, signal.SIGKILL) ``` ## Problem `SIGKILL` (signal 9) cannot be caught or ignored by the process, preventing graceful cleanup. This could leave: - Incomplete backup directories - Unclosed file descriptors - Corrupted backup metadata ## Proposed Solution Send `SIGTERM` first, wait briefly, then escalate to `SIGKILL` if needed: ```python _lg.warning("Sending SIGTERM to process %d", pid) os.kill(pid, signal.SIGTERM) time.sleep(5) if _pid_exists(pid): _lg.warning("Process %d didn't terminate, sending SIGKILL", pid) os.kill(pid, signal.SIGKILL) ``` ## Impact **High** - Current behavior can leave backup directory in inconsistent state.

snegov referenced this issue from a commit

2026-02-04 06:07:44 +00:00

Fix high-priority bugs and add comprehensive test coverage

snegov referenced this issue from a commit

2026-02-04 06:09:55 +00:00

Fix high-priority bugs and add comprehensive test coverage

snegov closed this issue

2026-02-04 06:09:55 +00:00

snegov referenced this issue from a commit

2026-10-11 18:58:10 +00:00

Make snapshot creation transactional (#5)

Sign in to join this conversation.