[serve] Fix a potential UnboundLocalError in `ActorReplicaWrapper.check_stopped()

[chenshi5012]

Description

Fix a potential UnboundLocalError in ActorReplicaWrapper.check_stopped() (python/ray/serve/_private/deployment_state.py).

Related issues

none

Additional information

The fix is a single-line defensive initialization. It does not change any behavior in the normal code path — stopped will still be correctly set to True by either the try body or the except ValueError handler when those paths execute successfully.

def check_stopped(self) -> bool:
    """Check if the actor has exited."""
    stopped = False  # Ensure 'stopped' is always defined for finally/return
    try:
        handle = ray.get_actor(self._actor_name, namespace=SERVE_NAMESPACE)
        stopped = check_obj_ref_ready_nowait(self._graceful_shutdown_ref)
        ...
    except ValueError:
        stopped = True
    finally:
        if stopped and self._placement_group is not None:
            ...
    return stopped

[gemini-code-assist]

Code Review

This pull request initializes the "stopped" variable in the check_stopped method to prevent an UnboundLocalError. The reviewer identified a potential robustness issue where self._graceful_shutdown_ref is not checked for None before being passed to check_obj_ref_ready_nowait, which could cause a TypeError and crash the controller loop.

[abrarsheikh]

thank you