OSAPI wsgi-related code needs to be grouped into a common module
Bug #752782 reported by
Brian Waldon
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Compute (nova) |
Fix Released
|
Wishlist
|
Brian Waldon | ||
Bug Description
> We should keep as much code out of api/openstack/
> possible. We have already disregarded that guideline and put the OS-specific
> wsgi code directly in __init__.py (such as APIRouter and FaultWrapper). My end
> goal here is to make a wsgi-specific module (perhaps api/openstack/
> ... I would be fine leaving it in common.py with the assumption that it will
> be moving for Diablo.
Reporting this simply so we remember to get this done for Diablo (at Vish's request).
Related branches
lp:~rackspace-titan/nova/osapi-serialization
- Dan Prince (community): Approve
- Ed Leafe (community): Approve
- Mark Washenberger (community): Approve
-
Diff: 3221 lines (+1177/-827)30 files modifiednova/api/direct.py (+8/-4)
nova/api/openstack/__init__.py (+23/-20)
nova/api/openstack/accounts.py (+21/-12)
nova/api/openstack/backup_schedules.py (+18/-9)
nova/api/openstack/common.py (+0/-7)
nova/api/openstack/consoles.py (+18/-11)
nova/api/openstack/contrib/volumes.py (+10/-13)
nova/api/openstack/extensions.py (+58/-41)
nova/api/openstack/faults.py (+23/-16)
nova/api/openstack/flavors.py (+23/-15)
nova/api/openstack/image_metadata.py (+12/-8)
nova/api/openstack/images.py (+33/-19)
nova/api/openstack/ips.py (+22/-15)
nova/api/openstack/limits.py (+34/-15)
nova/api/openstack/server_metadata.py (+15/-10)
nova/api/openstack/servers.py (+68/-64)
nova/api/openstack/shared_ip_groups.py (+10/-22)
nova/api/openstack/users.py (+26/-17)
nova/api/openstack/versions.py (+21/-26)
nova/api/openstack/wsgi.py (+380/-0)
nova/api/openstack/zones.py (+21/-12)
nova/objectstore/s3server.py (+1/-1)
nova/tests/api/openstack/extensions/foxinsocks.py (+1/-3)
nova/tests/api/openstack/test_extensions.py (+2/-2)
nova/tests/api/openstack/test_limits.py (+2/-2)
nova/tests/api/openstack/test_servers.py (+29/-23)
nova/tests/api/openstack/test_wsgi.py (+293/-0)
nova/tests/api/test_wsgi.py (+0/-189)
nova/tests/integrated/test_xml.py (+2/-2)
nova/wsgi.py (+3/-249)
| Changed in nova: | |
| importance: | Undecided → Wishlist |
| status: | New → Confirmed |
| Changed in nova: | |
| assignee: | nobody → Brian Waldon (bcwaldon) |
| Changed in nova: | |
| status: | Confirmed → In Progress |
| Changed in nova: | |
| status: | In Progress → Fix Committed |
| Changed in nova: | |
| milestone: | none → diablo-2 |
| Changed in nova: | |
| milestone: | diablo-2 → 2011.3 |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
