From 28f57f8b5604ce2f3e7205ad83835410e899451e Mon Sep 17 00:00:00 2001 From: raigax9 Date: Mon, 14 Aug 2017 17:38:48 -0400 Subject: [PATCH] Added remaining error logs in resolvers Added remaing error loggers for resolvers in both flavor_manager and image_manager Change-Id: Ia63efa7081a0a2c86c2e6af3c458cad99d1941ef --- .../external_mock/orm_common/injector/fang/resolver.py | 5 +++-- .../ims/external_mock/orm_common/injector/fang/resolver.py | 5 +++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/orm/services/flavor_manager/fms_rest/external_mock/orm_common/injector/fang/resolver.py b/orm/services/flavor_manager/fms_rest/external_mock/orm_common/injector/fang/resolver.py index 5a58f8d6..1e313d02 100755 --- a/orm/services/flavor_manager/fms_rest/external_mock/orm_common/injector/fang/resolver.py +++ b/orm/services/flavor_manager/fms_rest/external_mock/orm_common/injector/fang/resolver.py @@ -1,4 +1,5 @@ -from .errors import ProviderNotFoundError +import logging +from orm.services.flavor_manager.fms_rest.externel_mock.orm_common.injector.fang.errors import ProviderNotFoundError # This is effectively what is sometimes termed a "dependency injection @@ -38,7 +39,7 @@ class DependencyResolver: try: self.resolve(resource_name) except ProviderNotFoundError as e: - # TODO: Add error logging here + logging.error("Provider not found! Make sure the provider is available.\n %s", e) return False else: return True diff --git a/orm/services/image_manager/ims/external_mock/orm_common/injector/fang/resolver.py b/orm/services/image_manager/ims/external_mock/orm_common/injector/fang/resolver.py index 5a58f8d6..7bea42a6 100755 --- a/orm/services/image_manager/ims/external_mock/orm_common/injector/fang/resolver.py +++ b/orm/services/image_manager/ims/external_mock/orm_common/injector/fang/resolver.py @@ -1,4 +1,5 @@ -from .errors import ProviderNotFoundError +import logging +from orm.services.image_manager.ims.externel_mock.orm_common.injector.fang.errors import ProviderNotFoundError # This is effectively what is sometimes termed a "dependency injection @@ -38,7 +39,7 @@ class DependencyResolver: try: self.resolve(resource_name) except ProviderNotFoundError as e: - # TODO: Add error logging here + logging.error("Provider not found! Make sure the provider is available.\n %s", e) return False else: return True