← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1625650] [NEW] placement api inventory handler conflates serialization function with response function

 

Public bug reported:

Jay points out, justly, at
https://review.openstack.org/#/c/365633/5/nova/api/openstack/placement/handlers/inventory.py
that the conflation of two functions in one function is meh so it would
be better to disaggregate them sooner than later to avoid repeating the
wrong pattern later.

** Affects: nova
     Importance: Undecided
     Assignee: Chris Dent (cdent)
         Status: New


** Tags: api placement scheduler

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

Title:
  placement api inventory handler conflates serialization function with
  response function

Status in OpenStack Compute (nova):
  New

Bug description:
  Jay points out, justly, at
  https://review.openstack.org/#/c/365633/5/nova/api/openstack/placement/handlers/inventory.py
  that the conflation of two functions in one function is meh so it
  would be better to disaggregate them sooner than later to avoid
  repeating the wrong pattern later.

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


Follow ups