Skip to content

Conversation

@naki3004
Copy link

@naki3004 naki3004 commented Apr 3, 2025

When deployed to k8s using helm, the service name was not picking dynamically according to chart name.

Have removed the hardcoded service and name and added chart name as the URL in kong declarative configuration.

What kind of change does this PR introduce?

Bug fix, feature, docs update, ...

What is the current behavior?

Please link any relevant issues here.

What is the new behavior?

Feel free to include screenshots if it includes visual changes.

Additional context

Add any other context or screenshots.

When deployed to k8s using helm, the service name was not picking dynamically according to chart name.

Have removed the hardcoded service and name and added chart name as the URL in kong declarative configuration.
@bbo76
Copy link

bbo76 commented May 19, 2025

why not merged ? it's a real problem this bug...

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>
@ErikBjare
Copy link

Duplicate of #99

@georgiev-anton
Copy link

why not merged ?

@luizfelmach
Copy link
Contributor

This bug has already been fixed in PRs #129, #107, #99, and #131.

PR #131 contains the latest contributions and consolidates the fixes.

It will likely be merged soon.

Thanks a lot for the work and the contribution! 🙌

@aantti
Copy link
Contributor

aantti commented Dec 29, 2025

Closing :)

@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.

6 participants