Skip to content

Conversation

@Pijukatel
Copy link
Contributor

Description

  • Add ResponseWatcher protocol.
  • Add response_watcher arguments to wait_for_finish methods.
  • wait_for_finish uses an optional response_watcher callable each time it processes the API response.

Previously, StatusWatcher was periodically calling RunClient.get to get the status and status message of another Actor run. Since ActorClient.call is already periodically calling RunClient.get through RunClient.wait_for_finish, there is no need for StatusWatcher to do the API calls again; it just needs to re-use the responses of already available in RunClient.wait_for_finish. It can do so, by passing the response_watcher argument to wait_for_finish.

@github-actions github-actions bot added this to the 126th sprint - Tooling team milestone Oct 30, 2025
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 30, 2025
@Pijukatel Pijukatel closed this Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants