You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@vcl.apache.org by ar...@apache.org on 2016/07/07 18:49:29 UTC
svn commit: r1751825 - /vcl/trunk/managementnode/lib/VCL/Module/OS/Linux.pm
Author: arkurth
Date: Thu Jul 7 18:49:29 2016
New Revision: 1751825
URL: http://svn.apache.org/viewvc?rev=1751825&view=rev
Log:
Changed notify message in Linux.pm::get_firewall_configuration and disable_firewall_port from warning to debug if iptables service doesn't exist. It was causing many warnings for RH/CentOS 7 images.
Modified:
vcl/trunk/managementnode/lib/VCL/Module/OS/Linux.pm
Modified: vcl/trunk/managementnode/lib/VCL/Module/OS/Linux.pm
URL: http://svn.apache.org/viewvc/vcl/trunk/managementnode/lib/VCL/Module/OS/Linux.pm?rev=1751825&r1=1751824&r2=1751825&view=diff
==============================================================================
--- vcl/trunk/managementnode/lib/VCL/Module/OS/Linux.pm (original)
+++ vcl/trunk/managementnode/lib/VCL/Module/OS/Linux.pm Thu Jul 7 18:49:29 2016
@@ -4219,8 +4219,8 @@ sub disable_firewall_port {
# Check to see if this distro has iptables
# If not return 1 so it does not fail
- if (!($self->service_exists("iptables"))) {
- notify($ERRORS{'WARNING'}, 0, "iptables does not exist on this OS");
+ if (!$self->service_exists("iptables")) {
+ notify($ERRORS{'DEBUG'}, 0, "iptables does not exist on this OS");
return 1;
}
@@ -4605,9 +4605,9 @@ sub get_firewall_configuration {
my $computer_node_name = $self->data->get_computer_node_name();
# Check to see if iptables service exists
- if (!($self->service_exists("iptables"))) {
- notify($ERRORS{'WARNING'}, 0, "iptables does not exist on this OS");
- return 1;
+ if (!$self->service_exists("iptables")) {
+ notify($ERRORS{'DEBUG'}, 0, "iptables does not exist on this OS");
+ return {};
}
my $port_command = "iptables --line-number -n -L";