x/ranger
Code Issues Proposed changes

Added remaining error logs in resolvers

Browse Source 
Added remaing error loggers for resolvers in both flavor_manager and image_manager

Change-Id: Ia63efa7081a0a2c86c2e6af3c458cad99d1941ef
This commit is contained in:
raigax9 2017-08-14 17:38:48 -04:00 committed by Nishad Shah
parent 0979e50eea
commit 28f57f8b56
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
orm/services/flavor_manager/fms_rest/external_mock/orm_common/injector/fang/resolver.py
View File

@ -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 # This is effectively what is sometimes termed a "dependency injection
@ -38,7 +39,7 @@ class DependencyResolver:
try: try:
self.resolve(resource_name) self.resolve(resource_name)
except ProviderNotFoundError as e: 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 return False
else: else:
return True return True

5
orm/services/image_manager/ims/external_mock/orm_common/injector/fang/resolver.py
View File

@ -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 # This is effectively what is sometimes termed a "dependency injection
@ -38,7 +39,7 @@ class DependencyResolver:
try: try:
self.resolve(resource_name) self.resolve(resource_name)
except ProviderNotFoundError as e: 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 return False
else: else:
return True return True