Three-step postgres exit may not complete

[e-gineer]

Description

The doThreeStepPostgresExit() function attempts graceful shutdown with SIGTERM, then SIGINT, then SIGQUIT, with timeouts at each step. However, if all three signals fail (process is hung or unresponsive), the function returns an error but the process may still be running. The function correctly avoids SIGKILL per Postgres documentation, but this means truly stuck processes may not be cleaned up.

Severity

MEDIUM - Resource leak (stuck process)

Test Reference

Test: TestDoThreeStepPostgresExit_Timeout in pkg/db/db_local/stop_services_test.go:117 (skipped)

Suggested Fix

Document this behavior clearly and potentially add monitoring/alerting for cases where postgres doesn't stop.

Related Code

• pkg/db/db_local/stop_services.go:252-295

[github-actions]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

[github-actions]

This issue was closed because it has been stalled for 90 days with no activity.