-
Notifications
You must be signed in to change notification settings - Fork 57
CCSINTL-471: 1st set of premigration updates to RPM install and upgrade docs #4231
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
[role="_additional-resources"] | ||
.Additional resources | ||
See {LinkGettingStarted} for post installation instructions. | ||
* {LinkGettingStarted} |
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 link sends a 404 error from the preview I built locally.
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.
@jonquilwilliams good catch. For some weird reason the attribute usage is giving a 404 error. Corrected it, and used the URL instead.
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.
Verified.
|
||
* 2.4 services will only include {ControllerName} and {HubName}. | ||
* 2.5 parts will always include {EDAName} and the unified UI ({Gateway}). | ||
* Combining install methods for these topologies is not supported. |
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.
suggest/ install -> installation (IBM word list - adjective/noun usage)
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.
Made the edit.
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.
Verified.
== Upgrade considerations | ||
|
||
* You must maintain two separate inventory files: one for the 2.4 services and one for the 2.5 services. | ||
* You must maintain two separate "installations" within this scenario: one for the 2.4 services and one for the 2.5 services. |
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 nit and might just be my personal preference, but I think you can remove all of the quotation marks in this section. I think it's clear enough without them. It's up to you though.
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.
Agreed, made the edit.
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.
Verified.
|
||
. Run `setup.sh` | ||
+ | ||
The installer upgrades {ControllerName} and {HubName} from 2.4 to {PlatformNameShort} 2.5.latest, {EDAName} and the unified UI ({Gateway}) from the fresh install of 2.5 to the latest version of 2.5, and connects {ControllerName} and {HubName} properly with the unified UI ({Gateway}) node to initialize the unified experience. |
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.
I think there should be a serial comma between these: "{EDAName}, and the unified UI ({Gateway})"
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.
Made the edit.
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.
Verified.
.Verification | ||
|
||
* Verify that everything has upgraded to 2.5 and is working properly in one of two ways: | ||
** performing an SSH to {ControllerName} and {EDAName}. |
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.
Suggest init cap: performing -> Performing
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.
Changed to 'Perform'
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.
Verified.
Verified updates that were suggested during thorough review from @jonquilwilliams . LGTM |
…de docs (ansible#4231) * CCSINTL-471: 1st set of premigration updates to RPM install and upgrade docs * CCSINTL-471: 2nd set of premigration updates * CCSINTL-471: Incorporated 3rd set of premigration check updates * CCSINTL-471: Incorporated peer feedback
…de docs (ansible#4231) * CCSINTL-471: 1st set of premigration updates to RPM install and upgrade docs * CCSINTL-471: 2nd set of premigration updates * CCSINTL-471: Incorporated 3rd set of premigration check updates * CCSINTL-471: Incorporated peer feedback
This PR addresses JIRA https://issues.redhat.com/browse/CCSINTL-471.
Changes made: