-
Notifications
You must be signed in to change notification settings - Fork 294
K8s: Redis Flex - Asgard release #2453
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
dwdougherty
left a comment
There was a problem hiding this 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. |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 | ||
| ``` |
There was a problem hiding this comment.
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)
There was a problem hiding this comment.
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.
dwdougherty
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
|
Content approved by Iren via slack. We might have to clarify a few things later, but for initial release, it's all accurate. |
* 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>
No description provided.