Skip to content

RS: Added separate local disk requirements for Auto Tiering and Flex databases#3568

Open
rrelledge wants to merge 2 commits into
mainfrom
DOC-5407
Open

RS: Added separate local disk requirements for Auto Tiering and Flex databases#3568
rrelledge wants to merge 2 commits into
mainfrom
DOC-5407

Conversation

@rrelledge

@rrelledge rrelledge commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Note

Low Risk
Documentation-only changes to sizing guidance; no code or runtime behavior.

Overview
Documentation now treats local SSD sizing differently for Auto Tiering versus Flex, instead of a single combined “Redis Flex and Auto Tiering” row.

The production hardware table adds a dedicated Flex local-disk row ( the total memory limit of all Flex databases on the node, min and recommended) and keeps Auto Tiering on (RAM+Flash) × 1.6 / 2.5, with a new footnote to size Flex disk for expected peak memory limits after increases.

The same split appears in flash/memory-limit guidance (section retitled to flash-enabled databases) and Flex plan hardware bullets, each clarifying Auto Tiering’s RAM+Flash floor vs Flex’s node-level 3× memory-limit rule.

Reviewed by Cursor Bugbot for commit 9850baf. Bugbot is set up for automated code reviews on this repo. Configure here.

@rrelledge rrelledge requested review from a team and gilcohen-pm June 29, 2026 20:59
@rrelledge rrelledge self-assigned this Jun 29, 2026
@rrelledge rrelledge added the rs Redis Software label Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

DOC-5407

@jit-ci

jit-ci Bot commented Jun 29, 2026

Copy link
Copy Markdown

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@dwdougherty dwdougherty left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9850baf. Configure here.

- For the best performance, the SSDs should be NVMe based. NVMe Gen 5 or Gen 4 is recommended. Gen 3 is also supported, but not recommended for best performance.

- Flash storage must be greater than the total provisioned database size to account for write buffers, space amplification, and more.
- Flash storage must be greater than the total provisioned database size to account for write buffers, space amplification, and more. Flash space should be approximately three times the total memory limit of all Flex databases on the node. Because you can increase a database's memory limit after creation, size flash storage for the expected peak memory limit.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Conflicting Flex flash sizing rules

High Severity

The updated hardware requirements bullet still requires Flex flash to be greater than a single database’s provisioned size, while the same change set defines the minimum as three times the total memory limit of all Flex databases on the node. On nodes with multiple Flex databases, the weaker rule can be read as sufficient and lead to severe under-provisioning.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9850baf. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rs Redis Software

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants