chore: adjust cleanup

Author: samrose

nix/tools/dbmate-tool.sh.in

@@ -27,23 +27,35 @@ start_postgres() {
 cleanup() {
     echo "Cleaning up..."
 
-    # Stop PostgreSQL processes gracefully
+    # Check if PostgreSQL processes exist
     if pgrep -f "postgres" >/dev/null; then
         echo "Stopping PostgreSQL gracefully..."
-        pkill -f "postgres" || true  # Adjust this if you have a specific way to stop the service
-        sleep 5  # Wait for PostgreSQL to shut down
+        
+        # Use a more specific signal handling approach
+        pkill -15 -f "postgres"  # Send SIGTERM
+        
+        # Wait a bit for graceful shutdown
+        sleep 5
+        
+        # If processes are still running, force kill
+        if pgrep -f "postgres" >/dev/null; then
+            echo "Forcing PostgreSQL processes to stop..."
+            pkill -9 -f "postgres"  # Send SIGKILL if SIGTERM didn't work
+        fi
     else
         echo "PostgreSQL is not running, skipping stop."
     fi
 
-    # Verify cleanup
+    # Always exit successfully, log any remaining processes
     if pgrep -f "postgres" >/dev/null; then
         echo "Warning: Some PostgreSQL processes could not be cleaned up:"
         pgrep -f "postgres"
-        exit 1  # Exit with an error code
     else
         echo "Cleanup completed successfully"
     fi
+
+    # Explicitly exit with 0 to prevent workflow failure
+    exit 0
 }