Skip to content

Conversation

@kkatsaev
Copy link

What kind of change does this PR introduce?

Bug fix when replacing value from secret "secretRef" from values.yaml.

What is the current behavior?

See:

What is the new behavior?

If "secretRef" is present, the data from secret will be used.

@sergey-vin
Copy link

sergey-vin commented Apr 8, 2025

Hi dear repo maintainers. @saltcod , @arpagon , @attiasr , @inian, @kiwicopple (not sure who's the right person to ping)
Any update on merging this?
It seems that this is a critical patch that fixes otherwise broken installation.
Thanks

@mattias-wiberg
Copy link

Duplicate of #94

kastakhov added a commit to wiseinfotec/helm-charts that referenced this pull request Jul 27, 2025
Merging some pull requests from
[original](https://github.com/supabase-community/supabase-kubernetes)
repository.

- supabase-community/supabase-kubernetes#62
- supabase-community/supabase-kubernetes#89
- supabase-community/supabase-kubernetes#91
- supabase-community/supabase-kubernetes#94
- supabase-community/supabase-kubernetes#95
- supabase-community/supabase-kubernetes#96
- supabase-community/supabase-kubernetes#101
- supabase-community/supabase-kubernetes#106
- supabase-community/supabase-kubernetes#107
- supabase-community/supabase-kubernetes#113

---------

Co-authored-by: Varantha <sam@kiessler.co.uk>
Co-authored-by: naki3004 <45875285+naki3004@users.noreply.github.com>
Co-authored-by: jingyu <francismajere@gmail.com>
Co-authored-by: Chris Alfano <chris@jarv.us>
Co-authored-by: Simone Lazzaris <slazzaris@gmail.com>
Co-authored-by: Mikey032 <26899585+Mikey032@users.noreply.github.com>
Co-authored-by: Upik Saleh <upxsal@gmail.com>
Co-authored-by: Scott Beardsley <scott@beards.ly>
@luizfelmach
Copy link
Contributor

Thanks for the contribution 🙏
This PR appears to be a duplicate of #94 and has already been fixed in PR #131.

Cc @aantti

@aantti
Copy link
Contributor

aantti commented Dec 29, 2025

Thanks, everyone 🙏

@aantti aantti closed this Dec 29, 2025
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.

5 participants