From 238f74e6eeb7f94f882049b4e816d23bb942d668 Mon Sep 17 00:00:00 2001 From: Copilot <198982749+Copilot@users.noreply.github.com> Date: Sun, 15 Feb 2026 16:30:23 +0100 Subject: [PATCH] incubator/valkey: replace remaining redis-cli probes with valkey-cli (#45014) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `incubator/valkey` still used Redis CLI references in probe scripts, which made the chart inconsistent with Valkey naming. This update aligns healthcheck command paths with Valkey while keeping existing runtime compatibility env vars intact. - **Probe command alignment** - Updated all health probe scripts in `charts/incubator/valkey/values.yaml` to call `valkey-cli` instead of `redis-cli`: - `ping_readiness_local.sh` - `ping_liveness_local.sh` - `ping_readiness_master.sh` - `ping_liveness_master.sh` - **Chart metadata** - Bumped `charts/incubator/valkey/Chart.yaml` version: - `0.0.2` → `0.0.3` - **Representative change** ```bash - redis-cli -h localhost -p $REDIS_PORT ping + valkey-cli -h localhost -p $REDIS_PORT ping ``` > [!WARNING] > >
> Firewall rules blocked me from connecting to one or more addresses (expand for details) > > #### I tried to connect to the following addresses, but was blocked by firewall rules: > > - `oci.trueforge.org` > - Triggering command: `/usr/local/bin/helm helm dependency build` (dns block) > > If you need me to access, download, or install something from one of these locations, you can either: > > - Configure [Actions setup steps](https://gh.io/copilot/actions-setup-steps) to set up my environment, which run before the firewall is enabled > - Add the appropriate URLs or hosts to the custom allowlist in this repository's [Copilot coding agent settings](https://github.com/trueforge-org/truecharts/settings/copilot/coding_agent) (admins only) > >
--- 💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey). --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: PrivatePuffin <7613738+PrivatePuffin@users.noreply.github.com> --- charts/incubator/valkey/Chart.yaml | 3 +-- charts/incubator/valkey/values.yaml | 8 ++++---- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/charts/incubator/valkey/Chart.yaml b/charts/incubator/valkey/Chart.yaml index 7fb98235b8d..04b39ee3528 100644 --- a/charts/incubator/valkey/Chart.yaml +++ b/charts/incubator/valkey/Chart.yaml @@ -42,5 +42,4 @@ sources: - https://github.com/trueforge-org/truecharts/tree/master/charts/incubator/valkey - https://hub.docker.com/r/bitnamisecure/valkey type: application -version: 0.0.2 - +version: 0.0.3 diff --git a/charts/incubator/valkey/values.yaml b/charts/incubator/valkey/values.yaml index 54c1c8e9f57..a615250b7a8 100644 --- a/charts/incubator/valkey/values.yaml +++ b/charts/incubator/valkey/values.yaml @@ -56,7 +56,7 @@ configmap: [[ -n "$REDIS_PASSWORD" ]] && export REDISCLI_AUTH="$REDIS_PASSWORD" response=$( timeout -s 3 $1 \ - redis-cli \ + valkey-cli \ -h localhost \ -p $REDIS_PORT \ ping @@ -70,7 +70,7 @@ configmap: [[ -n "$REDIS_PASSWORD" ]] && export REDISCLI_AUTH="$REDIS_PASSWORD" response=$( timeout -s 3 $1 \ - redis-cli \ + valkey-cli \ -h localhost \ -p $REDIS_PORT \ ping @@ -84,7 +84,7 @@ configmap: [[ -n "$REDIS_MASTER_PASSWORD" ]] && export REDISCLI_AUTH="$REDIS_MASTER_PASSWORD" response=$( timeout -s 3 $1 \ - redis-cli \ + valkey-cli \ -h $REDIS_MASTER_HOST \ -p $REDIS_MASTER_PORT_NUMBER \ ping @@ -98,7 +98,7 @@ configmap: [[ -n "$REDIS_MASTER_PASSWORD" ]] && export REDISCLI_AUTH="$REDIS_MASTER_PASSWORD" response=$( timeout -s 3 $1 \ - redis-cli \ + valkey-cli \ -h $REDIS_MASTER_HOST \ -p $REDIS_MASTER_PORT_NUMBER \ ping