incubator/valkey: replace remaining redis-cli probes with valkey-cli (#45014)
`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]
>
> <details>
> <summary>Firewall rules blocked me from connecting to one or more
addresses (expand for details)</summary>
>
> #### 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)
>
> </details>
<!-- START COPILOT CODING AGENT TIPS -->
---
💬 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>
This commit is contained in:
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
Reference in New Issue
Block a user