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

MTV-1852: Adding a Cold & Warm migration section #612

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

anarnold97
Copy link
Collaborator

@anarnold97 anarnold97 commented Dec 28, 2024

@anarnold97 anarnold97 requested review from RichardHoch, fabiand, mnecas and dgur and removed request for RichardHoch December 28, 2024 13:29
@anarnold97
Copy link
Collaborator Author

anarnold97 commented Dec 28, 2024

@dgur , @fabiand , @mnecas - here is the first draft for MTV-1852 Add Cold VS Warm migration Section into the MTV docs.

Please could you review and suggest any further content and any changes?

Please could I also as for some help improving this image:

image

I think a detailed infographic comparing the workflows of cold and warm migration would really help, but I am unsure where to begin. Even a rough napkin draft would be great as I can easily tidy it.

Thanks

@dgur
Copy link
Collaborator

dgur commented Dec 29, 2024

@anarnold97 thank you for the quick progress with this task!
I request you to pause a bit, as the team needs to update the slides with a couple of the latest Warm Migration updates found in the latest MTV 2.7

We, (Mordechai and David) Will update you and explain the changes.
Then I suggest instead of creating new separate section for it "2.7. MTV cold migration and warm migration introduction"
We work together to combine the latest s updates into the existing section "[1.1.3. Advantages and disadvantages of cold and warm migrations]

Mainly focusing on updating the comparison Table and the conclusion - when to use what.

Signed-off-by: A.Arnold <[email protected]>
@dgur
Copy link
Collaborator

dgur commented Jan 14, 2025

new table Table "2.1. Cold and warm migration compared" overlaps with tables at "2.2.3. Advantages and disadvantages of cold and warm migrations"
I believe all those needs to be combined to one under 2.2.3

@duduvaa
Copy link

duduvaa commented Jan 15, 2025

@anarnold97 ,
table 2.1 , warm , VM downtime
The VM is shutdown during the cutover step as described in slide 4

@dgur
Copy link
Collaborator

dgur commented Jan 15, 2025

new table Table "2.1. Cold and warm migration compared" overlaps with tables at "2.2.3. Advantages and disadvantages of cold and warm migrations" I believe all those needs to be combined to one under 2.2.3

Do we have an updated preview of it?

Signed-off-by: A.Arnold <[email protected]>
@anarnold97
Copy link
Collaborator Author

@dgur && @duduvaa - PR and Preview updated

@duduvaa
Copy link

duduvaa commented Feb 3, 2025

LGTM

@dgur
Copy link
Collaborator

dgur commented Feb 3, 2025

We have "Parallelism" in Table 2.1. and also the same in Table 2.2.
I Suggest to combine the 2 tables to one big table - aka combine section 2.2.3.1 and 2.2.3.2

Signed-off-by: A.Arnold <[email protected]>
@anarnold97
Copy link
Collaborator Author

@dgur - all three tables merged into one

However, the preview is hitting RH security, so I have deleted the preview until i have time to set up as a false positive. Please review the PR, the table is as follows

image

With the following taxonomy

image

Sorry for the rush but I need to get this pushed to merge as soon as possible, as we have other releases early next week that we need to work on

Thanks

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.

3 participants