yahoo-eng-team team mailing list archive

[Chris Dent <cdent+lp@xxxxxxxxxxxxx>]

Public bug reported:

Nova master, late november

When a resource provider fails to create after a POST
/resource_providers for some reason, the error message identifies the
provider by uuid. However, the uuid may not have been supplied by the
client, it may be generated server side. So the name should be included
at:

https://github.com/openstack/nova/blob/daa1cd6d7660a0fb41b501c44db307c3e43f7600/nova/api/openstack/placement/handlers/resource_provider.py#L145

However, because of JSONSchema, it's unlikely (impossible) that
ObjectActionError will ever be raised so another option may be to just
get rid of the handling.

To resolve this:

* figure out if the exception can happen
* if not, remove the handling
* if so, change the message to include the 'name' in the output

** Affects: nova
     Importance: Undecided
         Status: New


** Tags: placement

-- 
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/1735405

Title:
  Error message from placement when creating resource provider uses
  ambiguous identifier

Status in OpenStack Compute (nova):
  New

Bug description:
  Nova master, late november

  When a resource provider fails to create after a POST
  /resource_providers for some reason, the error message identifies the
  provider by uuid. However, the uuid may not have been supplied by the
  client, it may be generated server side. So the name should be
  included at:

  https://github.com/openstack/nova/blob/daa1cd6d7660a0fb41b501c44db307c3e43f7600/nova/api/openstack/placement/handlers/resource_provider.py#L145

  However, because of JSONSchema, it's unlikely (impossible) that
  ObjectActionError will ever be raised so another option may be to just
  get rid of the handling.

  To resolve this:

  * figure out if the exception can happen
  * if not, remove the handling
  * if so, change the message to include the 'name' in the output

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