Reviewed: https://review.openstack.org/516233 Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=0965e7ae6e17c4413571792c7e3a13e8f11400d6 Submitter: Zuul Branch: master
commit 0965e7ae6e17c4413571792c7e3a13e8f11400d6 Author: Takashi NATSUME <natsume.taka...@lab.ntt.co.jp> Date: Mon Oct 30 18:42:47 2017 +0900 [placement] Fix parameter order in placement API ref Optional parameters should be after required paramenters. Parameters added in microversions should be after original parameters. So fix them. Change-Id: I5e3f5df5601d3747debed273e09a8b9b18046a96 Closes-Bug: #1728557 ** Changed in: nova Status: In Progress => Fix Released -- 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/1728557 Title: placement-api-ref: wrong parameter order Status in OpenStack Compute (nova): Fix Released Bug description: Nova API reference guideline(*1) says: ------------------------------------------ 3. Order the fields as follows 1. header 2. path 3. query 4. body 1. top level object (i.e. server) 2. required fields 3. optional fields 4. fields added in microversions (by the microversion they were added) ------------------------------------------ Optional fields should be after required fields. But request bodies of the following APIs do not follows the guideline. They are reference for placement API, but it should be fixed for consistency. - PUT /resource_providers/{uuid}/inventories - PUT /resource_providers/{uuid}/inventories/{resource_class} *1: https://wiki.openstack.org/wiki/NovaAPIRef To manage notifications about this bug go to: https://bugs.launchpad.net/nova/+bug/1728557/+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