diff --git a/hooks/cinder_hooks.py b/hooks/cinder_hooks.py index c6307c97..fa185b22 100755 --- a/hooks/cinder_hooks.py +++ b/hooks/cinder_hooks.py @@ -200,7 +200,7 @@ def db_changed(): migrate_database() else: juju_log('allowed_units either not presented, or local unit ' - 'not in acl list: %s' % allowed_units) + 'not in acl list: %s' % repr(allowed_units)) @hooks.hook('pgsql-db-relation-changed') diff --git a/unit_tests/test_cinder_hooks.py b/unit_tests/test_cinder_hooks.py index 193c71dc..1cbef574 100644 --- a/unit_tests/test_cinder_hooks.py +++ b/unit_tests/test_cinder_hooks.py @@ -245,7 +245,7 @@ class TestChangedHooks(CharmTestCase): self.assertFalse(self.migrate_database.called) def test_db_changed_relation_db_missing_acls(self): - 'It does not migration when acl list is not present' + 'No database migration is attempted when ACL list is not present' self.relation_get.return_value = None self.local_unit.return_value = 'cinder/0' self.CONFIGS.complete_contexts.return_value = ['shared-db']