yahoo-eng-team team mailing list archive

[Salvatore Orlando <1353936@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

Recent commit [1] change the floating_ips API code to ensure the
FloatingIPPoolNotFound exception, raised by nova.network.neutronv2.api,
is handled.

However, a 400 error is raised in this case.
This is not correct, as the error expected is a 404.

404 is indeed the error expected by tempest negative cases, and the
error that would be raised in this situation by nova/network.

It might be argued that 400 might be more appropriate as the root cause
is a bad pool id in a request, but currently tempest and nova-network
are enforcing a different logic, so neutron should just adhere to it.

[1]
http://git.openstack.org/cgit/openstack/nova/commit/?id=e7d7fbecbdd6d15849f0f59b25755ae4385c0385

** Affects: nova
     Importance: Undecided
     Assignee: Salvatore Orlando (salvatore-orlando)
         Status: In Progress


** Tags: neutron-full-job

** Tags added: neutron-full-job

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1353936

Title:
  Nova/Neutron: raise 404 if floating IP not found

Status in OpenStack Compute (Nova):
  In Progress

Bug description:
  Recent commit [1] change the floating_ips API code to ensure the
  FloatingIPPoolNotFound exception, raised by
  nova.network.neutronv2.api, is handled.

  However, a 400 error is raised in this case.
  This is not correct, as the error expected is a 404.

  404 is indeed the error expected by tempest negative cases, and the
  error that would be raised in this situation by nova/network.

  It might be argued that 400 might be more appropriate as the root
  cause is a bad pool id in a request, but currently tempest and nova-
  network are enforcing a different logic, so neutron should just adhere
  to it.

  [1]
  http://git.openstack.org/cgit/openstack/nova/commit/?id=e7d7fbecbdd6d15849f0f59b25755ae4385c0385

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1353936/+subscriptions