Skip to content

Move WorkerPoolSyncer to ate-controller - #1203

Merged
Julian Gutierrez Oschmann (juli4n) merged 2 commits into
agent-substrate:mainfrom
sfunkenhauser:worker_pool_syncer_migrate
Aug 26, 2026
Merged

Move WorkerPoolSyncer to ate-controller#1203
Julian Gutierrez Oschmann (juli4n) merged 2 commits into
agent-substrate:mainfrom
sfunkenhauser:worker_pool_syncer_migrate

Conversation

@sfunkenhauser

Copy link
Copy Markdown
Collaborator

Fixes #729

  • [ x ] Tests pass
  • [ x ] Appropriate changes to documentation are included in the PR

@sfunkenhauser
sfunkenhauser force-pushed the worker_pool_syncer_migrate branch 3 times, most recently from fe23b1a to a599098 Compare August 25, 2026 19:36
@sfunkenhauser
sfunkenhauser marked this pull request as ready for review August 25, 2026 20:25
//
// A concurrent SuspendActor or ResumeActor wins the optimistic version check;
// this attempt fails as ABORTED so the caller retries against the newer state.
func (s *RPCService) releaseActorOnDeadWorker(ctx context.Context, worker *ateapipb.Worker) error {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we move this into a DeleteWorker workflow? That would be consistent with other RPC handlers that need to orchestrate an idempotent sequence of steps.

@sfunkenhauser
sfunkenhauser force-pushed the worker_pool_syncer_migrate branch from a599098 to 68deb82 Compare August 26, 2026 14:52
@sfunkenhauser
sfunkenhauser force-pushed the worker_pool_syncer_migrate branch from 68deb82 to 08c59b0 Compare August 26, 2026 15:11
@juli4n
Julian Gutierrez Oschmann (juli4n) merged commit 37735a6 into agent-substrate:main Aug 26, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move WorkerPoolSyncer out of api-server

2 participants