yahoo-eng-team team mailing list archive

[Lajos Katona <katonalala@xxxxxxxxx>]

Public bug reported:

The patch https://review.opendev.org/663980 introduced a serious misreading of placement API.
The lines https://review.opendev.org/#/c/663980/2/neutron/services/segments/plugin.py@220 assumes that "Show resource provider inventory" (see: https://developer.openstack.org/api-ref/placement/?expanded=show-resource-provider-inventory-detail#show-resource-provider-inventory) returns a dict with like 
{'IPV4_ADDRESS': {'allocation_ratio': 42....}}
but if we read the documentation the truth is that the response is a dict like:
{'allocation_ratio': 42....}

The other fix in that patch is good as it is
(https://review.opendev.org/#/c/663980/2/neutron/services/segments/plugin.py@255)
for "Update resource provider inventories" (see:
https://developer.openstack.org/api-ref/placement/?expanded=update-
resource-provider-inventories-detail#update-resource-provider-
inventories)

** Affects: neutron
     Importance: Undecided
     Assignee: Lajos Katona (lajos-katona)
         Status: New

** Changed in: neutron
     Assignee: (unassigned) => Lajos Katona (lajos-katona)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1836037

Title:
  Routed provider networks nova inventory update fails

Status in neutron:
  New

Bug description:
  The patch https://review.opendev.org/663980 introduced a serious misreading of placement API.
  The lines https://review.opendev.org/#/c/663980/2/neutron/services/segments/plugin.py@220 assumes that "Show resource provider inventory" (see: https://developer.openstack.org/api-ref/placement/?expanded=show-resource-provider-inventory-detail#show-resource-provider-inventory) returns a dict with like 
  {'IPV4_ADDRESS': {'allocation_ratio': 42....}}
  but if we read the documentation the truth is that the response is a dict like:
  {'allocation_ratio': 42....}

  The other fix in that patch is good as it is
  (https://review.opendev.org/#/c/663980/2/neutron/services/segments/plugin.py@255)
  for "Update resource provider inventories" (see:
  https://developer.openstack.org/api-ref/placement/?expanded=update-
  resource-provider-inventories-detail#update-resource-provider-
  inventories)

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