Skip to content

feat(ilf-app-backend): added support for redis url - #332

Merged
bosbaber merged 5 commits into
mainfrom
stephan/wal-456
Dec 9, 2025
Merged

bosbaber merged 5 commits into
mainfrom
stephan/wal-456

Conversation

@bosbaber

Copy link
Copy Markdown
Collaborator

This pull request adds support for configuring the Redis connection URL via a secret reference in the backend Helm chart. The main changes include introducing a new redis_url_ref configuration option and wiring it up so both the server and worker deployments receive the Redis URL from the referenced secret.

Redis URL configuration:

  • Added redis_url_ref to the config section in charts/interledger-app/backend/values.yaml to allow specifying the secret key for the Redis connection URL.

Environment variable injection:

  • Updated the server deployment to inject the REDIS_URL environment variable using the value from the secret referenced by redis_url_ref.
  • Updated the worker deployment to inject the REDIS_URL environment variable using the value from the secret referenced by redis_url_ref.

@bosbaber bosbaber self-assigned this Nov 18, 2025
@bosbaber
bosbaber requested a review from a team as a code owner November 18, 2025 10:45
Copilot AI review requested due to automatic review settings November 18, 2025 10:45
@linear

linear Bot commented Nov 18, 2025

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for configuring Redis connection URLs via secret references in the backend Helm chart, enabling both the server and worker deployments to access Redis through environment variables.

  • Added redis_url_ref configuration to specify the secret key for the Redis URL
  • Injected REDIS_URL environment variable into the server deployment
  • Injected REDIS_URL environment variable into the worker deployment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@raducristianpopa

Copy link
Copy Markdown
Contributor

Is this still needed?

CC @RaulBR

@bosbaber
bosbaber disabled auto-merge December 9, 2025 12:30
@bosbaber
bosbaber merged commit e5a2b97 into main Dec 9, 2025
16 checks passed
@bosbaber
bosbaber deleted the stephan/wal-456 branch December 9, 2025 12:31
github-actions Bot added a commit that referenced this pull request Dec 9, 2025
Automated chart versioning after merge of PR #332.
This PR updates Chart.yaml versions, packages updated charts into
docs/interledger,
and regenerates the Helm index.

Co-authored-by: bosbaber <1615407+bosbaber@users.noreply.github.com>
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.

4 participants