bad request for lswitches created through NVPAdvancedPlugin

Bug #1227927 reported by Armando Migliaccio
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Armando Migliaccio

Bug Description

Line https://github.com/openstack/neutron/blob/master/neutron/plugins/nicira/vshield/edge_appliance_driver.py#L628

should do truncation to avoid seeing errors like these:

2013-09-19 16:39:14.321 22194 TRACE neutron.plugins.nicira.dhcp_meta.rpc RequestBad: Request https://10.24.106.219/api/ws.v1/lswitch is Bad, response {"details":"Remote call failed","errorCode":600,"rootCauseString":"Remote call failed; nested exception is org.springframework.web.client.HttpClientErrorException: 400 Bad Request"

Tags: nicira
Changed in neutron:
assignee: nobody → Armando Migliaccio (armando-migliaccio)
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/47507

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/47507
Committed: http://github.com/openstack/neutron/commit/2a762867b2c5b67c5790d70e94c801d7d2f3f00c
Submitter: Jenkins
Branch: master

commit 2a762867b2c5b67c5790d70e94c801d7d2f3f00c
Author: armando-migliaccio <email address hidden>
Date: Thu Sep 19 17:27:29 2013 -0700

    Ensure names are truncated to accommodate NVP limit

    Switches created through Advanced Plugin should have
    their name truncated as well. To this aim, move things
    around to avoid awkward imports or code duplications

    Fixes bug #1227927

    Change-Id: I3cc69356227d3bd63603ee1e53d82e6561f4cad0

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: havana-rc1 → 2013.2
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.