Re: [et-mgmt-tools] [PATCH] [RESEND]Check the making domain's mac address(part 2)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Tatsuro Enokura wrote:
> Hi
> 
> Would you give me a comment on this patch?
> If not, please apply it.
> 
> Tatsuro Enokura wrote:
>> I made the patch the running domain's mac address check(changeset 125),
>> I make the patch in this time that the virt-install comand adds the
>> confirming message.
>>
>> The attached patch the following way:
>>
>>   1) Count that the making domian's MAC address is already use
>>      the exists domain's MAC address.
>>   2) Check the hots's  NIC MAC address.
>>   2) if count > 0 or , print confirmation message.
>>   3) if type yes then leave disk source as it is,
>>      else ask disk source again.
> 
> Signed-off-by: Tatsuro Enokura <fj7716hz@xxxxxxxxxxxxxxxxx>
> 
> Thanks,
> Tatsuro Enokura
> 

Hi. I had a look at the patch. The idea is OK, but there are two
problems: first, I'd rather you not use exceptions for flow control;
when you're testing for the mac address, if there's a problem, go
straight to the user check rather than going through the whole try:
except: sequence. Second problem is when I try it with a pre-existing
mac address from the command line for an inactive, I no longer get a
warning of a duplicate (which you set up in your last patch).

If you can check these things and resubmit, I'd be happy to take it.

Thanks,
--Hugh

-- 
Red Hat Virtualization Group http://redhat.com/virtualization
Hugh Brock           | virt-manager http://virt-manager.org
hbrock@xxxxxxxxxx    | virtualization library http://libvirt.org


[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux