Skip to content

Adds storage server to self hosting setup#1007

Merged
inian merged 3 commits intomasterfrom
docker/storage
Jul 23, 2021
Merged

Adds storage server to self hosting setup#1007
inian merged 3 commits intomasterfrom
docker/storage

Conversation

@inian
Copy link
Copy Markdown
Member

@inian inian commented Mar 31, 2021

  • Adds the storage schema to postgres
  • Updates the docker-compose file to use storage-api docker image

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 31, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

docs – ./web

🔍 Inspect: https://vercel.com/supabase/docs/49CV8fMVHVtY1pwNxF8k7B6kurBk
✅ Preview: https://docs-git-docker-storage-supabase.vercel.app

www – ./www

🔍 Inspect: https://vercel.com/supabase/www/EvC4xh6hzaVsvsxKXdmXEvnvN9v1
✅ Preview: Canceled

[Deployment for 424483f canceled]

main – ./

🔍 Inspect: https://vercel.com/supabase/main/5ihBdJgEpncGTcomR5bLxxxCF7qB
✅ Preview: https://main-git-docker-storage-supabase.vercel.app

supabase-slack-clone – ./examples/nextjs-slack-clone

🔍 Inspect: https://vercel.com/supabase/supabase-slack-clone/E8DpcgtDo74GGTwsW4vF4Eb1LEo8
✅ Preview: Canceled

nextjs-todo-list – ./examples/nextjs-todo-list

🔍 Inspect: https://vercel.com/supabase/nextjs-todo-list/X9wNa87NNniELSSuHifbR1vecpwm
✅ Preview: Canceled

config:
hide_credentials: true
- name: storage-v1
_comment: 'Storage: /storage/v1/* -> http://storage-api:5000/*'
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nitpick:

- _comment: 'Storage: /storage/v1/* -> http://storage-api:5000/*'
+ _comment: 'Storage: /storage/v1/* -> http://storage:5000/*'

else match call for update is not working
environment:
ANON_KEY: eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJzdXBhYmFzZSIsImlhdCI6MTYwMzk2ODgzNCwiZXhwIjoxNjkwNDQzMTM1LCJhdWQiOiIiLCJzdWIiOiIiLCJSb2xlIjoiYW5vbiIsImp0aSI6ImQ1N2ZkMjFlLTNkYjQtNDk3YS1hNGViLTI4ZDY3Yzg2NjEwMiJ9.YySz_BoYxYco0b2aP3iBMt-dVUqC6RpnQiSb4Zt-L-o
SERVICE_KEY: eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJzdXBhYmFzZSIsImlhdCI6MTYwMzk2ODgzNCwiZXhwIjoxNjkwNDQzMTM1LCJhdWQiOiIiLCJyb2xlIjoic2VydmljZV9yb2xlIiwianRpIjoiOTk4MzJiYmItNDdhNS00NzJhLTgxMDQtYzY4ZGVjZDQyZDVlIn0._DToQdxDcSLsp3tPpacsxswv0p1KRti0QOTmnbfs5JY
PROJECT_REF: bjwdssmqcnupljrqypxz # can be any random string
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'll extract the anon/service key out in at some stage after this is merged.

Can I double-check what the PROJECT_REF is for?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is for the multitenant part, so that the same storage backend can have objects from multiple projects. All objects are prefixed by this project_ref to properly scope them

PGOPTIONS: "-c search_path=storage"
FILE_SIZE_LIMIT: 52428800
REGION: us-east-1 # region where your bucket is located
GLOBAL_S3_BUCKET: supa-storage-testing # name of s3 bucket where you want to store objects
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can we remove this if the STORAGE_BACKEND is set to file?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yup, this needs a backend fix - supabase/storage#55 Will get it in

@inian inian merged commit 8207f03 into master Jul 23, 2021
@inian inian deleted the docker/storage branch July 23, 2021 06:40
NealShankarGit pushed a commit to NealShankarGit/supabase that referenced this pull request Feb 3, 2025
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

3 participants