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

redact LMS docs #18808

Merged
merged 3 commits into from
Aug 12, 2024
Merged

redact LMS docs #18808

merged 3 commits into from
Aug 12, 2024

Conversation

taroface
Copy link
Contributor

@taroface taroface commented Aug 8, 2024

Removing MOLT LMS content due to its private preview status.

  • Removed the LMS page from the docs repo.
  • Removed mentions of LMS (except those already commented-out / hidden) in the docs, including MOLT changelog.
  • Removed LMS from MOLT tooling diagram.
  • Added redirect from LMS doc to MOLT Overview page.

Note that LMS is still mentioned in the 23.2.0 release notes (when it was released in public preview), but the link will redirect to the MOLT Overview.

Copy link

github-actions bot commented Aug 8, 2024

Files changed:

Copy link

netlify bot commented Aug 8, 2024

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 1ecb81b
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-api-docs/deploys/66ba477d058e1f0008e5d4c0

Copy link

netlify bot commented Aug 8, 2024

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 1ecb81b
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-interactivetutorials-docs/deploys/66ba477d0aa2ac0008c43198

Copy link

netlify bot commented Aug 8, 2024

Netlify Preview

Name Link
🔨 Latest commit 1ecb81b
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/66ba477d9837ad0008daf944
😎 Deploy Preview https://deploy-preview-18808--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@awoods187 awoods187 left a comment

Choose a reason for hiding this comment

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

Thanks for doing this!

1. Use [MOLT Fetch]({% link molt/molt-fetch.md %}) to move the source data to CockroachDB. Use the tool to [**replicate ongoing changes**]({% link molt/molt-fetch.md %}#replication) after it performs the initial load of data into CockroachDB.
1. As the data is migrating, use [MOLT Verify]({% link molt/molt-verify.md %}) to validate the consistency of the data between the source database and CockroachDB.
1. After nearly all data from your source database has been moved to CockroachDB (for example, with a <1-second delay or <1000 rows), use MOLT LMS to begin a [*consistent cutover*]({% link molt/live-migration-service.md %}#consistent-cutover) and stop application traffic to your source database. **This begins downtime.**
1. After nearly all data from your source database has been moved to CockroachDB (for example, with a <1-second delay or <1000 rows), use MOLT LMS to begin a *consistent cutover* and stop application traffic to your source database. **This begins downtime.**
Copy link
Contributor

Choose a reason for hiding this comment

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

LMSis still here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed, thanks!

Choose a reason for hiding this comment

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

I still see MOLT LMS here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just fixed all 3 of these instances a few moments after you commented -- PTAL!

Choose a reason for hiding this comment

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

LGTM now. Thanks!

1. Use [MOLT Fetch]({% link molt/molt-fetch.md %}) to move the source data to CockroachDB. Use the tool to [**replicate ongoing changes**]({% link molt/molt-fetch.md %}#replication) after it performs the initial load of data into CockroachDB.
1. As the data is migrating, use [MOLT Verify]({% link molt/molt-verify.md %}) to validate the consistency of the data between the source database and CockroachDB.
1. After nearly all data from your source database has been moved to CockroachDB (for example, with a <1-second delay or <1000 rows), use MOLT LMS to begin a [*consistent cutover*]({% link molt/live-migration-service.md %}#consistent-cutover) and stop application traffic to your source database. **This begins downtime.**
1. After nearly all data from your source database has been moved to CockroachDB (for example, with a <1-second delay or <1000 rows), use MOLT LMS to begin a *consistent cutover* and stop application traffic to your source database. **This begins downtime.**
Copy link
Contributor

Choose a reason for hiding this comment

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

LMS is here too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

Choose a reason for hiding this comment

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

Same here

Copy link
Contributor

@kathancox kathancox left a comment

Choose a reason for hiding this comment

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

LGTM

@taroface
Copy link
Contributor Author

LGTM

TFTR!

@taroface taroface enabled auto-merge (squash) August 12, 2024 17:34
@taroface taroface merged commit 60df97d into main Aug 12, 2024
7 checks passed
@taroface taroface deleted the lms-private-preview branch August 12, 2024 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants