Juniper Openstack

vCPE : deleted VRF remains in agent

Bug #1572520 reported by Hari Prasad Killi on 2016-04-20

This bug affects 1 person

	Status	Importance	Assigned to	Milestone
Juniper Openstack	Status tracked in Trunk
R2.20	Fix Committed	High	Hari Prasad Killi	Juniper Openstack r2.23 "r2.23"
R2.21.x	Fix Committed	High	Hari Prasad Killi	Juniper Openstack r2.21.2
R2.22.x	Fix Committed	High	Hari Prasad Killi	Juniper Openstack r2.22.2
R3.0	Fix Committed	High	Hari Prasad Killi	Juniper Openstack r3.0.2.0
Trunk	Fix Committed	High	Hari Prasad Killi	Juniper Openstack r3.1.0.0-fcs

Bug Description

State set by ARP module is not deleted, resulting in the VRF remaining in the agent.

Tags:

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-04-20: [Review update] R2.21.x

Review in progress for https://review.opencontrail.org/19491
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-07: [Review update] R2.22.x

Review in progress for https://review.opencontrail.org/19978
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-07: [Review update] R2.20

Review in progress for https://review.opencontrail.org/19979
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-07: [Review update] R3.0

Review in progress for https://review.opencontrail.org/19980
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-09: A change has been merged

#11

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

commit e09183e2526e8fcbf4145c3f0c75c6a25724bed7
Author: Hari <email address hidden>
Date: Wed Apr 20 17:05:07 2016 +0530

Do not add entries to ARP cache when VRF is deleted.

As ARP entries were getting added even when VRF is delete marked,
state was not deleted on the VRF.

Change-Id: I6561835406eb0157c5d5543c3733fb01abf5375c
closes-bug: 1572520

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-09:

#12

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

commit a1da294a14f10de55261a1d9e4109c0eb2a6d5f7
Author: Hari <email address hidden>
Date: Wed Apr 20 17:05:07 2016 +0530

Do not add entries to ARP cache when VRF is deleted.

As ARP entries were getting added even when VRF is delete marked,
state was not deleted on the VRF.

Change-Id: I6561835406eb0157c5d5543c3733fb01abf5375c
closes-bug: 1572520
(cherry picked from commit e09183e2526e8fcbf4145c3f0c75c6a25724bed7)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-09:

#13

Reviewed: https://review.opencontrail.org/19978
Committed: http://github.org/Juniper/contrail-controller/commit/4c8e8c9f94e2c04794d1ea282b5a92bcd8280920
Submitter: Zuul
Branch: R2.22.x

commit 4c8e8c9f94e2c04794d1ea282b5a92bcd8280920
Author: Hari <email address hidden>
Date: Wed Apr 20 17:05:07 2016 +0530

Do not add entries to ARP cache when VRF is deleted.

As ARP entries were getting added even when VRF is delete marked,
state was not deleted on the VRF.

Change-Id: I6561835406eb0157c5d5543c3733fb01abf5375c
closes-bug: 1572520
(cherry picked from commit e09183e2526e8fcbf4145c3f0c75c6a25724bed7)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-10: [Review update] master

#14

Review in progress for https://review.opencontrail.org/20044
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-10: [Review update] R3.0

#16

Review in progress for https://review.opencontrail.org/19980
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-10: [Review update] master

#18

Review in progress for https://review.opencontrail.org/20044
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-11: A change has been merged

#20

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

commit b9f6aa7f13149bfabcddf13eb86c451af9d31502
Author: Hari <email address hidden>
Date: Wed Apr 20 17:05:07 2016 +0530

Do not add entries to ARP cache when VRF is deleted.

As ARP entries were getting added even when VRF is delete marked,
state was not deleted on the VRF.

closes-bug: 1572520
(cherry picked from commit 68472f29e47a3c37ba5da8dff7bf663831a1a691)

Conflicts:
src/vnsw/agent/services/arp_proto.cc

Change-Id: Ie9e3d7b6cc93cbd281efb8a1bbcf1dbc08815d47

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-12: [Review update] R2.22.x

#21

Review in progress for https://review.opencontrail.org/20151
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-12: [Review update] R2.20

#23

Review in progress for https://review.opencontrail.org/20153
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-12: [Review update] R2.21.x

#25

Review in progress for https://review.opencontrail.org/20154
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-12: A change has been merged

#27

Reviewed: https://review.opencontrail.org/20151
Committed: http://github.org/Juniper/contrail-controller/commit/f75abdd4f98f32d9fffdbf85316111ae9c93f2ea
Submitter: Zuul
Branch: R2.22.x

commit f75abdd4f98f32d9fffdbf85316111ae9c93f2ea
Author: Hari <email address hidden>
Date: Thu May 12 12:28:22 2016 +0530

Arp proto - do not check for delete flag on state.

The check added in a recent fix wouldnt work in case of VRF delete
and is removed here. Check isnt required.

Change-Id: Idc0971bb2089be0e26b216e0192ccf8f5c0c91b0
closes-bug: #1572520

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-13:

#28

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

commit f584ebe1d423522a06e7e8f977929723b9584670
Author: Hari <email address hidden>
Date: Thu May 12 12:28:22 2016 +0530

Arp proto - do not check for delete flag on state.

The check added in a recent fix wouldnt work in case of VRF delete
and is removed here. Check isnt required.

Change-Id: Idc0971bb2089be0e26b216e0192ccf8f5c0c91b0
closes-bug: #1572520
(cherry picked from commit 13c6b695f6e201ef331a24ba61d0c3cfe859fb20)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-16:

#29

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

commit 7587ce7d8939273bd3112c51b72ca8a58add1687
Author: Hari <email address hidden>
Date: Thu May 12 12:28:22 2016 +0530

Arp proto - do not check for delete flag on state.

The check added in a recent fix wouldnt work in case of VRF delete
and is removed here. Check isnt required.

Change-Id: Idc0971bb2089be0e26b216e0192ccf8f5c0c91b0
closes-bug: #1572520
(cherry picked from commit 13c6b695f6e201ef331a24ba61d0c3cfe859fb20)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-16: [Review update] master

#30

Review in progress for https://review.opencontrail.org/20044
Submitter: Hari Prasad Killi (<email address hidden>)

Revision history for this message

OpenContrail Admin (ci-admin-f) wrote on 2016-05-20: A change has been merged

#32

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

commit eddbd20e5b49ca7d3cf222a25e571e141d76d93a
Author: Hari <email address hidden>
Date: Wed Apr 20 17:05:07 2016 +0530

Do not add entries to ARP cache when VRF is deleted.

As ARP entries were getting added even when VRF is delete marked,
state was not deleted on the VRF.

Conflicts:
src/vnsw/agent/services/arp_proto.cc

Change-Id: I6561835406eb0157c5d5543c3733fb01abf5375c
closes-bug: 1572520
(cherry picked from commit e09183e2526e8fcbf4145c3f0c75c6a25724bed7)
(cherry picked from commit 4c8e8c9f94e2c04794d1ea282b5a92bcd8280920)

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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