openstack/nova
Code Issues Proposed changes

Merge "Not talking about V2 API code in review doc guide"

Browse Source
This commit is contained in:
Jenkins 2016-06-23 15:18:59 +00:00 committed by Gerrit Code Review
commit 81d0bf4619
1 changed files with 0 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
doc/source/code-review.rst
View File

@ -116,21 +116,6 @@ very long. So here are some key points:
* For all other successful operations: ``200 OK`` * For all other successful operations: ``200 OK``
Fixing bugs in v2.0
-------------------
The v2.1 API was made the default in Liberty [1]_ and the ability to disable
the v2.1 API was removed in Newton [2]_. The v2.0 legacy API code is also
deprecated [3]_. As such, latent bugs do not need to be fixed in the legacy
v2.0 API code, with the exception of critical bugs like security
vulnerabilities. Also, new regressions should be avoided, like if raising a
new exception would result in a 500 response from the REST API, that should be
handled in the v2.0 API code as well as the v2.1 API.
.. [1] https://github.com/openstack/nova/commit/4a18f7d3bafcdbede48500aac389e0a770b8e6a8
.. [2] https://github.com/openstack/nova/commit/e65557c1933a563a106763e06d0d4f564d7a4174
.. [3] https://github.com/openstack/nova/commit/fe057636a5eb5b95d03c15f13fff20f76b049d6c
Config Options Config Options
============== ==============