Skip to content

Commit

Permalink
Merge "Deletion of RI object in try and except block" into R3.1
Browse files Browse the repository at this point in the history
  • Loading branch information
Zuul authored and opencontrail-ci-admin committed Dec 23, 2016
2 parents d3f0484 + dca2eaa commit a6adc0e
Showing 1 changed file with 19 additions and 9 deletions.
28 changes: 19 additions & 9 deletions src/config/schema-transformer/config_db.py
Expand Up @@ -2285,19 +2285,26 @@ def delete_obj(self):
if rp:
rp.delete_routing_instance(self)
else:
rp_obj = RoutingPolicyST.read_vnc_obj(fq_name=rp_name)
if rp_obj:
rp_obj.del_routing_instance(self.obj)
self._vnc_lib.routing_policy_update(rp_obj)
try:
rp_obj = RoutingPolicyST.read_vnc_obj(fq_name=rp_name)
if rp_obj:
rp_obj.del_routing_instance(self.obj)
self._vnc_lib.routing_policy_update(rp_obj)
except NoIdError:
pass
for ra_name in self.route_aggregates:
ra = RouteAggregateST.get(ra_name)
if ra:
ra.delete_routing_instance(self)
else:
ra_obj = RouteAggregateST.read_vnc_obj(fq_name=ra_name)
if ra_obj:
ra_obj.del_routing_instance(self.obj)
self._vnc_lib.route_aggregate_update(ra_obj)
try:
ra_obj = RouteAggregateST.read_vnc_obj(fq_name=ra_name)
if ra_obj:
ra_obj.del_routing_instance(self.obj)
self._vnc_lib.route_aggregate_update(ra_obj)
except NoIdError:
pass

self.routing_policys = {}
self.route_aggregates = set()
bgpaas_server_name = self.obj.get_fq_name_str() + ':bgpaas-server'
Expand Down Expand Up @@ -4167,7 +4174,10 @@ def delete_routing_instance(self, ri):
return
self.obj.set_aggregate_route_nexthop(None)
self.obj.set_routing_instance_list([])
self._vnc_lib.route_aggregate_update(self.obj)
try:
self._vnc_lib.route_aggregate_update(self.obj)
except NoIdError:
pass
self.routing_instances.discard(ri.name)
# end delete_routing_instance

Expand Down

0 comments on commit a6adc0e

Please sign in to comment.