Skip to content

Conversation

@kaitlynmichael
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 24, 2025

DOC-6000

Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

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

A couple of link issues.

2. Upgrade the database to Redis 8.0 by updating `spec.redisVersion` to `"8.0"`.
3. Redis Server automatically converts the database to Redis Flex.
4. The operator detects the conversion and adapts its reconciliation behavior.
5. Redis now manages the `rofRamSize` field automatically. You can keep the field in the spec for backward compatibility.

Choose a reason for hiding this comment

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

RS don't do validation on minimum value . If the user will remove it - on upgrade we will preserve the value. On create new DB with spec.redisVersion= "8.0" if the rofRamSize will be missing RS will set default value of 0. Im not sure it will give good performance. I will verify with cluster devs.

When you upgrade a database from a Redis version earlier than 8.0 to Redis 8.0 or later, Redis Server automatically migrates the database from Auto Tiering to Redis Flex. The operator detects this migration and makes the following changes:

1. Stops validating the `rofRamSize` ratio requirement.
2. Stops reconciling the `bigstore_ram_size` field to avoid configuration drift.

Choose a reason for hiding this comment

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

Im not sure I get this part... rofRamSize is on the operator side mapped to bigstore_ram_size in RS.

- `isRof: true` - Enables Redis Flex
- `redisVersion` - Set to `"8.0"` or later
- `memorySize` - Defines the total combined memory size (RAM + flash)
- `rofRamSize` - (Optional) Defines the RAM capacity for the database

Choose a reason for hiding this comment

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

This is a bit confusing from RS side - on one hand there is no validation and the default is 0 but Im not sure from performance pov of the db. We need to verify with them

bigStoreDriver: speedb # Only 'speedb' is suitable for Redis Flex
storageClassName: local-scsi
flashDiskSize: 100G
```

Choose a reason for hiding this comment

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

For redis flex, operator exposes another field - rofRamRatio => in RS - bigstore_max_ram_ratio .
bigstore_max_ram_ratio is how much RAM from total data can be used in percentages.
so setting bigstore_ram_size = 1Gb means the user will have 1Gb RAM and bigstore_max_ram_ratio = 30% means the user will get 3mb of RAM for every 10mb of data he used up until he reaches close to full db (depends on the percentage) and then he will get the same RAM as configured in bigstore_ram_size (rofRamSize)

Choose a reason for hiding this comment

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

the default value when not set is 50%. The recommendation is to use default.

Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

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

LGTM.

@kaitlynmichael
Copy link
Contributor Author

Content approved by Iren via slack. We might have to clarify a few things later, but for initial release, it's all accurate.

@kaitlynmichael kaitlynmichael changed the base branch from main to release-k8s-asgard December 19, 2025 22:09
@kaitlynmichael kaitlynmichael merged commit 5574c80 into release-k8s-asgard Dec 19, 2025
7 checks passed
@kaitlynmichael kaitlynmichael deleted the DOC-6000 branch December 19, 2025 23:06
kaitlynmichael added a commit that referenced this pull request Dec 22, 2025
* 806 index page

* fixed bugs, downloads, highlights

* enhancements

* api changes

* supported distro in RN

* finish RN edits

* k8s api docs 8.0.6-6 (#2567)

* RC Q4 Rebrand: Langcache

* DEV: (cmds) update CLUSTER SLOT-STATS command page (#2563)

* DEV: add redis-cli usage information to its main page (#2566)

* DEV: add redis-cli usage information to its main page

* Fix whitespace

* k8s api docs 8.0.6-6

---------

Co-authored-by: Cameron Bates <cameron.bates@redis.com>
Co-authored-by: David Dougherty <david.dougherty@redis.com>
Co-authored-by: Cameron Bates <102550101+cmilesb@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* K8s: Redis Flex - Asgard release (#2453)

* redis flex updates

* edits for clarity and formatting

* fix build issue

* link fixes

* K8s: User-defined modules (Asgard release) (#2524)

* page improvements

* upgrade prereqs

* upgrade changes

* upgrade notes

* Update content/operate/kubernetes/upgrade/openshift-cli.md

Co-authored-by: David Dougherty <david.dougherty@redis.com>

* Update content/operate/kubernetes/upgrade/upgrade-olm.md

Co-authored-by: David Dougherty <david.dougherty@redis.com>

* fix links

* install and modules page updates

* rolling pod restart note

* helm upgrade note

* cluster recovery

* upgrade troubleshooting

* AA and flash limitations

* links

* add troubleshooting

* copy edits

* review feedback changes

---------

Co-authored-by: David Dougherty <david.dougherty@redis.com>

* supported distros updates

* fix RN weights

* update build numbers

* DEV: (cmds) add RR diag. to FT.HYBRID; update CLUSTER MIGRATION (#2565)

* Update note on 'attribute' in JSON index creation (#2569)

* DOC-6125 RS: Added TLS Web Client Authentication requirement for INE certs (#2574)

* RS 8.0.6 December release docs (#2522)

* RS: SAML SSO (#2498)

* DOC-5858 RS: Added SSO permissions to RS REST API reference

* DOC-5858 RS: Added SSO requests and objects to RS REST API reference

* DOC-5858 RS: Added more info about enforce_control_plane SSO to RS REST API reference

* DOC-5858 RS: Added SSO auth_method to user object in RS REST API reference

* DOC-5858 RS: Added SSO certs to RS REST API reference and certs list

* DOC-5858 RS: Initial draft of SAML SSO for RS

* DOC-5858 A few adjustments to RS SSO draft

* DOC-5858 Added SSO setup steps for uploading SP cert and downloading SP metadata

* DOC-5858 More adjustments to SSO setup in RS

* DOC-5858 More adjustments to RS SSO setup

* DOC-5858 More RS SSO edits

* DOC-5858 Fixed in-page link in RS SSO

* DOC-5858 Updated RS SSO REST API reference and examples

* DOC-5858 Added test screenshot for RS SSO

* DOC-5858 Added screenshots for RS SSO

* DOC-5858 Added additional details/limitations for RS SSO

* Feedback update to fix sso permissions tables

* Feedback updates for SAML 2.0 and SP Entity ID port

* Feedback update for SP-initiated SSO

* Fixed SP metadata screenshot

* DOC-5858 Feedback update to add instructions to change the SP address

* RS: Added error report to Replica Of migration status REST API responses

* RS: Added API references for migrate_slots and cancel_migrate_slots

* DOC-5855 RS: 8.0.6 release notes

* DOC-6083 RS: Add user-defined modules during bootstrapping (#2526)

* Added known SSO issue and workaround to 8.0.6-50 December release notes

* DOC-5856 Added build number to RS 8.0.6 December release notes

* DOC-5856 Added checksums to RS 8.0.6 December release notes

* Feedback update: remove slot migration

* Updated build number for RS 8.0.6 Dec release

* Updated checksums for RS 8.0.6 Dec release

* Added cert and private_key fields to CRDB DB config object

* Typo fix (#2575)

* update build numbers + correction

* link to RS rn

* K8s: SSO - Asgard release (#2549)

* AI rough draft

* changes based off demo

---------

Co-authored-by: redisdocsapp[bot] <177626021+redisdocsapp[bot]@users.noreply.github.com>
Co-authored-by: Cameron Bates <cameron.bates@redis.com>
Co-authored-by: David Dougherty <david.dougherty@redis.com>
Co-authored-by: Cameron Bates <102550101+cmilesb@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Ignacio Sambrailo <105213331+IgnacioSambrailo@users.noreply.github.com>
Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com>
Co-authored-by: Jonathan Salomon <56251201+joniredis@users.noreply.github.com>
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.

4 participants