Juniper Openstack

[R3.1-mainline-2738]: when policy is enabled-disabled and when component NH is tunnel NH, the mpls label is not updated

Bug #1603958 reported by alok kumar
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Ashok Singh
Juniper Openstack r3.0.4.0
R3.0.3.x
Fix Committed
High
Ashok Singh
Juniper Openstack r3.0.3.1
R3.1
Fix Committed
High
Ashok Singh
Juniper Openstack r3.1.0.0-fcs
Trunk
Fix Committed
High
Ashok Singh
Juniper Openstack r3.2.0.0-fcs "r3.2.0.0"

Bug Description

with same static routes configured on 2 VMIs, we will have ecmp routes.
when NH is tunnel NH, the mpls label is not updated, when enable-disable policy on VMIs.

agent restart, recovers from this state.

Ashok is aware about the issue.

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

Review in progress for https://review.opencontrail.org/22150
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/22329
Submitter: Ashok Singh (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/22329
Committed: http://github.org/Juniper/contrail-controller/commit/3a0c933337c66a0fbacac5541c68c91af0252d18
Submitter: Zuul
Branch: R3.1

commit 3a0c933337c66a0fbacac5541c68c91af0252d18
Author: ashoksingh <email address hidden>
Date: Fri Jul 22 13:01:45 2016 +0530

Update Path of static-route with correct label on VMI’s label change.

When VMI’s policy status is changed, its label is modified. The updated
label was not set in static-route’s (associated with VMI) path.

Added UT

Closes-Bug: #1603958
(cherry picked from commit 5245aba9fc1b2757c54f9ed6a81a8329a6708880)

Change-Id: Ifebc69668615b5636d640ac3116e077e86a585b2

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

Reviewed: https://review.opencontrail.org/22150
Committed: http://github.org/Juniper/contrail-controller/commit/5245aba9fc1b2757c54f9ed6a81a8329a6708880
Submitter: Zuul
Branch: master

commit 5245aba9fc1b2757c54f9ed6a81a8329a6708880
Author: ashoksingh <email address hidden>
Date: Fri Jul 22 13:01:45 2016 +0530

Update Path of static-route with correct label on VMI’s label change.

When VMI’s policy status is changed, its label is modified. The updated
label was not set in static-route’s (associated with VMI) path.

Added UT

Change-Id: I67bcc337cc33d9894ac19ad1ec41a3a558e0e899
Closes-Bug: #1603958

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

Review in progress for https://review.opencontrail.org/22363
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/22375
Submitter: Ashok Singh (<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/22363
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/22375
Submitter: Ashok Singh (<email address hidden>)

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

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

commit d2caa8d3ec6ddb7e693476ce064b5cce24dcc192
Author: ashoksingh <email address hidden>
Date: Wed Jul 27 13:23:49 2016 +0530

Update Allowed Address pair routes on policy change.

When policy changes, the label of VMI is updated. The updated VMI label was not
reflected in AAP routes. Added UT to verify this.

Remove the API InetUnicastAgentRouteTable::ReEvaluatePaths as we don’t need
separate handling for policy change because we change label on policy change.

Change-Id: Ic18662e62447aceb72492cc98043c2bb1f8d19e6
Closes-Bug: #1603958
(cherry picked from commit 9fce2d4d95dfa2a1433d5e416d40836e89e0709d)

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

Review in progress for https://review.opencontrail.org/22363
Submitter: Ashok Singh (<email address hidden>)

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

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

commit 0ba5a3cd2d696b14d07024147ad5433ee1c63a3d
Author: ashoksingh <email address hidden>
Date: Fri Aug 5 10:59:56 2016 +0530

When policy changes, the label of VMI is updated. The updated VMI label was not
reflected in AAP routes. Added UT to verify this.

Remove the API InetUnicastAgentRouteTable::ReEvaluatePaths as we don’t need
separate handling for policy change because we change label on policy change.

Change-Id: Ifbb719002f53c3527efa5b161d0b37e6b372c6be
Closes-Bug: #1603958

Revision history for this message
alok kumar (kalok) wrote :

test case: test_disable_enable_policy_inter_node

Avinash (avink)
information type: Proprietary → Private
information type: Private → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0.3.x

Review in progress for https://review.opencontrail.org/25128
Submitter: Ashok Singh (<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/25129
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/25128
Submitter: Ashok Singh (<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/25129
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/25128
Submitter: Ashok Singh (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/25129
Committed: http://github.org/Juniper/contrail-controller/commit/6b68ed02b9ee954d0fba695b802a5361c6e9f449
Submitter: Zuul
Branch: R3.0

commit 6b68ed02b9ee954d0fba695b802a5361c6e9f449
Author: ashoksingh <email address hidden>
Date: Fri Jul 22 13:01:45 2016 +0530

Update Path of static-route with correct label on VMI’s label change.

When VMI’s policy status is changed, its label is modified. The updated
label was not set in static-route’s (associated with VMI) path.

Added UT

Closes-Bug: #1603958
(cherry picked from commit 5245aba9fc1b2757c54f9ed6a81a8329a6708880)

When policy changes, the label of VMI is updated. The updated VMI label was not
reflected in AAP routes. Added UT to verify this.

Remove the API InetUnicastAgentRouteTable::ReEvaluatePaths as we don’t need
separate handling for policy change because we change label on policy change.

Closes-Bug: #1603958
(cherry picked from commit 0ba5a3cd2d696b14d07024147ad5433ee1c63a3d)
Change-Id: I2240adde60cde055d07e013ea9e6c9ac063dffd3

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

Review in progress for https://review.opencontrail.org/25128
Submitter: Ashok Singh (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/25128
Committed: http://github.org/Juniper/contrail-controller/commit/29c16223043f2d3787fa3c94cd351b81e1e25e8a
Submitter: Zuul
Branch: R3.0.3.x

commit 29c16223043f2d3787fa3c94cd351b81e1e25e8a
Author: ashoksingh <email address hidden>
Date: Sun Oct 23 15:58:28 2016 +0530

Update Path of static-route with correct label on VMI’s label change.

When VMI’s policy status is changed, its label is modified. The updated
label was not set in static-route’s (associated with VMI) path.

Added UT

Closes-Bug: #1603958
(cherry picked from commit 5245aba9fc1b2757c54f9ed6a81a8329a6708880)

When policy changes, the label of VMI is updated. The updated VMI label was not
reflected in AAP routes. Added UT to verify this.

Remove the API InetUnicastAgentRouteTable::ReEvaluatePaths as we don’t need
separate handling for policy change because we change label on policy change.

Closes-Bug: #1603958
(cherry picked from commit 0ba5a3cd2d696b14d07024147ad5433ee1c63a3d)
Change-Id: If05ea42f150f1c147ec04d6a5da6cc1a1f551113

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

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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