[Engine-patches] Change in ovirt-engine[master]: engine: Cleanup Logging in MacPoolManager

[msalem]

Muli Salem has uploaded a new change for review.

Change subject: engine: Cleanup Logging in MacPoolManager
......................................................................

engine: Cleanup Logging in MacPoolManager

This patch cleans up the logging in MacPoolManager.

Change-Id: If7d47189fe69effa90ce97ba0de83c6162a11e7f
Signed-off-by: Muli Salem <msa...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java
1 file changed, 1 insertion(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/33/10833/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java
index 32171dc..668ffa4 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java
@@ -140,7 +140,6 @@
 
     public String allocateNewMac() {
         String mac = null;
-        log.info("MacPoolManager::allocateNewMac entered");
         lockObj.writeLock().lock();
         try {
             if (!initialized) {
@@ -156,7 +155,6 @@
         } finally {
             lockObj.writeLock().unlock();
         }
-        log.infoFormat("MacPoolManager::allocateNewMac allocated mac = '{0}", 
mac);
         return mac;
     }
 
@@ -172,7 +170,6 @@
     }
 
     public int getAvailableMacsCount() {
-        log.infoFormat("MacPoolManager::getAvailableMacsCount - entered");
         lockObj.readLock().lock();
         try {
             if (!initialized) {
@@ -181,7 +178,7 @@
             }
 
             int availableMacsSize = availableMacs.size();
-            log.infoFormat("MacPoolManager:: AvailableMacsCount = {0}", 
availableMacsSize);
+            log.debugFormat("Number of available Mac addresses = {0}", 
availableMacsSize);
             return availableMacsSize;
         } finally {
             lockObj.readLock().unlock();
@@ -189,7 +186,6 @@
     }
 
     public void freeMac(String mac) {
-        log.infoFormat("MacPoolManager::freeMac(mac = '{0}') - entered", mac);
         lockObj.writeLock().lock();
         try {
             if (!initialized) {
@@ -259,7 +255,6 @@
     }
 
     public void freeMacs(List<String> macs) {
-        log.info("MacPoolManager::freeMacs - entered");
         lockObj.writeLock().lock();
         try {
             if (!initialized) {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If7d47189fe69effa90ce97ba0de83c6162a11e7f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <msa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches