Skip to content

Document external Redis/MariaDB support status and Redis bench-level architecture#4

Draft
Copilot wants to merge 4 commits into
mainfrom
copilot/check-external-redis-mariadb
Draft

Document external Redis/MariaDB support status and Redis bench-level architecture#4
Copilot wants to merge 4 commits into
mainfrom
copilot/check-external-redis-mariadb

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 12, 2026

The problem statement asked to analyze the codebase and clarify whether the operator supports external Redis and MariaDB. The documentation was incomplete and in some cases misleading.

Key Findings

External MariaDB/MySQL: Not Implemented

  • API defines mode: external but MariaDBProvider only handles shared/dedicated (lines 270-277)
  • CRD includes the enum value but implementation returns "unsupported database mode" error
  • Workaround: Reference external instances via MariaDB Operator CRs

External Redis: Not Implemented

  • ConnectionSecretRef field exists in API but marked TODO in controller (line 1504)
  • Only in-cluster Redis/Dragonfly currently supported

Redis Architecture: Bench-Level Sharing

  • Each FrappeBench creates 2 Redis instances (redis-cache, redis-queue)
  • All sites on same bench share these instances using Redis db 0
  • No per-site isolation—namespacing via Frappe key prefixes only
  • Unlike MariaDB which provides true per-site database isolation

Documentation Changes

Added "External Database and Redis" section to examples.md:

# This configuration is documented but returns error
dbConfig:
  mode: external  # ⚠️ Not implemented
  connectionSecretRef:
    name: external-db-credentials

Summary Table:

Feature Status Isolation Level
Shared/Dedicated MariaDB ✅ Supported Per-site database
In-cluster Redis ✅ Supported Bench-level (shared)
External MariaDB ⚠️ Planned -
External Redis ⚠️ Planned -

Includes architecture diagrams, configuration examples, workarounds, and code references for verification.

Original prompt

On examples.md, Understand the code and let me know if this operator supports external.redis and mariadb


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Jan 12, 2026
Copilot AI and others added 3 commits January 12, 2026 11:38
Co-authored-by: varun-krishnamurthy <128337350+varun-krishnamurthy@users.noreply.github.com>
… status

Co-authored-by: varun-krishnamurthy <128337350+varun-krishnamurthy@users.noreply.github.com>
Co-authored-by: varun-krishnamurthy <128337350+varun-krishnamurthy@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate operator support for external Redis and MariaDB Document external Redis/MariaDB support status and Redis bench-level architecture Jan 12, 2026
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