-
-
Notifications
You must be signed in to change notification settings - Fork 383
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
S3 Workspaces #1352
Merged
Merged
S3 Workspaces #1352
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The goal in this PR is to switch from locally persisted agent workspaces to putting them in S3 storage. Anyone deploying locally will be able to use a minio container, which will be added to the docker-compose files for anyone running locally. This PR will effectively enable deploying AGiXT on cloud services hosts such as AWS or Digital Ocean without the need to persist files.
This pull request introduces several significant updates to the
agixt
project, focusing on Docker configuration, agent management, and application initialization. The changes include modifications to Docker-related files, enhancements to agent handling, and improvements to the application startup process.Docker Configuration Updates:
Dockerfile
: Added a step to runTranscription.py
during the build process and updated the entry point to useDB.py
.docker-compose-dev.yml
anddocker-compose-nostreamlit-dev.yml
: Updated volumes and environment variables for better configuration management, including the addition of MinIO service for storage. [1] [2] [3] [4]docker-compose-nostreamlit.yml
: Added MinIO service configuration for non-streamlit environments.Agent Management Enhancements:
agixt/Agent.py
: Improved agent name handling to avoid duplicates by appending a number if the agent already exists.Application Initialization Improvements:
agixt/app.py
: Refactored the application startup to include workspace management, task monitoring, and ngrok tunnel setup. Added signal handlers for graceful shutdown and a new route for serving files from the workspace. [1] [2] [3] [4] [5]Miscellaneous Changes:
agixt/Globals.py
: Enhanced thegetenv
function to accept a default value parameter. [1] [2]agixt/Transcription.py
andagixt/providers/default.py
: Updated the transcription model initialization to useint8
compute type for improved performance. [1] [2]launch-backend.sh
script as its functionality has been integrated into the main application.These changes collectively improve the robustness, configurability, and maintainability of the
agixt
project.