Skip to content
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

Pd 1011 update core to scale migration articles #2499

Merged
merged 15 commits into from
Feb 29, 2024

Conversation

micjohnson777
Copy link
Contributor

This PR updates content in CORE to SCALE migration articles and snippets.
It deletes one snippet only used in one article (not found in a search of it in the repo) and moves that content into the MigratePrep..md article.
It reorganizes and improves the content in the MigratePrep.md article to emphasize the areas users need to address before they migrate/update to or clean install SCALE.

It adds a missing step in the modified MigratingFromCORE.md article.

It adds information about the force option when updating to Dragonfish when deprecated services are in use to the Release Notes and changes the word reinstall to install in the upgrade path mermaid.

Thanks for contributing to TrueNAS documentation! By opening a Pull Request, you're acknowledging that your changes will be distributed under the Creative Commons 4.0 license.

This PR updates the content in the SCALE/GettingStarted/Migrate/MigratePrep.md article by reorganizing and improving the content in the preparation steps. It adds additional information to the Deprecated Services section introduction and the S3 Service content.
It updates the MigratingFromCORE.md article by adding the missing step to upload the config file and verify the rest of the settings, beginning with network settings, then checking pools, and finally all other settings.
It deletes the MigrateCoreServicesToCobia.md snippet that was not used in other article (per search result) and moved the content into the preparation article.
Improves the content in MigrateCOREHAtoSCALEHA.md.
Updates the mermaid for the upgrade path in the SCALEReleaseNotes.md and adds information on the new force option for deprecated services.
@micjohnson777 micjohnson777 requested a review from a team as a code owner February 22, 2024 19:45
Mrt134
Mrt134 previously requested changes Feb 26, 2024
Copy link
Collaborator

@Mrt134 Mrt134 left a comment

Choose a reason for hiding this comment

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

Number of comments from reviewing the migration process with an HA box.

content/SCALE/GettingStarted/Migrate/MigratePrep.md Outdated Show resolved Hide resolved
content/SCALE/GettingStarted/Migrate/MigratePrep.md Outdated Show resolved Hide resolved
content/SCALE/GettingStarted/Migrate/MigratePrep.md Outdated Show resolved Hide resolved
content/SCALE/GettingStarted/Migrate/MigratePrep.md Outdated Show resolved Hide resolved
content/SCALE/GettingStarted/Migrate/MigratePrep.md Outdated Show resolved Hide resolved
content/SCALE/GettingStarted/Migrate/MigratePrep.md Outdated Show resolved Hide resolved
content/SCALE/GettingStarted/SCALEReleaseNotes.md Outdated Show resolved Hide resolved
content/_includes/RootToAdminUserAccount.md Outdated Show resolved Hide resolved
@Mrt134 Mrt134 added the Review (TM) A technical writer is reviewing the PR. label Feb 26, 2024
micjohnson777 and others added 9 commits February 27, 2024 13:05
Co-authored-by: Tim Moore <Mrt134@users.noreply.github.com>
Co-authored-by: Tim Moore <Mrt134@users.noreply.github.com>
Co-authored-by: Tim Moore <Mrt134@users.noreply.github.com>
Co-authored-by: Tim Moore <Mrt134@users.noreply.github.com>
Co-authored-by: Tim Moore <Mrt134@users.noreply.github.com>
Co-authored-by: Tim Moore <Mrt134@users.noreply.github.com>
This commit makes changes suggested in the PR review.

It creates a new snippet AddAdminUserAccount.md for the content originally in the ManageLocalUsersSCALE.md article and adds that snippet to that article and the Migrating from CORE article.

It changes the numbered sequence in the MigratePrep.md article to make the GELI step 2, and combines steps 6 and 7 into one on backing up data.
micjohnson777 and others added 4 commits February 28, 2024 08:27
Simplify the HA article down to a simple bullet of process requirements and encouragement to consult iX support.
…Only

Update PR #2499 with Enterprise-specific changes
Copy link
Collaborator

@Mrt134 Mrt134 left a comment

Choose a reason for hiding this comment

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

Additional local build test and review looks good. We'll need to address the scale CLI text at a future point.

@Mrt134 Mrt134 merged commit 72f60fa into master Feb 29, 2024
3 checks passed
@Mrt134 Mrt134 deleted the PD-1011-Update-CORE-to-SCALE-Migration-Articles branch February 29, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review (TM) A technical writer is reviewing the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants