search for: c15efb2

...to be hidden, resulting in the famous and uninformative: virt-v2v: Died at /usr/bin/virt-p2v-server line 404, <> line 7. --- p2v/server/virt-p2v-server.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/p2v/server/virt-p2v-server.pl b/p2v/server/virt-p2v-server.pl index c15efb2..0121ab2 100755 --- a/p2v/server/virt-p2v-server.pl +++ b/p2v/server/virt-p2v-server.pl @@ -405,7 +405,7 @@ sub convert my $err = $@; $g->close() if defined($g); - die($@); + die($err); } p2v_return_ok(); -- 1.8.3.1

...$err = $@ and then not use $err. Die with $err. It's possible something added here could overwrite $@ at some later time. --- p2v/server/virt-p2v-server.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/p2v/server/virt-p2v-server.pl b/p2v/server/virt-p2v-server.pl index c15efb2..0121ab2 100755 --- a/p2v/server/virt-p2v-server.pl +++ b/p2v/server/virt-p2v-server.pl @@ -405,7 +405,7 @@ sub convert my $err = $@; $g->close() if defined($g); - die($@); + die($err); } p2v_return_ok(); -- 1.8.3.1

Just spent a little time poking around w/ p2v and am attaching the following pseudo-code for consideration. Note the patches aren't commit ready yet, just looking for thoughts - The first is a fix to get client image building process working on F17. It seems the version of ksflatten there didn't expand the nested %includes which was causing errors (also --interpreter image-minimizer

Figure I'd share the latest revision to p2v I had regarding $subject. The patch now applies against HEAD and compiles fine. I was able to update the package on the client and run virt-p2v, though have yet to do a full end-to-end verification Most likely a little more work is needed to tidy up some edge cases and fully flush things out, but the majority of the work should be in place. On the