Skip to content

Commit

Permalink
Schema logging error bug fix
Browse files Browse the repository at this point in the history
Change-Id: If693bd6013fcb71064c0711249558f0c9a5a0b74
Closes-Bug: #1650802
(cherry picked from commit e378d80)
  • Loading branch information
Senthilnathan Murugappan committed Dec 18, 2016
1 parent 5b31861 commit 8c88b9c
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 13 deletions.
18 changes: 9 additions & 9 deletions src/config/schema-transformer/config_db.py
Expand Up @@ -54,8 +54,8 @@ def _access_control_list_update(acl_obj, name, obj, entries):
return acl_obj
except (NoIdError, BadRequest) as e:
DBBaseST._logger.error(
"Error while creating acl %s for %s: %s",
name, obj.get_fq_name_str(), str(e))
"Error while creating acl %s for %s: %s"%(
name, obj.get_fq_name_str(), str(e)))
return None
else:
if entries is None:
Expand All @@ -75,11 +75,11 @@ def _access_control_list_update(acl_obj, name, obj, entries):
DBBaseST._vnc_lib.access_control_list_update(acl_obj)
except HttpError as he:
DBBaseST._logger.error(
"HTTP error while updating acl %s for %s: %d, %s",
name, obj.get_fq_name_str(), he.status_code, he.content)
"HTTP error while updating acl %s for %s: %d, %s"%(
name, obj.get_fq_name_str(), he.status_code, he.content))
except NoIdError:
DBBaseST._logger.error("NoIdError while updating acl %s for %s",
name, obj.get_fq_name_str())
DBBaseST._logger.error("NoIdError while updating acl %s for %s"%(
name, obj.get_fq_name_str()))
return acl_obj
# end _access_control_list_update

Expand Down Expand Up @@ -3049,7 +3049,7 @@ def update_peering(self):
obj = self.read_vnc_obj(fq_name=self.name)
except NoIdError as e:
self._logger.error("NoIdError while reading bgp router "
"%s: %s", self.name, str(e))
"%s: %s"%(self.name, str(e)))
return

peerings = [ref['to'] for ref in (obj.get_bgp_router_refs() or [])]
Expand Down Expand Up @@ -3077,7 +3077,7 @@ def update_peering(self):
self._vnc_lib.bgp_router_update(obj)
except NoIdError as e:
self._logger.error("NoIdError while updating bgp router "
"%s: %s", self.name, str(e))
"%s: %s"%(self.name, str(e)))
# end update_peering

def handle_st_object_req(self):
Expand Down Expand Up @@ -3934,7 +3934,7 @@ def add_properties(self, props):
if (not self.left_vn_str or not self.right_vn_str):
self._logger.error(
"%s: route table next hop service instance must "
"have left and right virtual networks", self.name)
"have left and right virtual networks"% self.name)
return self.delete_properties()

policy_name = "_internal_" + self.name
Expand Down
8 changes: 4 additions & 4 deletions src/config/schema-transformer/to_bgp.py
Expand Up @@ -210,8 +210,8 @@ def reinit(self):
pass
except Exception as e:
self.logger.error(
"Error while deleting routing instance %s: %s",
ri.get_fq_name_str(), str(e))
"Error while deleting routing instance %s: %s"%(
ri.get_fq_name_str(), str(e)))

# end for ri

Expand Down Expand Up @@ -241,8 +241,8 @@ def reinit(self):
pass
except Exception as e:
self.logger.error(
"Error while deleting acl %s: %s",
acl.uuid, str(e))
"Error while deleting acl %s: %s"%(
acl.uuid, str(e)))
# end for acl

gevent.sleep(0.001)
Expand Down

0 comments on commit 8c88b9c

Please sign in to comment.