AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

tweak(platform): Remove unused supabase services in docker compose

Open aarushik93 opened this issue 1 year ago β€’ 4 comments

Background

Now that we've added supabase, we have soo many services, many that we dont use. Unpick and remove the ones we don't need.

Changes πŸ—οΈ

Removed realtime, storage, imgproxy, meta, functions

Testing πŸ”

[!NOTE] Only for the new autogpt platform, currently in autogpt_platform/

  • Create from scratch and execute an agent with at least 3 blocks
  • Import an agent from file upload, and confirm it executes correctly
  • Upload agent to marketplace
  • Import an agent from marketplace and confirm it executes correctly
  • Edit an agent from monitor, and confirm it executes correctly

aarushik93 avatar Sep 25 '24 22:09 aarushik93

PR Reviewer Guide πŸ”

⏱️ Estimated effort to review: 2 πŸ”΅πŸ”΅βšͺβšͺβšͺ
πŸ§ͺΒ No relevant tests
πŸ”’Β No security concerns identified
⚑ Key issues to review

Service Removal
Removal of several Supabase services (realtime, storage, imgproxy, meta, functions) from the docker-compose file. Ensure this doesn't impact the functionality of the platform.

qodo-code-review[bot] avatar Sep 25 '24 22:09 qodo-code-review[bot]

Deploy Preview for auto-gpt-docs ready!

Name Link
Latest commit a32cd2083289700eba66a4781b97d4b01bbcee06
Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/66f584d1540fe700086d2a91
Deploy Preview https://deploy-preview-8178--auto-gpt-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Sep 25 '24 22:09 netlify[bot]

imo set this up with profiles https://docs.docker.com/compose/how-tos/profiles/

ntindle avatar Sep 25 '24 22:09 ntindle

@ntindle why though? in which situation do we use all of these? even for local dev we don't need all of them

aarushik93 avatar Sep 26 '24 08:09 aarushik93

That’s totally fair, earlier Nick was wrong

ntindle avatar Sep 26 '24 15:09 ntindle