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

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to