BGPaaS: schema restart causes BGP session reset

Bug #1575323 reported by amit surana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
Medium
Sachin Bansal
Trunk
Fix Committed
Medium
Sachin Bansal

Bug Description

restart of schema is causing agent to RST and re-establish the BGP connection to the CN. Schema restart is causing 3 updates to be pushed to agent; the 2nd update has the wrong ASN.

root@csol2-node5:/var/tmp# grep b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6 config.txt
2016 Apr 26 10:57:47.317174 csol2-node5 [:contrail-api:] : VncApiConfigLog: [VncApiCommon: identifier_uuid = b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6, object_type = bgp_router, identifier_name = default-domain:symantec.Tenant.125:tenant125.test_id1.Private_VSRX_BGP_VN0:tenant125.test_id1.Private_VSRX_BGP_VN0:3bd30f9b-f0d7-4fc2-b0fc-c00fe5121235, url = http://172.16.180.25:8082/bgp-router/b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6, operation = put, useragent = csol2-node5:/usr/bin/contrail-schema, remote_ip = 172.16.180.25:8082, body = {"bgp-router": {"uuid": "b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6", "bgp_router_parameters": {"vendor": null, "admin_down": false, "local_autonomous_system": null, "auth_data": null, "autonomous_system": 652, "address": "1.3.0.3", "source_port": 50025, "router_type": "bgpaas-client", "identifier": "1.3.0.3", "hold_time": 90, "port": null, "address_families": null}}}, domain = default-domain]
2016 Apr 26 10:57:50.670946 csol2-node5 [:contrail-api:] : VncApiConfigLog: [VncApiCommon: identifier_uuid = b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6, object_type = bgp_router, identifier_name = default-domain:symantec.Tenant.125:tenant125.test_id1.Private_VSRX_BGP_VN0:tenant125.test_id1.Private_VSRX_BGP_VN0:3bd30f9b-f0d7-4fc2-b0fc-c00fe5121235, url = http://172.16.180.25:8082/bgp-router/b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6, operation = put, useragent = csol2-node5:/usr/bin/contrail-schema, remote_ip = 172.16.180.25:8082, body = {"bgp-router": {"uuid": "b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6", "bgp_router_parameters": {"vendor": null, "admin_down": false, "local_autonomous_system": null, "auth_data": null, "autonomous_system": 64512, "address": "1.3.0.3", "source_port": 50025, "router_type": "bgpaas-client", "identifier": "1.3.0.3", "hold_time": 90, "port": null, "address_families": null}}}, domain = default-domain]
2016 Apr 26 10:57:56.190751 csol2-node5 [:contrail-api:] : VncApiConfigLog: [VncApiCommon: identifier_uuid = b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6, object_type = bgp_router, identifier_name = default-domain:symantec.Tenant.125:tenant125.test_id1.Private_VSRX_BGP_VN0:tenant125.test_id1.Private_VSRX_BGP_VN0:3bd30f9b-f0d7-4fc2-b0fc-c00fe5121235, url = http://172.16.180.25:8082/bgp-router/b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6, operation = put, useragent = csol2-node5:/usr/bin/contrail-schema, remote_ip = 172.16.180.25:8082, body = {"bgp-router": {"uuid": "b32d1de4-be2c-44d4-8df3-6c6a0e3eb0d6", "bgp_router_parameters": {"vendor": null, "admin_down": false, "local_autonomous_system": null, "auth_data": null, "autonomous_system": 652, "address": "1.3.0.3", "source_port": 50025, "router_type": "bgpaas-client", "identifier": "1.3.0.3", "hold_time": 90, "port": null, "address_families": null}}}, domain = default-domain]

Tags: bgpaas config
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/19648
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/19649
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/19648
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/19649
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/19648
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/19649
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/19648
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/19649
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/19648
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/19649
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/19648
Committed: http://github.org/Juniper/contrail-controller/commit/0a7fe1483d5ee10ae313bf3a1a56296dd82d6356
Submitter: Zuul
Branch: master

commit 0a7fe1483d5ee10ae313bf3a1a56296dd82d6356
Author: Sachin Bansal <email address hidden>
Date: Tue Apr 26 12:17:58 2016 -0700

Do not update bgpaas-client objects if not necessary

Currently, bgpaas-client objects get updated with global asn when schema
transformer restarts. They get updated again later with correct asn, but this
causes a connection reset. We should avoid both these updates.

Change-Id: Ia61c6434a59978d083fed847ef0955f7049f8d3c
Closes-Bug: 1575323

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/19649
Committed: http://github.org/Juniper/contrail-controller/commit/1b8243a688565be6a86ffed65f2fa87298b6d9f2
Submitter: Zuul
Branch: R3.0

commit 1b8243a688565be6a86ffed65f2fa87298b6d9f2
Author: Sachin Bansal <email address hidden>
Date: Tue Apr 26 12:17:58 2016 -0700

Do not update bgpaas-client objects if not necessary

Currently, bgpaas-client objects get updated with global asn when schema
transformer restarts. They get updated again later with correct asn, but this
causes a connection reset. We should avoid both these updates.

Change-Id: Ia61c6434a59978d083fed847ef0955f7049f8d3c
Closes-Bug: 1575323
(cherry picked from commit 0a7fe1483d5ee10ae313bf3a1a56296dd82d6356)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.