Tempest Rhel7: External network cant be deleted while holding unassociated floating IPs

Bug #1528979 reported by Nagendra Prasath
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
High
Sahil Sabharwal
R3.0.2.x
Fix Committed
High
Sahil Sabharwal
R3.0.3.x
Fix Committed
High
Sahil Sabharwal
R3.1
Fix Committed
High
Sahil Sabharwal
R3.1.1.x
Fix Committed
High
Sahil Sabharwal
R3.2
Fix Committed
High
Sahil Sabharwal
Trunk
Fix Committed
High
Sahil Sabharwal

Bug Description

Mainline/2683/Rhel7/Kilo

A network can be deleted when holding unassociated floating IPs. However below error is seen.

[root@a5s165 ~(keystone_admin)]# neutron net-create np-test-net --router:external
Created a new network:
+-------------------------+--------------------------------------+
| Field | Value |
+-------------------------+--------------------------------------+
| admin_state_up | True |
| contrail:fq_name | default-domain |
| | admin |
| | np-test-net |
| contrail:instance_count | 0 |
| id | 25290893-7705-4b7e-a990-07f69b2c6cd5 |
| name | np-test-net |
| router:external | True |
| shared | False |
| status | ACTIVE |
| subnets | |
| tenant_id | 2eb241bca66f43cfa9e595948d8ecfd4 |
+-------------------------+--------------------------------------+
[root@a5s165 ~(keystone_admin)]# neutron subnet-create --name np-test-subnet np-test-net 10.1.1.0/24
Created a new subnet:
+-------------------+--------------------------------------------+
| Field | Value |
+-------------------+--------------------------------------------+
| allocation_pools | {"start": "10.1.1.2", "end": "10.1.1.254"} |
| cidr | 10.1.1.0/24 |
| dns_nameservers | |
| enable_dhcp | True |
| gateway_ip | 10.1.1.1 |
| host_routes | |
| id | 578b202c-dcf9-4c61-b691-9a2a47d8b029 |
| ip_version | 4 |
| ipv6_address_mode | |
| ipv6_ra_mode | |
| name | np-test-subnet |
| network_id | 25290893-7705-4b7e-a990-07f69b2c6cd5 |
| tenant_id | 2eb241bca66f43cfa9e595948d8ecfd4 |
+-------------------+--------------------------------------------+
[root@a5s165 ~(keystone_admin)]#

[root@a5s165 ~(keystone_admin)]# neutron floatingip-create np-test-net
Created a new floatingip:
+---------------------+--------------------------------------+
| Field | Value |
+---------------------+--------------------------------------+
| fixed_ip_address | |
| floating_ip_address | 10.1.1.3 |
| floating_network_id | 25290893-7705-4b7e-a990-07f69b2c6cd5 |
| id | b45fd128-35c3-4d02-baa6-5610912b4da9 |
| port_id | |
| router_id | |
| status | ACTIVE |
| tenant_id | 2eb241bca66f43cfa9e595948d8ecfd4 |
+---------------------+--------------------------------------+
[root@a5s165 ~(keystone_admin)]# neutron net-delete np-test-net
Unable to complete operation on network 25290893-7705-4b7e-a990-07f69b2c6cd5. There are one or more ports still in use on the network.
[root@a5s165 ~(keystone_admin)]#

Because of this behavior, following cases are failing:
tempest.api.network.admin.test_external_network_extension.ExternalNetworksTestJSON.test_delete_external_networks_with_floating_ip

summary: - Tempest Rhel7: External network cant be deleted while still holding
+ Tempest Rhel7: External network cant be deleted while holding
unassociated floating IPs
tags: added: rhosp-tempest
information type: Proprietary → Public
tags: added: tempest
tags: added: osp7-cert osp8-cert rhos-tempest
removed: rhosp-tempest
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

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

Review in progress for https://review.opencontrail.org/28008
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/28008
Committed: http://github.org/Juniper/contrail-controller/commit/ff14444c6839aaa88e0a75ae47c6571e465f6d28
Submitter: Zuul (<email address hidden>)
Branch: master

