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.

Sign up for GitHub

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

Jump to bottom

MTV-1433: Creating source and destination providers, separated by provider #567

Merged
merged 1 commit into from
Feb 20, 2025
Merged

MTV-1433: Creating source and destination providers, separated by provider #567

merged 1 commit into from
Feb 20, 2025

Conversation

RichardHoch
Copy link
Collaborator

@RichardHoch RichardHoch commented Oct 7, 2024 •
edited
Loading

MTV 2.8

Partially resolves https://issues.redhat.com/browse/MTV-1433

Previews:

In this PR, I have grouped the procedures for creating source and a destination provider per vendor. You can ignore creating-migration-plans-2.8.adoc. I work with it in #585 .

@RichardHoch RichardHoch changed the title WIP draft WIP -- Restructuring Section 4 of the MTV user guide Oct 8, 2024
@RichardHoch RichardHoch marked this pull request as ready for review October 8, 2024 13:34
@RichardHoch RichardHoch changed the title WIP -- Restructuring Section 4 of the MTV user guide MTV-1433: Creating source and destination providers, separated by provider Nov 26, 2024
@RichardHoch RichardHoch force-pushed the WIP_section_4 branch 2 times, most recently from e2a9d42 to e0b263d Compare November 26, 2024 12:10
shdeshpa07
shdeshpa07 reviewed Nov 27, 2024
View reviewed changes
@RichardHoch
Copy link
Collaborator Author

RichardHoch commented Nov 27, 2024 •
edited
Loading

@sgratch Please review this PR -- just the sections that have previews. You can ignore the changes to the file creating-migration-plans-2.8.adoc. I work with it in #585 . @mnecas --if you have time, please also review.

@RichardHoch RichardHoch requested a review from mnecas December 3, 2024 15:53
@RichardHoch
Copy link
Collaborator Author

@mnecas I made the changes you suggested -- the discussion of warm/cold migration is in #585.

Please review this PR again and also #585. Thanks.

@RichardHoch
Copy link
Collaborator Author

@mnecas @fabiand Please review this PR -- I made the changes Martin suggested.

@RichardHoch
Copy link
Collaborator Author

@anarnold97 I made the changes you suggested. Please review this PR again. Thanks.

@anarnold97 anarnold97 merged commit 081da9d into kubev2v:main Feb 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shdeshpa07 shdeshpa07 shdeshpa07 left review comments

@anarnold97 anarnold97 anarnold97 approved these changes

@mnecas mnecas Awaiting requested review from mnecas

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@RichardHoch @mnecas @anarnold97 @shdeshpa07