Skip to content

docs(agent): corrects the Redis sizing requirement to one logical database - #12898

Merged
mergify[bot] merged 1 commit into
Mergifyio:mainfrom
mergify-ci-bot:docs-agent/health-audit-platform-1--redis-requires-one-logical-database-not-sixteen
Sep 21, 2026
Merged

mergify[bot] merged 1 commit into
Mergifyio:mainfrom
mergify-ci-bot:docs-agent/health-audit-platform-1--redis-requires-one-logical-database-not-sixteen

Conversation

@mergify-ci-bot

Copy link
Copy Markdown
Contributor

The Redis requirements asked operators to provide "at least 16 logical
databases". That was true when the engine sharded its state across a range of
numbered databases, and it stopped being true once the queue and the event
stores moved to PostgreSQL. The engine now opens exactly one Redis connection,
for its cache, and on an on-premise deployment that connection is whatever
MERGIFYENGINE_REDIS_URL points at — database and all.

The number mattered in practice. Sixteen is the default databases setting of
a stock Redis, but several managed Redis offerings expose one database per
instance, and a reader sizing against this page would rule out a provider that
serves the engine perfectly well, or provision a second instance nobody needs.

Evidence: the engine's Redis settings expose a single client URL, and the
database-sharding map behind it has one entry left.

@mergify
mergify Bot deployed to Mergify Merge Protections September 20, 2026 22:41 Active
@mergify

mergify Bot commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 5 merge protections satisfied — ready to merge.

Show 5 satisfied protections

🟢 👀 Review Requirements

  • any of:
    • #approved-reviews-by >= 1
    • author = dependabot[bot]
    • author = renovate[bot]
    • all of:
      • -head ~= ^docs-agent/
      • author = mergify-ci-bot

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team September 20, 2026 22:43
@mergify

mergify Bot commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 2 minutes 57 seconds in the queue, including 2 minutes 34 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Sep 21, 2026
@mergify
mergify Bot merged commit ec48624 into Mergifyio:main Sep 21, 2026
12 checks passed
@mergify mergify Bot removed the queued label Sep 21, 2026

This branch was successfully deployed

1 active deployment
Mergify Merge Protections — d943943b Deployed Sep 20, 2026 by mergify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants