[node-patches] Change in ovirt-node[master]: firewall: Add some logging

[fabiand at fedoraproject.org]

Fabian Deutsch has uploaded a new change for review.

Change subject: firewall: Add some logging
......................................................................

firewall: Add some logging

Change-Id: I32e70035200ff5f092f9e34c1b47b926a730c68b
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/firewall.py
1 file changed, 6 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/10/19210/1

diff --git a/src/ovirt/node/utils/firewall.py b/src/ovirt/node/utils/firewall.py
index 5617cfa..c8ab7e4 100644
--- a/src/ovirt/node/utils/firewall.py
+++ b/src/ovirt/node/utils/firewall.py
@@ -20,8 +20,11 @@
 # also available at http://www.gnu.org/copyleft/gpl.html.
 
 import os
+from ovirt.node import log
 from ovirt.node.utils import process
 from glob import glob
+
+LOGGER = log.getLogger(__name__)
 
 PLUGIN_DIR = "/etc/ovirt-plugins.d/"
 PLUGIN_XML_OUT = "/etc/firewalld/services/node-plugin.xml"
@@ -70,10 +73,12 @@
 
 
 def process_plugins():
+    LOGGER.debug("Handling plugin firewall rules")
     for plugin in glob(PLUGIN_DIR + "*.firewall"):
         plugin_files.append(plugin)
 
     for f in plugin_files:
+        LOGGER.debug("Parsing firewall rules: %s" % f)
         with open(f) as i:
             conf = i.readlines()
         for line in conf:
@@ -82,6 +87,7 @@
                 fw_conf.append((port, proto))
 
     for i in fw_conf:
+        LOGGER.debug("Opening firewall ports: %s" % str(i))
         port, proto = i
         if is_firewalld():
             setup_firewalld(port, proto)


-- 
To view, visit http://gerrit.ovirt.org/19210
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I32e70035200ff5f092f9e34c1b47b926a730c68b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>