-
Notifications
You must be signed in to change notification settings - Fork 251
Feature/supabase 2025 #131
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
Feature/supabase 2025 #131
Conversation
|
thank you for the work! only complaint i have would be the chart still freaks out when using unencrypted db.password. as in the deployment the chart always expect encrypted version ill start using through out the week and let you know if there's anything else |
|
Thanks for the update @luizfelmach ! I was planning to update the supabase-community Helm chart as well, so I’m checking before duplicating work. Also, does this chart fully match the latest official supabase/docker-compose setup? |
It’s important to keep it updated with what exists in https://github.com/supabase/supabase/blob/master/docker/docker-compose.yml. I didn’t deploy Supavisor, but it would be very interesting to include it. I believe the only thing missing is that component. |
|
This is some monumental work here @luizfelmach, appreciate it! :) |
|
We’ve been using this version in our staging environment for two weeks now, and it looks good. |
|
As the state of this repo hasn't changed much for some time, I'm very much leaning towards accepting this PR as-is to bring it back to a working condition :) → then getting back to the accumulated issues and PRs and figuring out what was solved and can be closed, and what still requires work. Thoughts, @luizfelmach and everyone? Cc @arpagon Offtopic: @arpagon - tried to reach via internal channels; @luizfelmach - sent you an email :) |
Sounds good to me. Getting this PR in to bring the repo back to a working state and then going through the backlog of issues/PRs to see what’s already addressed vs. what still needs work seems like a very reasonable approach. I can help review and triage the existing issues and PRs after this is merged. |
|
Sounds great, @luizfelmach - very much appreciated and looking forward. |
|
Apparently the pipeline failed due to formatting issues and tests that weren't updated. The last two commits fix this. @aantti. |
aantti
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm approving this as the new baseline. @luizfelmach will review the accumulated issues and PRs. A lot of stuff has been already solved by this PR #131, but there are still quite a few things in the backlog.
Related: #120
Feature: Migrate to Supabase 2025 Updates
This pull request introduces significant updates to the
supabase-kubernetesHelm chart to align with the latest changes and features in the Supabase 2025 platform.The primary goal of this migration is to ensure compatibility, incorporate new best practices, and resolve critical issues related to the deployment of a self-hosted Supabase instance on Kubernetes.
Key Changes
refactor: migrate project to Supabase 2025 updatesThis commit includes the bulk of the migration work, which involves updating configurations and scripts to match the Supabase 2025 architecture.
charts/supabase/Chart.yamlhas been incremented from0.1.3to0.2.0.99-jwt.sql._analyticsand_supavisor, in the database initialization scripts (97-_supabase.sql,99-logs.sql,99-pooler.sql,99-realtime.sql). This suggests an update to the logging, connection pooling, and real-time components.supabase_functions.http_request()function in98-webhooks.sqlhas been modified, likely to update its signature or internal logic to align with the latestpg_netextension usage or Supabase's function calling standards.charts/supabase/templates/db/statefulset.yaml,charts/supabase/templates/realtime/deployment.yaml,charts/supabase/templates/studio/deployment.yaml) to reflect new image tags, environment variables, and configuration mounts for the updated services.This PR is ready for review and merge. Please let me know if you have any questions.