Skip to content

fix(gservice): remove unused PoolAgentDriver universal agent - #488

Closed
VasilyStepanov wants to merge 1 commit into
feat/machine-pool-migrate-spec-to-kindfrom
del/poolagentdriver
Closed

fix(gservice): remove unused PoolAgentDriver universal agent#488
VasilyStepanov wants to merge 1 commit into
feat/machine-pool-migrate-spec-to-kindfrom
del/poolagentdriver

Conversation

@VasilyStepanov

Copy link
Copy Markdown
Contributor

The machine_pool_agent service wired PoolAgentDriver into a UniversalAgentService but nothing consumed its capability, and no docs or tests reference it, so drop the dead wiring from GeneralService.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request removes the machine_pool_agent initialization, its driver import, and its registration from the service initialization in exordos_core/gservice/service.py. There are no review comments, so we have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

The machine_pool_agent service wired PoolAgentDriver into a
UniversalAgentService but nothing consumed its capability, and no
docs or tests reference it, so drop the dead wiring from
GeneralService.
@VasilyStepanov VasilyStepanov self-assigned this Jul 16, 2026
@VasilyStepanov
VasilyStepanov changed the base branch from master to feat/machine-pool-migrate-spec-to-kind July 16, 2026 14:18
@VasilyStepanov

Copy link
Copy Markdown
Contributor Author

Will be merged in #494

@VasilyStepanov
VasilyStepanov deleted the del/poolagentdriver branch July 17, 2026 16:04
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.

1 participant