Skip to content

Conversation

KOTungseth
Copy link
Contributor

@KOTungseth KOTungseth commented Aug 12, 2025

@KOTungseth KOTungseth self-assigned this Aug 12, 2025
@KOTungseth KOTungseth added the Team:Ingest Issues owned by the Ingest Docs Team label Aug 12, 2025
@KOTungseth KOTungseth requested review from a team and LolloneS August 12, 2025 20:40
Copy link

github-actions bot commented Aug 12, 2025

@KOTungseth KOTungseth marked this pull request as ready for review August 12, 2025 21:24
@KOTungseth KOTungseth requested a review from a team as a code owner August 12, 2025 21:24
Copy link
Contributor

@LolloneS LolloneS left a comment

Choose a reason for hiding this comment

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

Looks great, and thank you for opening the PR! :)

…s-with-minimal-downtime.md

Co-authored-by: Visha Angelova <[email protected]>
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

mostly comments around adding interlinking and appropriately contextualizing this doc in the larger docs set

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

I added a reference in the parent page: KOTungseth@2ee8c59

@@ -0,0 +1,77 @@
---
navigation_title: Migrate Elasticsearch data with minimal downtime
Copy link
Collaborator

Choose a reason for hiding this comment

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

because this is in the migration section, we could try:

Suggested change
navigation_title: Migrate Elasticsearch data with minimal downtime
navigation_title: Minimize downtime with incremental snapshots

Copy link
Contributor

Choose a reason for hiding this comment

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

@shainaraskas I think the goal of this article is to provide an answer to all our customers asking for "zero-downtime" migrations. Incremental snapshots are the technology enabling it, not the end goal. I am totally ok with changing the title but I would like to keep the "downtime" part in there :)

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 did a keyword search, and to optimize this page for SEO, we need to keep Elasticsearch in the title.

Copy link
Collaborator

Choose a reason for hiding this comment

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

this is the nav title, not the page title, so we could simplify this so it's easier to read in the sidebar.

Copy link
Contributor

Choose a reason for hiding this comment

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

Got it! I think for the nav title @shainaraskas's suggestion makes sense as we are already under the "migrate" section.

* **Custom integrations** – Some custom integrations that directly use the {{es}} API can require additional handling during the cutover from the old cluster to the new cluster.
* **Resource usage** – Initial and incremental snapshot and restore operations can be resource-intensive, potentially affecting cluster performance.

## Additional topics [additional-incremental-snapshot-topics]
Copy link
Collaborator

Choose a reason for hiding this comment

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

consider integrating most of these into the body.

LolloneS and others added 12 commits August 13, 2025 21:35
…s-with-minimal-downtime.md

Co-authored-by: shainaraskas <[email protected]>
…s-with-minimal-downtime.md

Co-authored-by: shainaraskas <[email protected]>
…s-with-minimal-downtime.md

Co-authored-by: shainaraskas <[email protected]>
…s-with-minimal-downtime.md

Co-authored-by: Visha Angelova <[email protected]>
…s-with-minimal-downtime.md

Co-authored-by: Visha Angelova <[email protected]>
…s-with-minimal-downtime.md

Co-authored-by: shainaraskas <[email protected]>
…s-with-minimal-downtime.md

Co-authored-by: shainaraskas <[email protected]>
@KOTungseth KOTungseth merged commit 3458e90 into elastic:main Aug 19, 2025
7 checks passed
@KOTungseth KOTungseth deleted the manage-data/zero-downtime-migrate branch August 19, 2025 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Ingest Issues owned by the Ingest Docs Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants