Re: [PATCH] util: eliminate erroneous VIR_WARNs in (eb|ip)tables.c

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

 



On 08/22/2012 05:18 PM, Eric Blake wrote:
> On 08/22/2012 02:07 PM, Laine Stump wrote:
>> Several VIR_DEBUG()'s were changed to VIR_WARN() while I was testing
>> the firewalld support patch, and I neglected to change them back
>> before I pushed.
>>
>> In the meantime I've decided that it would be useful to have them be
>> VIR_INFO(), just so there will be logged evidence of which method is
>> being used (firewall-cmd vs. (eb|ip)tables) without needing to crank
>> logging to 11. (at most this adds 2 lines to libvirtd's logs per
>> libvirtd start).
>> ---
>>  src/util/ebtables.c | 8 ++++----
>>  src/util/iptables.c | 8 ++++----
>>  2 files changed, 8 insertions(+), 8 deletions(-)
> ACK.
>
>> diff --git a/src/util/ebtables.c b/src/util/ebtables.c
>> index 1a78f89..3170ab0 100644
>> --- a/src/util/ebtables.c
>> +++ b/src/util/ebtables.c
>> @@ -55,19 +55,19 @@ virEbTablesOnceInit(void)
>>  {
>>      firewall_cmd_path = virFindFileInPath("firewall-cmd");
>>      if (!firewall_cmd_path) {
>> -        VIR_WARN("firewall-cmd not found on system. "
>> -                 "firewalld support disabled for ebtables.");
>> +        VIR_INFO("firewall-cmd not found on system. "
>> +                  "firewalld support disabled for ebtables.");
> Indentation looks off.

Right. Fixed and pushed. Thanks!

--
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]