Remove router interface on Arista Plugin fails

Bug #1453323 reported by Sukhdev Kapur
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Sukhdev Kapur
Kilo
Fix Committed
Low
Sukhdev Kapur

Bug Description

Removing a router interface on Arista L3 Plugin causes the following exception:

2015-05-07 18:43:35.329 24639 INFO neutron.wsgi [-] (24639) accepted ('172.28.129.192', 59983)
2015-05-07 18:43:35.428 DEBUG neutron.services.l3_router.l3_arista [req-268e85d5-f013-44a0-bbf0-d8ec83ea2155 admin demo] neutron.services.l3_router.l3_arista.AristaL3ServicePlugin method remove_router_interface called with arguments (<neutron.context.Context object at 0x7fada0f53dd0>, u'e24f4f29-83b1-4aed-afcb-3adfebad22d5', {u'port_id': u'867118df-949f-4b8f-82d5-734a82c2ec77'}) {} wrapper /opt/stack/neutron/neutron/common/log.py:33
2015-05-07 18:43:35.429 ERROR neutron.api.v2.resource [req-268e85d5-f013-44a0-bbf0-d8ec83ea2155 admin demo] remove_router_interface failed
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource Traceback (most recent call last):
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource File "/opt/stack/neutron/neutron/api/v2/resource.py", line 83, in resource
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource result = method(request=request, **args)
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource File "/opt/stack/neutron/neutron/api/v2/base.py", line 207, in _handle_action
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource return getattr(self._plugin, name)(*arg_list, **kwargs)
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource File "/opt/stack/neutron/neutron/common/log.py", line 34, in wrapper
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource return method(*args, **kwargs)
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource File "/opt/stack/neutron/neutron/services/l3_router/l3_arista.py", line 212, in remove_router_interface
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource context, router_id, interface_info))
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource File "/opt/stack/neutron/neutron/db/l3_db.py", line 1322, in remove_router_interface
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource context, router_id, interface_info)
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource File "/opt/stack/neutron/neutron/db/l3_db.py", line 724, in remove_router_interface
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource self._validate_interface_info(interface_info, for_removal=True)
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource TypeError: _validate_interface_info() got an unexpected keyword argument 'for_removal'
2015-05-07 18:43:35.429 24639 TRACE neutron.api.v2.resource

description: updated
Changed in neutron:
assignee: nobody → Sukhdev Kapur (sukhdev-8)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

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

Reviewed: https://review.openstack.org/181594
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=9c857dab6289047a272a4519479903c92a84dc4c
Submitter: Jenkins
Branch: master

commit 9c857dab6289047a272a4519479903c92a84dc4c
Author: Sukhdev Kapur <email address hidden>
Date: Fri May 8 17:19:58 2015 -0700

    remove router interface on Arista L3 plugin fails

    The failure is because of mismatch of the
    parameters to _validate_interface_info(). This patch removes
    this code as it can be inherited from upstream

    Change-Id: I5a92c6d05876e9ab5201e8fac018433eeb5c89e4
    Closes-Bug: #1453323

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/181665

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (neutron-pecan)

Fix proposed to branch: neutron-pecan
Review: https://review.openstack.org/185072

Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Low
milestone: none → liberty-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/kilo)

Reviewed: https://review.openstack.org/181665
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=c2860ba68fd9c8e39fbdfc079319dac8dedb7952
Submitter: Jenkins
Branch: stable/kilo

commit c2860ba68fd9c8e39fbdfc079319dac8dedb7952
Author: Sukhdev Kapur <email address hidden>
Date: Fri May 8 17:19:58 2015 -0700

    remove router interface on Arista L3 plugin fails

    The failure is because of mismatch of the
    parameters to _validate_interface_info(). This patch removes
    this code as it can be inherited from upstream

    Change-Id: I5a92c6d05876e9ab5201e8fac018433eeb5c89e4
    Closes-Bug: #1453323
    (cherry picked from commit 9c857dab6289047a272a4519479903c92a84dc4c)

Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: liberty-1 → 7.0.0
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.