[PR #507] [MERGED] chore: remove orchestrators #509

Closed
opened 2026-02-16 02:15:24 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_deploy/pull/507
Author: @masci
Created: 5/22/2025
Status: Merged
Merged: 5/22/2025
Merged by: @masci

Base: mainHead: massi/orchestrators


📝 Commits (2)

📊 Changes

12 files changed (+52 additions, -242 deletions)

View changed files

📝 llama_deploy/__init__.py (+0 -4)
📝 llama_deploy/apiserver/deployment.py (+1 -5)
📝 llama_deploy/control_plane/server.py (+49 -10)
📝 llama_deploy/control_plane/utils.py (+0 -0)
📝 llama_deploy/deploy/deploy.py (+1 -9)
llama_deploy/orchestrators/__init__.py (+0 -11)
llama_deploy/orchestrators/base.py (+0 -29)
llama_deploy/orchestrators/simple.py (+0 -96)
📝 tests/control_plane/test_server.py (+0 -1)
📝 tests/control_plane/test_utils.py (+1 -1)
tests/orchestrators/__init__.py (+0 -0)
tests/orchestrators/test_simple_orchestrator.py (+0 -76)

📄 Description

Remove the orchestrator abstraction and relative boilerplate. The two methods used are now moved to the ControlPlane class directly, which used to be the only client of that api.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/llama_deploy/pull/507 **Author:** [@masci](https://github.com/masci) **Created:** 5/22/2025 **Status:** ✅ Merged **Merged:** 5/22/2025 **Merged by:** [@masci](https://github.com/masci) **Base:** `main` ← **Head:** `massi/orchestrators` --- ### 📝 Commits (2) - [`9ccb64e`](https://github.com/run-llama/llama_deploy/commit/9ccb64eec073066de8a1fb28fa03b34d06282dd6) chore: remove orchestrators - [`41c9c30`](https://github.com/run-llama/llama_deploy/commit/41c9c30861bcc082db4afbf216626f15c47e99bd) fix get_next_messages ### 📊 Changes **12 files changed** (+52 additions, -242 deletions) <details> <summary>View changed files</summary> 📝 `llama_deploy/__init__.py` (+0 -4) 📝 `llama_deploy/apiserver/deployment.py` (+1 -5) 📝 `llama_deploy/control_plane/server.py` (+49 -10) 📝 `llama_deploy/control_plane/utils.py` (+0 -0) 📝 `llama_deploy/deploy/deploy.py` (+1 -9) ➖ `llama_deploy/orchestrators/__init__.py` (+0 -11) ➖ `llama_deploy/orchestrators/base.py` (+0 -29) ➖ `llama_deploy/orchestrators/simple.py` (+0 -96) 📝 `tests/control_plane/test_server.py` (+0 -1) 📝 `tests/control_plane/test_utils.py` (+1 -1) ➖ `tests/orchestrators/__init__.py` (+0 -0) ➖ `tests/orchestrators/test_simple_orchestrator.py` (+0 -76) </details> ### 📄 Description Remove the orchestrator abstraction and relative boilerplate. The two methods used are now moved to the `ControlPlane` class directly, which used to be the only client of that api. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 02:15:24 -05:00
yindo closed this issue 2026-02-16 02:15:24 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_deploy#509