Skip to content

docs: add missing port mapping to minimal redis-proxy docker run example#368

Merged
bootjp merged 3 commits intofeature/redis-proxyfrom
copilot/sub-pr-367
Mar 20, 2026
Merged

docs: add missing port mapping to minimal redis-proxy docker run example#368
bootjp merged 3 commits intofeature/redis-proxyfrom
copilot/sub-pr-367

Conversation

Copy link
Contributor

Copilot AI commented Mar 20, 2026

The "Minimal (redis-only)" docker run example omitted -p 6379:6379, making the proxy unreachable from the host in a typical local setup.

Change

  • Added -p 6379:6379 to the minimal docker run snippet in docs/redis-proxy-deployment.md
docker run --rm \
  -p 6379:6379 \
  ghcr.io/bootjp/elastickv/redis-proxy:latest \
  -listen :6379 \
  -primary redis.internal:6379 \
  -mode redis-only

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: bootjp <1306365+bootjp@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback on redis-proxy Dockerfile and GHCR workflow adjustments docs: add missing port mapping to minimal redis-proxy docker run example Mar 20, 2026
Copilot AI requested a review from bootjp March 20, 2026 09:35
@bootjp bootjp marked this pull request as ready for review March 20, 2026 09:49
@bootjp bootjp merged commit 6e91ade into feature/redis-proxy Mar 20, 2026
@bootjp bootjp deleted the copilot/sub-pr-367 branch March 20, 2026 09:49
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.

2 participants