Skip to content

Add check for existing env vars in backend templates #49

Open
cedricc13 wants to merge 1 commit intolangflow-ai:mainfrom
cedricc13:add-env-check-in-backend
Open

Add check for existing env vars in backend templates #49
cedricc13 wants to merge 1 commit intolangflow-ai:mainfrom
cedricc13:add-env-check-in-backend

Conversation

@cedricc13
Copy link
Copy Markdown

What does this PR do?

  • Adds a helper function envHasName to check if an environment variable is already defined in the backend env list.
  • Uses this function in backend-statefulset.yaml to avoid adding duplicate environment variables.
  • Fixes Helm template errors when backend.env is a list.

Why is this needed?

  • Helm templates failed because .Values.langflow.backend.env is a list, not a map.
  • Prevents overwriting or duplicating environment variables.

How to test?

  • Run helm template with updated values.yaml to verify no duplicate env vars.

@flefevre
Copy link
Copy Markdown

flefevre commented Jun 3, 2025

Seems good and will be very useful.

@blegry
Copy link
Copy Markdown

blegry commented Jun 10, 2025

Yes, need this update !

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.

3 participants