Skip to content

Conversation

@propilideno
Copy link
Contributor

@propilideno propilideno commented Nov 14, 2025

What kind of change does this PR introduce?

Feature: support envFrom to allow deploy secretRef to use as a replacement for edge functions secrets

What is the current behavior?

Allow only plain-values inside values.yaml

What is the new behavior?

Allow pass envFrom to edge-runtime container

@propilideno propilideno force-pushed the feature/functions_support_envFrom branch from 990940b to 6c30e84 Compare November 14, 2025 20:37
@propilideno propilideno changed the title pfeat: add support to envFrom to deploy edge functions secrets feat: add support to envFrom to deploy edge functions secrets Nov 14, 2025
@luizfelmach
Copy link
Contributor

Thanks for working on this! 🙌
This is already being addressed in #131.
The only remaining change needed here is to add envFrom: [] to both values.yaml and values.example.yaml.

@luizfelmach
Copy link
Contributor

This is just an update to values.yaml.
This can be merged. If the CI fails, it’s probably because there’s no version bump. If you’re able to, you can merge it even with the CI failing @aantti.

@propilideno
Copy link
Contributor Author

Thanks for working on this! 🙌 This is already being addressed in #131. The only remaining change needed here is to add envFrom: [] to both values.yaml and values.example.yaml.

Hello @luizfelmach, sorry for the late response.
The values.example.yaml file only includes values that override the defaults defined in values.yaml, so I don’t think it’s necessary to update both files.

@luizfelmach
Copy link
Contributor

Makes sense, let's just keep the values.yaml, which is enough.

Cc @aantti

@aantti aantti added the edge functions Related to Edge Runtime label Jan 23, 2026
@aantti aantti merged commit 40e70c5 into supabase-community:main Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edge functions Related to Edge Runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants