Change in vdsm[master]: migration: log the convergence schedule only if provided

tjelinek at redhat.com tjelinek at redhat.com
Thu May 12 11:19:25 UTC 2016


Tomas Jelinek has uploaded a new change for review.

Change subject: migration: log the convergence schedule only if provided
......................................................................

migration: log the convergence schedule only if provided

No need to flood the logs with logs which don't give any added value.

Change-Id: Id34f02deea261cd7e09197771950316efc09ebd5
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M vdsm/virt/migration.py
1 file changed, 2 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/70/57370/1

diff --git a/vdsm/virt/migration.py b/vdsm/virt/migration.py
index ec3cb55..f1eb653 100644
--- a/vdsm/virt/migration.py
+++ b/vdsm/virt/migration.py
@@ -127,9 +127,8 @@
         if 'convergenceSchedule' in kwargs:
             self._convergence_schedule = kwargs.get('convergenceSchedule')
             self._use_convergence_schedule = True
-
-        self.log.debug('convergence schedule set to: %s',
-                       str(self._convergence_schedule))
+            self.log.debug('convergence schedule set to: %s',
+                           str(self._convergence_schedule))
 
     @property
     def hibernating(self):


-- 
To view, visit https://gerrit.ovirt.org/57370
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id34f02deea261cd7e09197771950316efc09ebd5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>


More information about the vdsm-patches mailing list