Skip to content

WiP: rework migrate to XCP-ng documentation #346

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
Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

WiP: rework migrate to XCP-ng documentation #346

wants to merge 16 commits into from

Conversation

AtaxyaNetwork
Copy link
Contributor

Work in progress.

I added Prerequisites and Troubleshooting section, and rearrange/rewrite the other

Before submitting the pull request, you must agree with the following statements by checking both boxes with a 'x'.

  • "I accept that my contribution is placed under the CC BY-SA 2.0 license [1]."
  • "My contribution complies with the Developer Certificate of Origin [2]."

[1] https://creativecommons.org/licenses/by-sa/2.0/
[2] https://docs.xcp-ng.org/project/contributing/#developer-certificate-of-origin-dco

thomas-dkmt
thomas-dkmt reviewed May 26, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed May 26, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 3, 2025
View reviewed changes
AtaxyaNetwork and others added 5 commits June 16, 2025 15:06
@AtaxyaNetwork
WiP: rework migrate to XCP-ng documentation
e7e87d9 
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
083e9d4 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange  <[email protected]>
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
920a336 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange  <[email protected]>
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
a486499 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange  <[email protected]>
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
bebaf2b 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange  <[email protected]>
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
AtaxyaNetwork and others added 7 commits June 16, 2025 15:08
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
62adcb4 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
a65b7c5 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
f9be311 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
0b9505a 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
d7b2045 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
3d90e12 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
@AtaxyaNetwork
Changes suggested by Thomas
a3d4963 
Signed-off-by: Cécile Morange <[email protected]>
Signed-off-by: Cécile - AtaxyaNetwork <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
c3781c5 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork
Copy link
Contributor Author

All suggestion are reviewed !

thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
@stormi
Copy link
Member

stormi commented Jun 16, 2025

All suggestion are reviewed !

But it's still marked WIP. Main reason for which I haven't reviewed yet.

thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
thomas-dkmt
thomas-dkmt reviewed Jun 16, 2025
View reviewed changes
AtaxyaNetwork and others added 2 commits June 17, 2025 11:39
@AtaxyaNetwork @thomas-dkmt
Update docs/installation/migrate-to-xcp-ng.md
8ea7c6a 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork @thomas-dkmt
Apply suggestions from code review
e0423f3 
Co-authored-by: Thomas Moraine <[email protected]>
Signed-off-by: Cécile Morange <[email protected]>
@AtaxyaNetwork
Copy link
Contributor Author

@stormi it marked it as WiP because I didn't want someone merging it without having some proofreading and opinion from other people.

@stormi
Copy link
Member

stormi commented Jun 17, 2025

@stormi it marked it as WiP because I didn't want someone merging it without having some proofreading and opinion from other people.

Just ask people for review, we'll not merge until it's reviewed.

In our processes, WIP usually means "I still have work to do on this". So I usually wait before I review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stormi stormi stormi left review comments

@thomas-dkmt thomas-dkmt thomas-dkmt approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@AtaxyaNetwork @stormi @thomas-dkmt