Skip to content

docs(proposals): add proposal to replace Redis with Valkey#281

Merged
Vad1mo merged 1 commit into
goharbor:mainfrom
chlins:feat/valkey
Apr 23, 2026
Merged

docs(proposals): add proposal to replace Redis with Valkey#281
Vad1mo merged 1 commit into
goharbor:mainfrom
chlins:feat/valkey

Conversation

@chlins

@chlins chlins commented Apr 21, 2026

Copy link
Copy Markdown
Member

Issue: goharbor/harbor#22935

This pull request introduces a proposal to replace Harbor's internal Redis runtime with Valkey in a carefully phased approach. The initial phase focuses on a minimal, low-risk swap of the runtime package and image name, ensuring full backward compatibility for users and existing configurations. More extensive terminology and SDK changes are explicitly deferred to a future phase.

Key highlights of the proposal:

Phase 1: Minimal Valkey Adoption

  • Swap the internal Redis runtime in Harbor's bundled image with Valkey by changing the installed package from redis to valkey, and rename the image from goharbor/redis-photon to goharbor/valkey-photon. All other build directories, service names, configuration keys, environment variables, and on-disk layouts remain unchanged for maximum compatibility.
  • Update deployment templates (docker-compose, offline installer, and harbor-helm) to reference the new valkey-photon image, while keeping all service names, configuration paths, and environment variables the same.
  • Ensure compatibility with existing configurations, external Redis/Valkey deployments, and in-place upgrades from previous Harbor releases.

Phase 2: Future Work (Not in This Release)

  • Plan for renaming internal build directories, configuration keys, and service names from redis to backend-neutral or valkey

Signed-off-by: chlins <chlins.zhang@gmail.com>
@chlins

chlins commented Apr 21, 2026

Copy link
Copy Markdown
Member Author

@goharbor/all-maintainers Please take a look for this proposal, thanks!

@Vad1mo Vad1mo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We are running our Saas registries with valkey. It was a drop in replacement

@wy65701436 wy65701436 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm, thanks.

@Vad1mo Vad1mo merged commit 9f49688 into goharbor:main Apr 23, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants