chore: add optional redis uri output#147
Conversation
|
Full scan comparison results are too large to post as a comment. Link to full results artifact: Results bitnamilegacy/valkey
|
There was a problem hiding this comment.
Pull request overview
Adds an optional Zarf component intended to expose a REDIS_URI variable derived from the deployed Valkey instance’s Kubernetes secret/service.
Changes:
- Adds a new optional component (
valkey-redis-uri-output) to the package and common import that reads the Valkey password secret and setsREDIS_URI. - Documents the new component and the constructed URI formats.
- Enables the optional component in the dev/test UDS bundle.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
zarf.yaml |
Exposes the new optional component at the package root by importing from common. |
common/zarf.yaml |
Implements the component action to compute and set REDIS_URI. |
docs/configuration.md |
Documents how the optional component determines and formats the URI. |
bundle/uds-bundle.yaml |
Turns on the optional component for one bundle package instance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 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.
Description
Adds an optional Zarf component intended to expose a REDIS_URI variable derived from the deployed Valkey instance’s Kubernetes secret/service.
Related Issue
Fixes #
Relates to #
Type of change
Checklist before merging