gateway_device should be blocked on deletion if any l2gateway is related to the gateway_device.

Bug #1564228 reported by Kengo Hobo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-midonet
Fix Released
Medium
Kengo Hobo

Bug Description

gateway_device should be blocked on deletion if any l2gateway is related to the gateway_device.

However, in current implementaion, gateway_device cannot be deleted if router that is specified in resource_id has any ports.
Thus, even though there is no l2gateways that is related to the gateway_device,
deleting gateway_device is failed if the router has ports(e.g. uplink port).

Though there is no influence to user communication,
the validation is not proper, I think.

Kengo Hobo (hobo-kengo)
Changed in networking-midonet:
assignee: nobody → Kengo Hobo (hobo-kengo)
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :

you mean that "gateway_device cannot be deleted" is the current state and
"Deleting gateway_device should be blocked" is the desirable one?
i don't understand the difference.

Changed in networking-midonet:
status: New → Incomplete
Revision history for this message
Kengo Hobo (hobo-kengo) wrote :

Difference between two description is condition for blocking deletion.
Current:
block deletion when there is no router port of backend router.

Proper:
block deletion when there is no l2gateway that related to the gateway_device.

summary: - gateway_device cannot be deleted if router that is specified in
- resource_id has any ports
+ gateway_device should be blocked on deletion if any l2gateway is related
+ to the gateway_device.
description: updated
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :

i got it. thank you for explanation.

Changed in networking-midonet:
importance: Undecided → Medium
milestone: none → 2.0.0
status: Incomplete → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to networking-midonet (master)

Fix proposed to branch: master
Review: https://review.openstack.org/300567

Changed in networking-midonet:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-midonet (master)

Reviewed: https://review.openstack.org/300567
Committed: https://git.openstack.org/cgit/openstack/networking-midonet/commit/?id=cb81f20dabab3d421df395066f241da6e5d33e5d
Submitter: Jenkins
Branch: master

commit cb81f20dabab3d421df395066f241da6e5d33e5d
Author: HoboK <email address hidden>
Date: Fri Apr 1 16:23:31 2016 +0000

    Change validation logic from router port to l2gateway in gateway_device deletion

    Change-Id: Ia73ca62f2e6e9ff00e0fde27d9f80a40bd79c2ae
    Closes-Bug: #1564228

Changed in networking-midonet:
status: In Progress → Fix Released
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.