Deepti B Kalakeri wrote:
Kaitlin Rupert wrote:
>> - else:
>> - status = FAIL
>> + status = verify_device_values(assoc_eafp_info, CCName, +
>> eafp_values, virt='Xen')
>> +
>
> There's an extra param included here. CCName isn't needed.
>
The definition of the verify_device_values() in logicaldevices.py takes
the 4 arguments and hence all the 4 which includes CCName is required.
The following is the definition in logicaldevices.py :
def verify_device_values(assoc_info, CCName, list_values, virt='Xen'):
dev_cnames = ['LogicalDisk', 'Memory', 'NetworkPort',
'Processor']
for i in range(len(dev_cnames)):
dev_cnames[i] = get_typed_class(virt, dev_cnames[i])
Yes, my mistake here. I had applied this patch before the patch with
the logicaldevices fix. This patch looks good!
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com