[Libguestfs] [PATCH] ESX: Fix storage URL if storage has a snapshot

Matthew Booth mbooth at redhat.com
Mon Jun 7 12:48:41 UTC 2010


On 07/06/10 13:45, Richard W.M. Jones wrote:
> On Mon, Jun 07, 2010 at 01:37:40PM +0100, Matthew Booth wrote:
>> If an ESX guest has a snapshot, the path the libvirt driver gives us will look
>> like:
>>    [yellow:storage1] RHEL4-X/RHEL4-X-000003.vmdk
>> instead of:
>>    [yellow:storage1] RHEL4-X/RHEL4-X.vmdk
>> The current path mangling code does take this into account.
>>
>
> As you say it's a flakey heuristic, but there doesn't seem to be any
> danger in it.  So ACK.
>
> BTW your description here:
>
>> This change makes it use the current mechanism first, [...]
>
> doesn't seem to match what the patch does.  Surely the patch
> is trying the original name first?

That's what I meant. I'll update the wording when I add BZ numbers to 
the commit messages.

Matt
-- 
Matthew Booth, RHCA, RHCSS
Red Hat Engineering, Virtualisation Team

M:       +44 (0)7977 267231
GPG ID:  D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490




More information about the Libguestfs mailing list