Comment 5 for bug 1826396

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron-specs (master)

Reviewed: https://review.opendev.org/676365
Committed: https://git.openstack.org/cgit/openstack/neutron-specs/commit/?id=02b9204aa18b96d8c9c0c36dc9767c3d1d947fed
Submitter: Zuul
Branch: master

commit 02b9204aa18b96d8c9c0c36dc9767c3d1d947fed
Author: Bence Romsics <email address hidden>
Date: Wed Aug 14 09:27:48 2019 +0200

    Retroactively update improve-extraroute-api spec

    As we discovered during implementation it wasn't a good idea of mine to
    make the request and response body a list on the top level. Neither the
    documentation tools nor python-neutronclient were prepared for non-dict
    request/response bodies.

    Another reason I suspect to always have dict-wrapped bodies is that it
    makes possible future changes to the request/response format a bit easier.

    Yet another reason is that the request/response body format in this change
    is now the exact same as it was used in the PUT /v2.0/routers/router-id
    request/response. Therefore clients wishing to use the new API only have
    to change the URL, but not the body of their requests.

    Since the api-def was already merged, this change just retroactively
    updates the spec to avoid possible future confusion between the spec
    and the api-ref.

    Change-Id: I3c7e9ab4ab6518eb1d30e3711a2d9571dfc2fa15
    Partial-Bug: #1826396