Skip to content
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

Add a minio, garage, or similar S3 provider as a subchart for testing #55

Open
jessebot opened this issue Jan 24, 2025 · 2 comments · May be fixed by #60
Open

Add a minio, garage, or similar S3 provider as a subchart for testing #55

jessebot opened this issue Jan 24, 2025 · 2 comments · May be fixed by #60

Comments

@jessebot
Copy link
Collaborator

@holysoles was working on this, but if there's further discussion on it, we can chat about it here.

@jessebot jessebot added enhancement New feature or request and removed enhancement New feature or request labels Jan 24, 2025
@holysoles
Copy link
Contributor

So I sorta have a POC, still a work in progress though, its over here

Important note: if using a separate deployment for the Horizon worker, they need to share storage, since apparently its initially uploaded to the primary webserver under /var/www/storage (was getting an "image source not readable" error), then uploaded by the Horizon worker to the S3 bucket

The things I'm still working on:

  • Some more testing around bucket ACLs (can it be public readonly, etc) and generating that. Might need to switch to the official minio community subchart instead of bitnami for that..
  • The object storage URL that minio uploads to is also expected to be reachable from the client via HTTPS to retrieve images.

I'm working on trying to figure out a solution for particularly the latter, since otherwise inclusion of the subchart seems not incredibly useful.

Random resources I found helpful that I figured are worth sharing:

@holysoles
Copy link
Contributor

Okay I have the deployment working, I'll open a PR!

@holysoles holysoles linked a pull request Jan 25, 2025 that will close this issue
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 a pull request may close this issue.

2 participants