[openstack-dev] [ironic] [api] Fixing bugs in old microversions

Vladyslav Drok vdrok at mirantis.com
Tue Apr 12 09:54:49 UTC 2016


Thank you Jay, Dmitry and Sean for your input! On the yesterday's ironic
meeting the consensus
was to leave the removal possibility in older api versions and not to
bikeshed with new microversions.

Vlad

On Mon, Apr 11, 2016 at 5:36 PM, Jay Pipes <jaypipes at gmail.com> wrote:

> On 04/11/2016 10:11 AM, Sean Dague wrote:
>
>> On 04/11/2016 09:54 AM, Jay Pipes wrote:
>>
>>> On 04/11/2016 09:48 AM, Dmitry Tantsur wrote:
>>>
>>>> On 04/11/2016 02:00 PM, Jay Pipes wrote:
>>>>
>>>>> On 04/11/2016 04:48 AM, Vladyslav Drok wrote:
>>>>>
>>>>>> Hi all!
>>>>>>
>>>>>> There is a bug <https://bugs.launchpad.net/ironic/+bug/1565663> in
>>>>>> ironic API that allows to remove node name using any API version,
>>>>>> while node names were added in version 1.5. There are concerns that
>>>>>> fixing this might
>>>>>> be a breaking change, and I'm not sure how to proceed with that.
>>>>>> Here is
>>>>>> a change <https://review.openstack.org/300983> that
>>>>>> fixes the bug by just forbidding to do PATCH remove request on /name
>>>>>> path if requested
>>>>>> API version is less than 1.5. Is it enough to just mention this in a
>>>>>> release note, maybe
>>>>>> both in fixes and upgrade sections? As bumping API microversion to fix
>>>>>> some previous
>>>>>> microversion seems weird to me.
>>>>>>
>>>>>> Any suggestions?
>>>>>>
>>>>>
>>>>> I think the approach you've taken -- just fix it and not add a new
>>>>> microversion -- is the correct approach.
>>>>>
>>>>
>>>> Do we really allow breaking API changes, covering old microversions?
>>>>
>>>
>>> Generally we have said that if a patch is fixing only an error response
>>> code (as would be the case here -- changing from a 202 to a 400 when
>>> name is attempted to be changed) then it doesn't need a microversion.
>>>
>>> Sean, am I remembering that correctly?
>>>
>>
>> No, in Nova land a 2xx -> 4xx would use a microversion. These sorts of
>> things actually break people (we've seen it happen in Tempest / Shade).
>>
>> Fixing a 5xx does not, as the server is never supposed to 5xx. 5xx is
>> always a bug.
>>
>
> OK, my apologies Vlad and Dmitry. This is why I defer to Sean :)
>
> Best,
> -jay
>
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160412/90308ebd/attachment.html>


More information about the OpenStack-dev mailing list