[Feature]: Document Valkey support for the Redis-backed features
Maintainers usually reply within 1 day
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 82/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- markdown, redis
- Domain
- documentation
Research direction
Read guides/14-ragfs-cache.md and the relevant Redis-backed sections in guides/01-configuration.md, then find their Chinese counterparts under docs/zh. Update the three feature sections to document Valkey 7.2–9.1 using the existing Redis configuration, and show valkey-server as a RAGFS Quick Start alternative. Done when both language directories describe all three supported features consistently.
Written by the indexing model from the issue text.
Description
Issue Origin
Observed or reproduced in a real environment
Problem Statement
Valkey is an open source fork of Redis, maintained under the Linux Foundation and licensed under BSD-3-Clause. It was forked just before Redis moved to source-available licenses. It keeps the Redis protocol and commands, and existing Redis clients work with it unchanged. Many cloud providers offer it as a managed service, including Amazon ElastiCache, Google Cloud Memorystore, Tencent Cloud, and DigitalOcean.
OpenViking has three optional features that use a Redis server:
| Feature | Configuration | Docs |
|---|---|---|
| RAGFS cache (CacheFS, QueueFS, PathLock, multi-write backups) | cache.provider: "redis" |
guides/14-ragfs-cache.md |
| Account store cache | account_store.provider: "redis_mysql" |
guides/01-configuration.md |
| Distributed session expiry | idle_backend: "redis" |
guides/01-configuration.md |
The docs for these features only mention Redis. A team that runs Valkey, or a managed service built on it, can't tell whether OpenViking supports it. Today they have to run their own validation, or deploy a separate Redis server just for OpenViking.
Proposed Solution
Document Valkey as a supported server for these three features. All three already work with Valkey through the existing redis configuration, so this is a docs-only change with no code or configuration changes.
- Add a note to each feature's section that Valkey 7.2 to 9.1 is supported, with the same configuration as Redis.
- In the RAGFS Quick Start, show
valkey-serveras an alternative toredis-server. - Update
docs/enanddocs/zh.
I would send this as three small PRs, one per feature. I'd need to use machine translation for the docs/zh changes, assuming that's preferable to excluding them, as I only speak English.
Alternatives Considered
- A separate
valkeyprovider name. OpenViking's clients never check which server they're connected to, so this would only add a second name for identical behavior. - Leave it undocumented. Valkey users would keep validating it themselves before relying on it.
Feature Area
Documentation
Use Case
A team that already runs Valkey for caching or queues can use the same deployment for OpenViking's Redis-backed features.
Example API (Optional)
{
"cache": {
"provider": "redis",
"params": {
"mode": "standalone",
"endpoints": ["redis://valkey.example.com:6379"]
}
}
}
Additional Context
I tested at 0a2578e against Valkey 7.2.14, 8.0.11, 8.1.10, 9.0.6, and 9.1.2. These are the newest patch releases of every maintained Valkey line. I used Redis 8.10.2 as a control.
I tested each of the features as follows:
- RAGFS cache: the existing
cargo test -p ragfs --features cachesuite, plus the Quick Start and multi-write backups through a runningopenviking-server. - Account store: the existing
redis_mysqllive test, plus a probe built through the server configuration. - Session expiry: two
openviking-serverinstances withidle_backend: "redis". Idle sessions were committed exactly once, including after one instance was killed and the other took over.
Each feature ran on multiple topologies where supported. This included standalone, cluster, and Sentinel modes. The runs also covered ACL users and TLS. I ran every check against Redis 8.10.2 as well, and Valkey gave the same result in every case.
Area contacts from CONTRIBUTING.md: @baojun-zhang for Storage (RAGFS cache) and @qin-ctx for Platform (account store, session expiry).
Contribution
- I am willing to contribute to implementing this feature
- Dominant language
- Python
- Stars
- 40.2k
- Forks
- 3.2k
- Avg merge
- 1d 5h
- Merged PRs (30d)
- 382
Getting set up
- Ships a Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from volcengine/OpenViking
-
[Bug]: write normalizes whitespace-bearing segments silently while extraction keeps interior spaces — silent forkPossibly taken @r266-tech claimed this 1 day ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
volcengine/OpenViking#6521 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
volcengine/OpenViking#6489 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
volcengine/OpenViking#6471 · 3 comments ·
Maintainers usually reply within 1 day
-
[Bug]: Python SDK duplicates URI scheme after leading whitespacePossibly taken @ktz03 claimed this 3 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
volcengine/OpenViking#6445 ·
Maintainers usually reply within 1 day
-
[Bug]: memory_policy.working_memory.enabled is silently ignored on 0.5.0 — Working Memory stays offPossibly taken @dannyward630 claimed this 6 days ago. Openbug kernel
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
volcengine/OpenViking#6207 · 1 comment ·
Maintainers usually reply within 1 day
All issues in volcengine/OpenViking
Similar issues
-
Project submission:Open
Difficulty 1/5 Under an hour Newbie friendliness 72/100
slavakurilyak/awesome-ai-agents#845 ·
Maintainers usually reply within 1 day
-
tests: pin that the atomic writers in utils/paths.py stage the temporary file in the target's directoryPossibly taken A pull request linked to this issue is open or already merged. Opendifficulty:2 enhancement good first issue
Difficulty 1/5 1-3 hours Newbie friendliness 92/100
MakazhanAlpamys/Soup#2178 ·
Maintainers usually reply within 1 day
-
[BUG] dcg.waggr permutation p-values are folded twice, so near-zero scores get the smallest p-valuesOpen
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
scverse/rapids-singlecell#854 ·
Maintainers usually reply within 1 day
-
Slang input: int8_t4_packed and uint8_t4_packed are copied into CrossGLPossibly taken @ninglinLiu claimed this 1 day ago. Openarea:frontend backend:slang bug direction:input
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
random_elements(unique=True) on an OrderedDict: ZeroDivisionError once only zero weights remain, AssertionError with use_weighting=FalsePossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Maintainers usually reply within 13 days