yahoo-eng-team team mailing list archive

[Eric Fried <efried@xxxxxxxxxx>]

Public bug reported:

SchedulerReportClient._delete_inventory uses the DELETE
/resource_providers/{u}/inventories API, which does not provide a way to
send the generation down (see related bug [1]), and is therefore subject
to concurrency errors.

Until/unless an alternative becomes available, we should be using PUT
/resource_providers/{u}/inventories with an empty 'inventories' dict,
because that API *does* take the generation in the payload.  (If we do
this, we also make moot part of related bug [2].)

Related bugs:
[1] https://bugs.launchpad.net/nova/+bug/1746075
[2] https://bugs.launchpad.net/nova/+bug/1746373

** 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/1746374

Title:
  Report client _delete_inventory violates generation consistency

Status in OpenStack Compute (nova):
  New

Bug description:
  SchedulerReportClient._delete_inventory uses the DELETE
  /resource_providers/{u}/inventories API, which does not provide a way
  to send the generation down (see related bug [1]), and is therefore
  subject to concurrency errors.

  Until/unless an alternative becomes available, we should be using PUT
  /resource_providers/{u}/inventories with an empty 'inventories' dict,
  because that API *does* take the generation in the payload.  (If we do
  this, we also make moot part of related bug [2].)

  Related bugs:
  [1] https://bugs.launchpad.net/nova/+bug/1746075
  [2] https://bugs.launchpad.net/nova/+bug/1746373

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