[PR #520] [MERGED] chore: remove low-level deployment api #521

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

📋 Pull Request Information

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

Base: mainHead: massi/remove-deploy


📝 Commits (3)

  • 17378f1 chore: move deployment utils to e2e_tests package
  • db2e2d6 chore: remove low-level deployment api
  • 9936015 fix e2e tests

📊 Changes

17 files changed (+24 additions, -156 deletions)

View changed files

📝 .github/workflows/e2e_test.yml (+0 -1)
📝 e2e_tests/basic_hitl/conftest.py (+1 -1)
📝 e2e_tests/basic_session/conftest.py (+1 -1)
📝 e2e_tests/basic_streaming/conftest.py (+1 -1)
📝 e2e_tests/basic_workflow/conftest.py (+1 -1)
📝 e2e_tests/conftest.py (+2 -1)
📝 e2e_tests/core/conftest.py (+1 -1)
e2e_tests/deploy/__init__.py (+0 -0)
e2e_tests/deploy/test_deploy.py (+0 -128)
📝 e2e_tests/message_queues/kafka/conftest.py (+1 -2)
📝 e2e_tests/message_queues/rabbitmq/conftest.py (+1 -1)
📝 e2e_tests/message_queues/redis/conftest.py (+1 -2)
📝 e2e_tests/utils.py (+1 -1)
📝 llama_deploy/__init__.py (+0 -4)
llama_deploy/deploy/__init__.py (+0 -3)
📝 llama_deploy/services/network_service_manager.py (+7 -2)
📝 tests/services/test_network_workflow.py (+6 -6)

📄 Description

Remove deploy() and deploy_workflow() from the public API. Those were meant to offer a "low level" api for advanced users, but we're moving away from it. The same functionalities are still available (we use them in the E2E tests) but need to be manually coded.


🔄 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/520 **Author:** [@masci](https://github.com/masci) **Created:** 5/29/2025 **Status:** ✅ Merged **Merged:** 5/29/2025 **Merged by:** [@masci](https://github.com/masci) **Base:** `main` ← **Head:** `massi/remove-deploy` --- ### 📝 Commits (3) - [`17378f1`](https://github.com/run-llama/llama_deploy/commit/17378f1b203fcf4938c3d4b43e2b0e8c6873294f) chore: move deployment utils to e2e_tests package - [`db2e2d6`](https://github.com/run-llama/llama_deploy/commit/db2e2d6dec34cfda031c6ac0d388a441521670fb) chore: remove low-level deployment api - [`9936015`](https://github.com/run-llama/llama_deploy/commit/99360151cbb105dd962cc653eeeeb39e28973891) fix e2e tests ### 📊 Changes **17 files changed** (+24 additions, -156 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/e2e_test.yml` (+0 -1) 📝 `e2e_tests/basic_hitl/conftest.py` (+1 -1) 📝 `e2e_tests/basic_session/conftest.py` (+1 -1) 📝 `e2e_tests/basic_streaming/conftest.py` (+1 -1) 📝 `e2e_tests/basic_workflow/conftest.py` (+1 -1) 📝 `e2e_tests/conftest.py` (+2 -1) 📝 `e2e_tests/core/conftest.py` (+1 -1) ➖ `e2e_tests/deploy/__init__.py` (+0 -0) ➖ `e2e_tests/deploy/test_deploy.py` (+0 -128) 📝 `e2e_tests/message_queues/kafka/conftest.py` (+1 -2) 📝 `e2e_tests/message_queues/rabbitmq/conftest.py` (+1 -1) 📝 `e2e_tests/message_queues/redis/conftest.py` (+1 -2) 📝 `e2e_tests/utils.py` (+1 -1) 📝 `llama_deploy/__init__.py` (+0 -4) ➖ `llama_deploy/deploy/__init__.py` (+0 -3) 📝 `llama_deploy/services/network_service_manager.py` (+7 -2) 📝 `tests/services/test_network_workflow.py` (+6 -6) </details> ### 📄 Description Remove `deploy()` and `deploy_workflow()` from the public API. Those were meant to offer a "low level" api for advanced users, but we're moving away from it. The same functionalities are still available (we use them in the E2E tests) but need to be manually coded. --- <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:27 -05:00
yindo closed this issue 2026-02-16 02:15:27 -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#521