Don't log a successful image if no update was performed

When no update is necessary, there should no successful
message be logged.

diff --git a/src/ddns/__init__.py b/src/ddns/__init__.py
index 4e4093e2690d731376f2a0ca2603be18f5bd03ac..22764e6e230feed5365c01bdc8d6b876e881ef4d 100644 (file)
--- a/src/ddns/__init__.py
+++ b/src/ddns/__init__.py
@@ -161,7 +161,3 @@ class DDNSCore(object):
                except Exception, e:
                        logger.error(_("Dynamic DNS update for %(hostname)s (%(provider)s) throwed an unhandled exception:") % \
                                { "hostname" : entry.hostname, "provider" : entry.name }, exc_info=True)
-
-               else:
-                       logger.info(_("Dynamic DNS update for %(hostname)s (%(provider)s) successful") % \
-                               { "hostname" : entry.hostname, "provider" : entry.name })

diff --git a/src/ddns/providers.py b/src/ddns/providers.py
index de87b7c9bf32725da548b69d8ea592a5d044f097..59ccd728cd1f7e26f30a4054e6d13c6abff33895 100644 (file)
--- a/src/ddns/providers.py
+++ b/src/ddns/providers.py
@@ -129,12 +129,16 @@ class DDNSProvider(object):
 
                # Check if we actually need to update this host.
                elif self.is_uptodate(self.protocols):
-                       logger.debug(_("%s is already up to date") % self.hostname)
+                       logger.info(_("The dynamic host %(hostname)s (%(provider)s) is already up to date") % \
+                               { "hostname" : self.hostname, "provider" : self.name })
                        return
 
                # Execute the update.
                self.update()
 
+               logger.info(_("Dynamic DNS update for %(hostname)s (%(provider)s) successful") % \
+                       { "hostname" : self.hostname, "provider" : self.name })
+
        def update(self):
                raise NotImplementedError