list of exported route target get wrong when we restart schema-transformer

Bug #1518415 reported by Nachi Ueno on 2015-11-20
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Medium
Sachin Bansal
R2.21.x
Medium
Sachin Bansal
Trunk
Medium
Sachin Bansal

Bug Description

list of exported route target get wrong when we restart schema-transformer

Version. R2.21 107

[ How to reproduce ]

1. Create following resources

VN
------
Blue network
Red network
Transitive network ( <- allow transit true )

VM
------
Blue
Red

SI
------
Blue-a ( transparent )
Blue-b ( in-network )

Red-a ( transparent )
Red-b ( in-network )

Policy
-----

blue_policy src: blue dest: transitive SI: Blue-a, Blue-b
red_policy src: red dest: transitive SI: Red-a,Red-b

2. Restart schema-transformer

RT export in the RI next to the transitive network get lost.

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

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

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

Nischal Sheth (nsheth) on 2015-11-21
tags: added: config service-chain

Reviewed: https://review.opencontrail.org/15313
Committed: http://github.org/Juniper/contrail-controller/commit/47346c25f02775eb858587a2a7b8b290d636039c
Submitter: Zuul
Branch: master

commit 47346c25f02775eb858587a2a7b8b290d636039c
Author: Sachin Bansal <email address hidden>
Date: Fri Nov 20 11:38:35 2015 -0800

Schema transformer should restore route target list at restart

When schema transformer restarts, it resets route targets to what is currently
applicable. But it is not restoring the primary routing-instance's target as an
export target on service routing instances if needed. With this patch, that is
fixed.

Change-Id: Id829af2e3d16f98b52f2b21a8c9f2fdff21ecde5
Closes-Bug: 1518415

information type: Proprietary → Public
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/15311
Committed: http://github.org/Juniper/contrail-controller/commit/f50ca65696205792e83f909b5eee65da00e96187
Submitter: Zuul
Branch: R2.20

commit f50ca65696205792e83f909b5eee65da00e96187
Author: Sachin Bansal <email address hidden>
Date: Fri Nov 20 10:57:47 2015 -0800

Schema transformer should restore route target list at restart

When schema transformer restarts, it resets route targets to what is currently
applicable. But it is not restoring the primary routing-instance's target as an
export target on service routing instances if needed. With this patch, that is
fixed.

Change-Id: Id829af2e3d16f98b52f2b21a8c9f2fdff21ecde5
Closes-Bug: 1518415

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

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

Reviewed: https://review.opencontrail.org/15316
Committed: http://github.org/Juniper/contrail-controller/commit/cfc28d08da8e4389c2748ef7e2d9dcc6c2ce8ba5
Submitter: Zuul
Branch: R2.21.x

commit cfc28d08da8e4389c2748ef7e2d9dcc6c2ce8ba5
Author: Sachin Bansal <email address hidden>
Date: Fri Nov 20 10:57:47 2015 -0800

Schema transformer should restore route target list at restart

When schema transformer restarts, it resets route targets to what is currently
applicable. But it is not restoring the primary routing-instance's target as an
export target on service routing instances if needed. With this patch, that is
fixed.

Change-Id: Id829af2e3d16f98b52f2b21a8c9f2fdff21ecde5
Closes-Bug: 1518415
(cherry picked from commit f50ca65696205792e83f909b5eee65da00e96187)

Reviewed: https://review.opencontrail.org/15365
Committed: http://github.org/Juniper/contrail-controller/commit/c1cb737c4a0cad88948dbc680357f4f1a59f4678
Submitter: Zuul
Branch: R2.20

commit c1cb737c4a0cad88948dbc680357f4f1a59f4678
Author: Sachin Bansal <email address hidden>
Date: Mon Nov 23 14:32:51 2015 -0800

Use get_route_target() instead of _route_target

_route_target returns and integer. We should use the wrapper function that
returns properly formatted route target name.

Change-Id: Ie441c1f38d93996d4cc0d702255f4f4cc25fca65
Closes-Bug: 1518415

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

Other bug subscribers