Do not disable GR if GR timer is zero

Bug #1688133 reported by Ananth Suryanarayana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.2
Fix Committed
Medium
Ananth Suryanarayana
R3.2.3.x
Fix Committed
Medium
Ananth Suryanarayana
Trunk
Fix Committed
Medium
Ananth Suryanarayana

Bug Description

The following issues were found during customer testing for GR/LLGR:

o GR/LLGR capabilities are not advertised when configured GR time is zero
even though configured LLGR time is non-zero
o GR/LLGR helper mode is disabled if received GR time is zero even though
received LLGR time is non-zero

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

Review in progress for https://review.opencontrail.org/31027
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/31028
Submitter: Ananth Suryanarayana (<email address hidden>)

Nischal Sheth (nsheth)
tags: added: contrail-control graceful-restart
Nischal Sheth (nsheth)
description: updated
summary: - Do not disable GR if GR timer is 0
+ Do not disable GR if GR timer is zero
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/31027
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/31028
Submitter: Ananth Suryanarayana (<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/31027
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/31028
Submitter: Ananth Suryanarayana (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/31194
Submitter: Nischal Sheth (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/31027
Committed: http://github.com/Juniper/contrail-controller/commit/a6caa4127cc9f9fb66bf94018834156bd65ec21e
Submitter: Zuul (<email address hidden>)
Branch: master

commit a6caa4127cc9f9fb66bf94018834156bd65ec21e
Author: Ananth Suryanarayana <email address hidden>
Date: Thu May 4 17:04:19 2017 -0700

Provide Llgr only support

o Send GR capability even when configured restart-time is 0
o Enable GR helper even if received restart-time is 0

Fix a few issues in LLGR

o Process received notification code-subcode correctly and trigger
GR/LLGR Helper mode accordingly
o Add UTs to cover most of the scenarios described above

Change-Id: Ibcef9c059408a61e8513b6a9f6e1d015128e6515
Closes-Bug: 1688133

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

Reviewed: https://review.opencontrail.org/31028
Committed: http://github.com/Juniper/contrail-controller/commit/4e4af5f0b1339ca5e303ee049828731d433fdaf7
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit 4e4af5f0b1339ca5e303ee049828731d433fdaf7
Author: Ananth Suryanarayana <email address hidden>
Date: Thu May 4 17:04:19 2017 -0700

Provide Llgr only support

o Send GR capability even when configured restart-time is 0
o Enable GR helper even if received restart-time is 0

Fix a few issues in LLGR

o Process received notification code-subcode correctly and trigger
GR/LLGR Helper mode accordingly
o Add UTs to cover most of the scenarios described above

Change-Id: Ibcef9c059408a61e8513b6a9f6e1d015128e6515
Closes-Bug: 1688133

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

Reviewed: https://review.opencontrail.org/31194
Committed: http://github.com/Juniper/contrail-controller/commit/fc8eb3c80fff09fd6ac20f677223d131afab0755
Submitter: Zuul (<email address hidden>)
Branch: R4.0

commit fc8eb3c80fff09fd6ac20f677223d131afab0755
Author: Ananth Suryanarayana <email address hidden>
Date: Thu May 4 17:04:19 2017 -0700

Provide Llgr only support

o Send GR capability even when configured restart-time is 0
o Enable GR helper even if received restart-time is 0

Fix a few issues in LLGR

o Process received notification code-subcode correctly and trigger
GR/LLGR Helper mode accordingly
o Add UTs to cover most of the scenarios described above

Change-Id: Ibcef9c059408a61e8513b6a9f6e1d015128e6515
Closes-Bug: 1688133

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

Review in progress for https://review.opencontrail.org/32576
Submitter: Vinay Vithal Mahuli (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/32576
Committed: http://github.com/Juniper/contrail-controller/commit/b6925255c67927338347daab2f81d823529d423d
Submitter: Zuul (<email address hidden>)
Branch: R3.2.3.x

commit b6925255c67927338347daab2f81d823529d423d
Author: Ananth Suryanarayana <email address hidden>
Date: Thu May 4 17:04:19 2017 -0700

Provide Llgr only support

o Send GR capability even when configured restart-time is 0
o Enable GR helper even if received restart-time is 0

Fix a few issues in LLGR

o Process received notification code-subcode correctly and trigger
GR/LLGR Helper mode accordingly
o Add UTs to cover most of the scenarios described above

Change-Id: Ibcef9c059408a61e8513b6a9f6e1d015128e6515
Closes-Bug: 1688133

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.