commit ff14444c6839aaa88e0a75ae47c6571e465f6d28
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c

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

Review in progress for https://review.opencontrail.org/28412
Submitter: Sachin Bansal (<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/28413
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/28414
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/28412
Committed: http://github.org/Juniper/contrail-controller/commit/9648c079d9f97b4de3be5b0a949e956867ddca25
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit 9648c079d9f97b4de3be5b0a949e956867ddca25
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c
(cherry picked from commit ff14444c6839aaa88e0a75ae47c6571e465f6d28)

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

Reviewed: https://review.opencontrail.org/28414
Committed: http://github.org/Juniper/contrail-controller/commit/a7da7407093e2c2ed44c0fdfdcb9459401403111
Submitter: Zuul (<email address hidden>)
Branch: R3.0

commit a7da7407093e2c2ed44c0fdfdcb9459401403111
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c
(cherry picked from commit ff14444c6839aaa88e0a75ae47c6571e465f6d28)

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

Reviewed: https://review.opencontrail.org/28413
Committed: http://github.org/Juniper/contrail-controller/commit/880716b4b6d6a63cf7c3c74931b21a40b1428ce1
Submitter: Zuul (<email address hidden>)
Branch: R3.1

commit 880716b4b6d6a63cf7c3c74931b21a40b1428ce1
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c
(cherry picked from commit ff14444c6839aaa88e0a75ae47c6571e465f6d28)

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

Review in progress for https://review.opencontrail.org/30543
Submitter: Atul Moghe (<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/30545
Submitter: Atul Moghe (<email address hidden>)

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

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

commit fd2d6b8a8f69153834c17957b3b75e4a0e24dfff
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c
(cherry picked from commit ff14444c6839aaa88e0a75ae47c6571e465f6d28)

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

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

commit dc68dec5adca4419756f0c85e026cae7a3bbfb17
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c
(cherry picked from commit ff14444c6839aaa88e0a75ae47c6571e465f6d28)

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

Review in progress for https://review.opencontrail.org/30579
Submitter: Atul Moghe (<email address hidden>)

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

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

commit 3e6afe3b1822f9692ad6c3b12bf0d6126c10e093
Author: Sahil <email address hidden>
Date: Thu Jan 19 10:40:58 2017 -0800

Added cascaded delete when deleting a VN

Closes-Bug: 1528979

Change-Id: I32ea59b611ad33fd01edf411de38ffdce9c4ae7c
(cherry picked from commit ff14444c6839aaa88e0a75ae47c6571e465f6d28)

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

Review in progress for https://review.opencontrail.org/30779
Submitter: <email address hidden> (<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/30801
Submitter: <email address hidden> (<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/30802
Submitter: <email address hidden> (<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/30803
Submitter: <email address hidden> (<email address hidden>)

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

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

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

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

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

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

commit 5e53bd2bb4d93f3c133712e4635423a690763c33
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Closes-Bug: 1528979

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

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

commit 28b1108be494d39924f0e402fab8dacdd1b80e4f
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Closes-Bug: 1528979

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

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

commit 4baa14aca04082dc1ea3c9db43751bbcee712cc3
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Closes-Bug: 1528979

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

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

commit 6bc35229a73b996f7723125ccac5cb9752e70cab
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Closes-Bug: 1528979

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

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

commit 4166c778b8cd4a9645ae3175a28323ed4bd264dc
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Closes-Bug: 1528979

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

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

commit 93fc38888e86e65087198300a403372f8691c4fb
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

Closes-Bug: 1528979

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Conflicts:
 src/config/vnc_openstack/vnc_openstack/neutron_plugin_db.py

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

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

commit ae3205f686064583c484073de721a47fca12afb1
Author: Sahil <email address hidden>
Date: Wed Apr 26 14:32:14 2017 -0700

Fix: Added cascaded delete when deleting a VN

Added a check when deleteing floatingIp object to
make sure that it's not associated with any ports.

-Added UT for master branch

Change-Id: I2c34abadd21dd11b5d1f77aacba62a33a3b566cb
Closes-Bug: 1528979

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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