[virt-tools-list] [PATCH 1/2] virt install: Remove check for absolute path for fs target

Deepak C Shetty deepakcs at linux.vnet.ibm.com
Fri Dec 23 08:41:19 UTC 2011


On 12/23/2011 01:21 AM, Cole Robinson wrote:
> On 12/21/2011 12:52 AM, Deepak C Shetty wrote:
>> In case of qemu for default fs type (mount) target is not
>> actually a directory, it is merely a arbitrary string tag
>> that is exported to the guest as a hint for where to mount.
>>
>> Signed-off-by: Deepak C Shetty<deepakcs at linux.vnet.ibm.com>
>> ---
>>
>>   virtinst/VirtualFilesystem.py |   12 +++++++++++-
>>   1 files changed, 11 insertions(+), 1 deletions(-)
>>
>> diff --git a/virtinst/VirtualFilesystem.py b/virtinst/VirtualFilesystem.py
>> index 5732a9e..a74d009 100644
>> --- a/virtinst/VirtualFilesystem.py
>> +++ b/virtinst/VirtualFilesystem.py
>> @@ -116,7 +116,17 @@ class VirtualFilesystem(VirtualDevice.VirtualDevice):
>>       def _get_target(self):
>>           return self._target
>>       def _set_target(self, val):
>> -        if not os.path.isabs(val):
>> +        if self.conn:
>> +            is_qemu = self.is_qemu()
>> +
> If self.conn is None, is_qemu will be referenced before assignment.
>
> Thanks,
> Cole
Posted v2 of this patch @ 
https://www.redhat.com/archives/virt-tools-list/2011-December/msg00083.html




More information about the virt-tools-list mailing list