Trunk and port are residual in VM deleting scenario when southbound plugin/agent failed

Bug #1938972 reported by Kevin Li
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
New
Medium
Unassigned

Bug Description

With following reproduce steps, trunk and port will be residual:

 1. Operator create trunk , parent port and subport
 2. Operator create VM with parent port
 3. Port is bound succeed. Host and device information are updated.
 4. Operator start to delete VM
 5. Due to 3rd party plugin issue, port unbind failed. Host and device information are not updated. Trunk and port binding information are not updated. Port's status is normal and active.

Test result:
 1. For nova, it will delete VM ignoring port update failed response code. So VM is deleted successfully.
 2. For Neutron, operator's expected behavior is trunk and port will be deleted while the reality is operator fail to delete trunk as port is in bound status.

 6315384D-49A8-E88F-E911-0FB32A841FBB:/home/fsp # openstack network trunk delete 8aae96b5-18f1-4b67-b0eb-9fdffb0c9a1b
 Failed to delete trunk with name or ID '8aae96b5-18f1-4b67-b0eb-9fdffb0c9a1b': Trunk 8aae96b5-18f1-4b67-b0eb-9fdffb0c9a1b is currently in use.
 Neutron server returns request_ids: ['req-3d0dbe95-250f-4791-835f-ef9617ee7871']
 1 of 1 trunks failed to delete.

Alternative:

 1. Port bound check is ignored when trunk is deleted just as port. When trunk delete is triggered, its trunk_port_validator.can_be_trunked_or_untrunked(context) is removed.

 2. Adding a new API:DELETE /v2.0/trunks/{trunk_id}/force_delete_trunk . Since neutron does not support action in POST as nova, we do not have an existing structure to add a parameter to support force delete as nova.

Changed in neutron:
status: New → Triaged
importance: Undecided → Medium
tags: added: trunk
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/808179

Changed in neutron:
status: Triaged → In Progress
Revision history for this message
Slawek Kaplonski (slaweq) wrote : auto-abandon-script

This bug has had a related patch abandoned and has been automatically un-assigned due to inactivity. Please re-assign yourself if you are continuing work or adjust the state as appropriate if it is no longer valid.

Changed in neutron:
status: In Progress → New
tags: added: timeout-abandon
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by "Slawek Kaplonski <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/808179
Reason: This review is > 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

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.