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

Enhance tracing to hunt for restore bug #34

Merged
merged 1 commit into from
Aug 1, 2024
Merged

Conversation

ydirson
Copy link
Collaborator

@ydirson ydirson commented Jul 16, 2024

This is xenserver#157 for xcp-ng 8.3.0

@ydirson ydirson requested a review from stormi July 16, 2024 15:09
Copy link
Member

@stormi stormi left a comment

Choose a reason for hiding this comment

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

Nit: typos in commit message "enough info and messes up /dev/disk/by-id/ on which restore relies becAuse of /etc/xensource-inveNtory:"

The particular issue turned out to be that the nested UEFI host being
restored had several disks attached.  Since this is done as NVMe namespaces
of a single disk, all sharing the same ID, udev in dom0 does not have
enough info and messes up /dev/disk/by-id/ on which restore relies because
of /etc/xensource-inventory:

 lrwxrwxrwx 1 root root 13 Jul 15 14:28 nvme-QEMU_NVMe_Ctrl_nvme0 -> ../../nvme0n3
 lrwxrwxrwx 1 root root 15 Jul 15 14:28 nvme-QEMU_NVMe_Ctrl_nvme0-part1 -> ../../nvme0n2p1
 lrwxrwxrwx 1 root root 15 Jul 15 14:28 nvme-QEMU_NVMe_Ctrl_nvme0-part2 -> ../../nvme0n1p2

Only the addition of the `tool.dump()` call allowed me to connect the
"Failed to create root filesystem" error with that longstanding (in)famous
issue, but a few other additions were useful to get there.

Signed-off-by: Yann Dirson <[email protected]>
(cherry picked from commit 02d72e6, with
one additional safety guard that does not apply to master any more)
@ydirson ydirson force-pushed the xs8/enhance-traces branch from f1b2f28 to fd1a6ee Compare August 1, 2024 08:07
@ydirson ydirson merged commit fddab1c into 10.10.19-8.3 Aug 1, 2024
1 check passed
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.

2 participants