Remove legacy v2 API code

Registered by Ken'ichi Ohmichi

There are two implementation code for similar API in Nova repository.
One is newer: v2.1 API, another is legacy: v2 API.
v2.1 API has been used as the default API since Liberty and legacy v2 API has been marked as deprecated.
We have used and tested v2.1 API so well and now is nice time to remove legacy API code based on the consensus of the design summit of Austin.
https://etherpad.openstack.org/p/newton-nova-api

Blueprint information

Status:
Complete
Approver:
Matt Riedemann
Priority:
High
Drafter:
Ken'ichi Ohmichi
Direction:
Approved
Assignee:
Ken'ichi Ohmichi
Definition:
Approved
Series goal:
Accepted for newton
Implementation:
Implemented
Milestone target:
milestone icon newton-1
Started by
Matt Riedemann
Completed by
Matt Riedemann

Related branches

Sprints

Whiteboard

We discussed this during the newton summit design session, we don't need a spec for this, the work items could be enumerated in the blueprint. This is just removing deprecated code. We obviously need a release note with the removal change though. -- mriedem 20160429

Remove experimental job of this legacy code before doing this: https://review.openstack.org/#/c/311360/

Gerrit topic: https://review.openstack.org/#q,topic:bp/remove-legacy-v2-api-code,n,z

Addressed by: https://review.openstack.org/311653
    WIP: Remove legacy v2 API code completely

Addressed by: https://review.openstack.org/311656
    Remove legacy v2 unit tests[q-v]

Addressed by: https://review.openstack.org/311658
    Remove legacy v2 unit tests[f-n]

Addressed by: https://review.openstack.org/311659
    Remove legacy v2 unit tests[a-e]

Addressed by: https://review.openstack.org/312313
    Remove the legacy v2 API test scenarios from API sample tests

Addressed by: https://review.openstack.org/312314
    Remove the legacy v2 API entry from api-paste.ini

Addressed by: https://review.openstack.org/312320
    Fix unit tests for v2.1 API

Addressed by: https://review.openstack.org/312422
    Remove legacy v2 API functional tests

Addressed by: https://review.openstack.org/312909
    Remove Limits dependency of legacy v2 API code

Addressed by: https://review.openstack.org/312910
    Remove fping dependency on legacy v2 API code

Addressed by: https://review.openstack.org/313185
    Remove legacy v2 tests which use APIRouter

Addressed by: https://review.openstack.org/313191
    Make DEFAULT of list_opts() use v2.1 API code

Addressed by: https://review.openstack.org/313192
    Remove legacy v2 API tests which use wsgi_app()

Addressed by: https://review.openstack.org/313901
    Switch api unit tests to use v2.1 API

Addressed by: https://review.openstack.org/313902
    Remove APIRouter of legacy v2 API code

Gerrit topic: https://review.openstack.org/#q,topic:remove-legacy,n,z

Addressed by: https://review.openstack.org/320751
    Remove legacy v2 policy rules

Addressed by: https://review.openstack.org/320752
    Remove all the policy check points from ComputeAPI and SecurityGroupAPI

Addressed by: https://review.openstack.org/321235
    Remove unused use_no_auth from wsgi_app_v21()

Addressed by: https://review.openstack.org/321240
    Remove unused args from RateLimitingMiddleware

Addressed by: https://review.openstack.org/321247
    Remove unused WsgiLimiter

Addressed by: https://review.openstack.org/321259
    Move RateLimitingMiddleware into fakes of unittest

Addressed by: https://review.openstack.org/321498
    Remove InstanceUsageAuditLogTest for legacy API

Addressed by: https://review.openstack.org/321566
    Remove legacy API code from sample tests

Addressed by: https://review.openstack.org/324043
    Remove unused inner_app_v21 and ext_mgr

Addressed by: https://review.openstack.org/325641
    Remove policy checkpoints for SecurityGroupAPI and NetworkAPI

Gerrit topic: https://review.openstack.org/#q,topic:remove_legacy_v2_policy,n,z

Addressed by: https://review.openstack.org/325684
    Remove the skip_policy_check flags

Addressed by: https://review.openstack.org/326810
    Remove v2 extension setting from functional tests

Addressed by: https://review.openstack.org/326822
    Remove v2 extension setting from functional tests

Addressed by: https://review.openstack.org/327066
    Update stable API doc to indicate code removal

Addressed by: https://review.openstack.org/327265
    WIP: Remove unused authorizer methods

Addressed by: https://review.openstack.org/328931
    Remove python code validation specific to legacy_v2

Addressed by: https://review.openstack.org/329133
    Remove core extension list

Addressed by: https://review.openstack.org/329554
    Remove api_rate_limit config option

Gerrit topic: https://review.openstack.org/#q,topic:fold_disk_config,n,z

Addressed by: https://review.openstack.org/331799
    remove load_standard_extensions method

Addressed by: https://review.openstack.org/333125
    Remove rate_limit param in builder

Addressed by: https://review.openstack.org/333134
    Not talking about V2 API code in review doc guide

If there is lingering cleanup we can still track it against this blueprint but I'm going to mark it done. -- mriedem 20160629

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.