Re: [PATCH 1/3] Only check for IP forwarding, do not enable it

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

 



On 10/15/2012 10:54 AM, Michal Privoznik wrote:
> On 15.10.2012 12:26, Benjamin Cama wrote:

>>  
>>  static int
>>  networkSetIPv6Sysctls(virNetworkObjPtr network)
>> @@ -2140,11 +2172,9 @@ networkStartNetworkVirtual(struct network_driver *driver,
>>      if (virNetDevSetOnline(network->def->bridge, 1) < 0)
>>          goto err2;
>>  
>> -    /* If forwardType != NONE, turn on global IP forwarding */
>> +    /* If forwardType != NONE, check for IP forwarding */
>>      if (network->def->forwardType != VIR_NETWORK_FORWARD_NONE &&
>> -        networkEnableIpForwarding(v4present, v6present) < 0) {
>> -        virReportSystemError(errno, "%s",
>> -                             _("failed to enable IP forwarding"));
>> +        networkCheckIpForwarding(v4present, v6present) < 0) {
>>          goto err3;
>>      }
>>  
>>
> 
> Well, I am not sure if we can do this. What would happen if some of our
> users rely on this already? I mean, it's there since ages.
> 
> Michal

Indeed this kinda defeats the purpose of the default virtual network that
should 'just work' out of the box. Maybe we could add some libvirtd.conf
option to enable this check-if-set behavior, but we can't change the default here.

- Cole

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]