RDSC-3785: Document known issue for RDI upgrade from < 1.8.0 to >= 1.8.0 #1783
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
Contributor
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds documentation for a known high-availability upgrade issue when moving RDI from versions below 1.8.0 to 1.8.0 or higher, including symptoms and a manual-cluster-ID workaround.
- Added a Known Issues section in the RDI 1.8.0 release notes describing the HA lock upgrade problem and workaround.
- Inserted a warning block in the upgrade guide with the same issue details and remediation steps.
- Specified how to edit the configmap to set a unique
RDI_CLUSTER_ID.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| content/.../release-notes/rdi-1-8-0.md | Documented HA upgrade issue, symptoms, and configmap workaround |
| content/.../installation/upgrade.md | Added warning for VM HA upgrade issue and manual cluster ID workaround |
Comments suppressed due to low confidence (1)
content/integrate/redis-data-integration/release-notes/rdi-1-8-0.md:95
- [nitpick] In this paragraph, mix of "instances" and later "clusters" may confuse readers. Consider using a single term (e.g., always "instances").
When upgrading from RDI < 1.8.0 to RDI >= 1.8.0 in an HA setup, both RDI instances may incorrectly consider themselves active after the upgrade. This occurs because the upgrade process doesn't update the `rdi:ha:lock` value from the legacy `cluster-1` identifier, causing both clusters to assume they are the active cluster.
content/integrate/redis-data-integration/installation/upgrade.md
Outdated
Show resolved
Hide resolved
content/integrate/redis-data-integration/installation/upgrade.md
Outdated
Show resolved
Hide resolved
andy-stark-redis
approved these changes
Jul 1, 2025
Contributor
andy-stark-redis
left a comment
andy-stark-redis
left a comment
There was a problem hiding this comment.
One minor suggestion (Copilot already suggested the other one I was going to add), but otherwise LGTM.
content/integrate/redis-data-integration/release-notes/rdi-1-8-0.md
Outdated
Show resolved
Hide resolved
stoyanr
reviewed
Jul 1, 2025
content/integrate/redis-data-integration/installation/upgrade.md
Outdated
Show resolved
Hide resolved
yaronp68
requested changes
Jul 1, 2025
Collaborator
yaronp68
left a comment
yaronp68
left a comment
There was a problem hiding this comment.
please merge stoyan propsals
Co-authored-by: Stoyan Rachev <stoyanr@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
andy-stark-redis
approved these changes
Jul 1, 2025
andy-stark-redis
deleted the
RDSC-3785-add-note-for-rdi-1.8.0-upgrade-known-issue
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See https://redislabs.atlassian.net/browse/RDSC-3785