OpenStack Compute (nova)

Remove cells v1

Registered by Stephen Finucane

The cells v1 service has been deprecated since Pike in favour of cells v2. It's a blocker for the removal of nova-net and makes the compute API far more complicated that it needs to be. Time to remove it.

Blueprint information

Status:
Complete
Approver:
Eric Fried
Priority:
Medium
Drafter:
Stephen Finucane
Direction:
Approved
Assignee:
Stephen Finucane
Definition:
Approved
Series goal:
Accepted for train
Implementation:
Implemented
Milestone target:
milestone icon train-3
Started by
Matt Riedemann
Completed by
Matt Riedemann

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#/q/topic:bp/remove-cells-v1

Addressed by: https://review.openstack.org/651289
    Remove cells v1 jobs

Addressed by: https://review.openstack.org/651290
    Remove 'nova-cells' service

Addressed by: https://review.openstack.org/651291
    Remove '/os-cells' REST APIs

Addressed by: https://review.openstack.org/651292
    Stop handling cells v1 in '/os-hypervisors' API

Addressed by: https://review.openstack.org/651293
    Stop handling cells v1 in '/os-servers' API

Addressed by: https://review.openstack.org/651294
    Remove 'nova-manage cell' commands

Addressed by: https://review.openstack.org/651295
    Stop handling cells v1 for console authentication

Addressed by: https://review.openstack.org/651296
    Remove old-style cell v1 instance listing

Addressed by: https://review.openstack.org/651297
    Remove 'bdm_(update_or_create|destroy)_at_top'

Addressed by: https://review.openstack.org/651298
    Remove 'instance_fault_create_at_top'

Addressed by: https://review.openstack.org/651299
    Remove 'instance_info_cache_update_at_top'

Addressed by: https://review.openstack.org/651300
    Remove 'get_keypair_at_top'

Addressed by: https://review.openstack.org/651301
    Remove 'instance_update_at_top', 'instance_destroy_at_top'

Addressed by: https://review.openstack.org/651302
    Remove 'instance_update_from_api'

Addressed by: https://review.openstack.org/651303
    Stop handling 'update_cells' on 'BandwidthUsage.create'

Addressed by: https://review.openstack.org/651304
    Stop handling cells v1 for instance naming

Addressed by: https://review.openstack.org/651305
    Stop handling 'cell_name' field on Instance.save()

Addressed by: https://review.openstack.org/651306
    Remove cells code

Addressed by: https://review.openstack.org/651307
    Stop handling 'InstanceUnknownCell' exception

Addressed by: https://review.openstack.org/651308
    Remove unnecessary wrapper

Addressed by: https://review.openstack.org/651309
    db: Remove cell APIs

Addressed by: https://review.openstack.org/651310
    conf: Remove cells v1 options, group

Addressed by: https://review.openstack.org/651311
    Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance'

Addressed by: https://review.openstack.org/651312
    Stop passing 'kwargs' to 'rebuild_instance'

Addressed by: https://review.openstack.org/651313
    Stop passing 'delete_type' to 'terminate_instance'

Addressed by: https://review.openstack.org/651314
    filters: Stop handling cells v1

Addressed by: https://review.openstack.org/651315
    WIP: Enable multiattach in all cases

Addressed by: https://review.openstack.org/651316
    WIP: conductor: Remove cells v1 workarounds

efried 20190411 agreed to approve as specless in nova meeting http://eavesdrop.openstack.org/meetings/nova/2019/nova.2019-04-11-21.00.log.html#l-106

Addressed by: https://review.openstack.org/652649
    Add TODO to remove function attribute

Gerrit topic: https://review.opendev.org/#/q/topic:bp/remove-cells-v1

Addressed by: https://review.opendev.org/651298
    Remove 'instance_fault_create_at_top'

Addressed by: https://review.opendev.org/651299
    Remove 'instance_info_cache_update_at_top'

Addressed by: https://review.opendev.org/651300
    Remove 'get_keypair_at_top'

Addressed by: https://review.opendev.org/651301
    Remove 'instance_update_at_top', 'instance_destroy_at_top'

Addressed by: https://review.opendev.org/651302
    Remove 'instance_update_from_api'

Addressed by: https://review.opendev.org/651303
    Stop handling 'update_cells' on 'BandwidthUsage.create'

Addressed by: https://review.opendev.org/651304
    Stop handling cells v1 for instance naming

Addressed by: https://review.opendev.org/651306
    Remove cells code

Addressed by: https://review.opendev.org/651307
    Stop handling 'InstanceUnknownCell' exception

Addressed by: https://review.opendev.org/651308
    Remove unnecessary wrapper

Addressed by: https://review.opendev.org/651309
    db: Remove cell APIs

Addressed by: https://review.opendev.org/651310
    conf: Remove cells v1 options, group

Addressed by: https://review.opendev.org/651059
    Remove conductor_api and _last_host_check from manager.py

Addressed by: https://review.opendev.org/660146
    Remove 'etc/nova/cells.json'

Addressed by: https://review.opendev.org/660527
    Remove nova.compute.*API() shims

Addressed by: https://review.opendev.org/651311
    Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance'

Addressed by: https://review.opendev.org/651312
    Stop passing 'kwargs' to 'rebuild_instance'

Addressed by: https://review.opendev.org/651313
    Stop passing 'delete_type' to 'terminate_instance'

Addressed by: https://review.opendev.org/651314
    filters: Stop handling cells v1

Addressed by: https://review.opendev.org/660774
    Remove 'ComputeManager._reschedule'

Addressed by: https://review.opendev.org/660950
    Ensure controllers all call super

Addressed by: https://review.opendev.org/662031
    Add reno for removed cells v1 policies

Addressed by: https://review.opendev.org/662411
    Remove 'InstanceUnknownCell' exception

Addressed by: https://review.opendev.org/663873
    Fold ComputeManager._resize to its only caller

Addressed by: https://review.opendev.org/663959
    Use named kwargs for resize_api

Addressed by: https://review.opendev.org/621061
    Drop pre-cinder 3.44 version compatibility

Addressed by: https://review.opendev.org/651315
    Remove 'MultiattachNotSupportedByVirtDriver'

Addressed by: https://review.opendev.org/664967
    Follow-up for I6a777b4b7a5729488f939df8c40e49bd40aec3dd

Addressed by: https://review.opendev.org/651316
    WIP: conductor: Remove cells v1 workarounds

Addressed by: https://review.opendev.org/665146
    WIP: Start rewriting all these tests

Addressed by: https://review.opendev.org/667107
    Remove comments about mirroring changes to nova/cells/messaging.py

There are still a couple of open changes for this series but the removal is effectively complete so that's why we can mark this complete for Train. -- mriedem 20190828

(?)

Work Items

Nearby

This blueprint contains Public information 
Everyone can see this information.
Edit subscription

Subscribers

No subscribers.