Skip to content

chore: add optional redis uri output#147

Merged
zachariahmiller merged 3 commits intomainfrom
chore/redis-uri-output
Apr 16, 2026
Merged

chore: add optional redis uri output#147
zachariahmiller merged 3 commits intomainfrom
chore/redis-uri-output

Conversation

@ntwkninja
Copy link
Copy Markdown
Contributor

@ntwkninja ntwkninja commented Apr 15, 2026

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 15, 2026

Full scan comparison results are too large to post as a comment. Link to full results artifact: Results
Summarized results are below:

bitnamilegacy/valkey

bitnamilegacy/valkey 8.1.3-debian-12-r3 -> 8.1.3-debian-12-r3

New vulnerabilities: 0
Fixed vulnerabilities: 0
Existing vulnerabilities: 218


bitnamilegacy/redis-exporter

bitnamilegacy/redis-exporter 1.76.0-debian-12-r0 -> 1.76.0-debian-12-r0

New vulnerabilities: 0
Fixed vulnerabilities: 0
Existing vulnerabilities: 212


bitnamilegacy/valkey-sentinel

bitnamilegacy/valkey-sentinel 8.1.3-debian-12-r3 -> 8.1.3-debian-12-r3

New vulnerabilities: 0
Fixed vulnerabilities: 0
Existing vulnerabilities: 182


quay.io/rfcurated/valkey/valkey

quay.io/rfcurated/valkey/valkey 8.1.6-jammy-fips-rfcurated -> 8.1.6-jammy-fips-rfcurated

New vulnerabilities: 0
Fixed vulnerabilities: 0
Existing vulnerabilities: 13


quay.io/rfcurated/redis-exporter

quay.io/rfcurated/redis-exporter 1.81.0-jammy-scratch-bnt-fips-rfcurated -> 1.81.0-jammy-scratch-bnt-fips-rfcurated

New vulnerabilities: 0
Fixed vulnerabilities: 0
Existing vulnerabilities: 9


quay.io/rfcurated/valkey-sentinel

quay.io/rfcurated/valkey-sentinel 8.1.6-jammy-bnt-fips-rfcurated -> 8.1.6-jammy-bnt-fips-rfcurated

New vulnerabilities: 0
Fixed vulnerabilities: 0
Existing vulnerabilities: 9



generated with uds-pk scan compare

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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 sets REDIS_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.

Comment thread common/zarf.yaml Outdated
Comment thread docs/configuration.md
Comment thread bundle/uds-bundle.yaml Outdated
Copy link
Copy Markdown

Copilot AI left a comment

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

Comment thread common/zarf.yaml Outdated
Comment thread common/zarf.yaml Outdated
Comment thread common/zarf.yaml
@zachariahmiller zachariahmiller merged commit b203259 into main Apr 16, 2026
15 checks passed
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