make logging consistent

octodns/provider/dyn.py

@@ -291,7 +291,8 @@ class DynProvider(BaseProvider):
                 try:
                     fqdn, _type = td.label.split(':', 1)
                 except ValueError as e:
-                    self.log.warn("Failed to load TraficDirector '{}': {}".format(td.label, e))
+                    self.log.warn("Failed to load TrafficDirector %s: %s",
+                            td.label, e.message)
                     continue
                 tds[fqdn][_type] = td
             self._traffic_directors = dict(tds)

tests/test_octodns_provider_dyn.py

@@ -650,7 +650,11 @@ class TestDynProviderGeo(TestCase):
                           set(tds.keys()))
         self.assertEquals(['A'], tds['unit.tests.'].keys())
         self.assertEquals(['A'], tds['geo.unit.tests.'].keys())
-        provider.log.warn.assert_called_with("Failed to load TraficDirector 'something else': need more than 1 value to unpack")
+        provider.log.warn.assert_called_with(
+                u'Failed to load TrafficDirector %s: %s',
+                u'something else',
+                'need more than 1 value to unpack'
+                )
 
 
     @patch('dyn.core.SessionEngine.execute')