Remove nova-network (final)

Registered by Stephen Finucane on 2019-09-24

The nova-network service has been deprecated since Newton and with the recent removal of cells v1 can no longer be used

This blueprint is for tracking the complete removal of nova-network which will likely impact a lot of the code base (docs, tests, APIs, CI job configuration, etc).

Blueprint information

Status:
Complete
Approver:
Eric Fried
Priority:
Low
Drafter:
Stephen Finucane
Direction:
Approved
Assignee:
Stephen Finucane
Definition:
Approved
Series goal:
Accepted for ussuri
Implementation:
Implemented
Milestone target:
milestone icon ussuri-3
Started by
Eric Fried on 2019-10-03
Completed by
Stephen Finucane on 2020-03-27

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.opendev.org/#/q/topic:bp/remove-nova-network-ussuri

Addressed by: https://review.opendev.org/684330
    nova-net: Disable neutron explicitly for 'ServersSampleBase' subclasses

Addressed by: https://review.opendev.org/684331
    nova-net: Remove *even* more unnecessary 'USE_NEUTRON = False'

Addressed by: https://review.opendev.org/684332
    nova-net: Migrate 'test_availability_zone' functional tests

Addressed by: https://review.opendev.org/684333
    nova-net: Migrate 'test_floating_ip_pools' functional tests

Addressed by: https://review.opendev.org/684334
    nova-net: Migrate 'test_quota_sets' functional tests

Addressed by: https://review.opendev.org/684335
    nova-net: Migrate 'test_server_tags' functional tests

Addressed by: https://review.opendev.org/684336
    nova-net: Migrate 'test_servers' functional tests

Addressed by: https://review.opendev.org/684337
    nova-net: Migrate 'test_hosts' functional tests

Addressed by: https://review.opendev.org/684338
    nova-net: Migrate 'test_networks_associate' functional tests

Addressed by: https://review.opendev.org/684339
    nova-net: Migrate 'test_rescue' functional tests

Addressed by: https://review.opendev.org/684340
    nova-net: Migrate 'test_hypervisors' functional tests

Addressed by: https://review.opendev.org/684341
    nova-net: Migrate 'test_attach_interfaces' functional tests

Addressed by: https://review.opendev.org/684342
    nova-net: Migrate 'test_simple_tenant_usage' functional tests

Addressed by: https://review.opendev.org/684343
    trivial: Neutron fixture cleanup

Addressed by: https://review.opendev.org/684344
    nova-net: Migrate 'test_floating_ips' functional tests

Addressed by: https://review.opendev.org/684345
    nova-net: Add TODOs for remaining nova-network functional tests

Addressed by: https://review.opendev.org/684326
    nova-net: Use nova-net explicitly in '_IntegratedTestBase' subclasses

Addressed by: https://review.opendev.org/684328
    nova-net: Use nova-net explicitly in 'ApiSampleTestBase' subclasses

Addressed by: https://review.opendev.org/685927
    tests: Correctly mock out security groups in NeutronFixture

Addressed by: https://review.opendev.org/685937
    nova-net: Remove explicit 'USE_NEUTRON = True'

[efried 20191003] agreed to approve "definition" of this specless bp in today's nova meeting, leaving open the possibility of deferring the "direction" between now and spec freeze time if it falls "below the line".

Addressed by: https://review.opendev.org/686795
    fixtures: Beef up NeutronFixture

Addressed by: https://review.opendev.org/686796
    fixtures: Handle iterable params for 'NeutronFixture.list_*'

Addressed by: https://review.opendev.org/686797
    fixtures: Store 'device_id' when creating port in NeutronFixture

Addressed by: https://review.opendev.org/686798
    trivial: Change name of network provided by NeutronFixture

Addressed by: https://review.opendev.org/686799
    nova-net: Stop mocking the instance network cache

Addressed by: https://review.opendev.org/686800
    trivial: Make it obvious where we're getting our names from

Addressed by: https://review.opendev.org/686801
    nova-net: Make even more nova-net stuff optional

Addressed by: https://review.opendev.org/686802
    fixtures: Add support for security groups

Addressed by: https://review.opendev.org/686807
    Remove 'os-security-group-default-rules' REST API

Addressed by: https://review.opendev.org/686808
    nova-net: Remove unused '*_default_rules' security group DB APIs

Addressed by: https://review.opendev.org/686809
    Remove (most) '/os-networks' REST APIs

Addressed by: https://review.opendev.org/686810
    Remove '/os-tenant-networks' REST API

Addressed by: https://review.opendev.org/686811
    nova-net: Remove 'USE_NEUTRON' from functional tests

Addressed by: https://review.opendev.org/686812
    nova-net: Remove 'networks' quota

Addressed by: https://review.opendev.org/686813
    Remove 'nova-manage network' commands

Addressed by: https://review.opendev.org/686814
    nova-net: Remove associate, disassociate network APIs

Addressed by: https://review.opendev.org/686815
    nova-net: Remove 'nova-dhcpbridge' binary

Addressed by: https://review.opendev.org/686816
    nova-net: Remove 'nova-network' binary

Addressed by: https://review.opendev.org/686817
    docs: Blast most references to nova-network

Addressed by: https://review.opendev.org/686818
    WIP

Addressed by: https://review.opendev.org/686901
    trivial: Remove unused API sample template

Addressed by: https://review.opendev.org/688139
    nova-net: Use deepcopy on value returned by NeutronFixture

Addressed by: https://review.opendev.org/696498
    docs: Blast final references to nova-network

Addressed by: https://review.opendev.org/696499
    nova-net: Remove final references to nova-net from functional tests

Addressed by: https://review.opendev.org/696500
    libvirt: Remove 'enable_hairpin'

Addressed by: https://review.opendev.org/696501
    libvirt: Remove vestigial nova-network support

Addressed by: https://review.opendev.org/696502
    hyperv: Remove vestigial nova-network support

Addressed by: https://review.opendev.org/696503
    vmware: Remove vestigial nova-network support

Addressed by: https://review.opendev.org/696504
    zvm: Remove vestigial nova-network support

Addressed by: https://review.opendev.org/696505
    xenapi: Remove vestigial nova-network support

Addressed by: https://review.opendev.org/696506
    nova-net: Flatten class hierarchy for neutron SG tests

Addressed by: https://review.opendev.org/696507
    nova-net: Remove SG tests that don't apply to neutron

Addressed by: https://review.opendev.org/696508
    nova-net: Drop nova-network-base security group tests

Addressed by: https://review.opendev.org/696509
    nova-net: Convert remaining API tests to use neutron

Addressed by: https://review.opendev.org/696510
    nova-net: Convert remaining unit tests to neutron

Addressed by: https://review.opendev.org/696511
    nova-net: Remove 'is_neutron_security_groups' function

Addressed by: https://review.opendev.org/696512
    nova-net: Remove nova-network security group driver

Addressed by: https://review.opendev.org/696513
    nova-net: Correct some broken VIF tests

Addressed by: https://review.opendev.org/696514
    nova-net: Remove firewall support

Addressed by: https://review.opendev.org/696515
    nova-net: Copy shared utils from nova-net module

Addressed by: https://review.opendev.org/696516
    nova-net: Remove final references to nova-network

Addressed by: https://review.opendev.org/696517
    objects: Add TODOs to remove security group-related objects

Addressed by: https://review.opendev.org/696518
    WIP: Remove 'nova-network'

Addressed by: https://review.opendev.org/696740
    Follow up I18d73212f9d98bc75974a024cf6fd872fdfb1ca4

Addressed by: https://review.opendev.org/696745
    Rename 'nova.network.neutronv2' -> 'nova.network'

Addressed by: https://review.opendev.org/696746
    Rename 'nova.network.security_group.neutron_driver' -> 'nova.network.security_group'

Addressed by: https://review.opendev.org/696748
    nova-net: Remove 'MetadataManager'

Addressed by: https://review.opendev.org/696749
    nova-net: Remove dependency on nova-net from fake cache

Addressed by: https://review.opendev.org/696776
    Remove unnecessary 'neutronv2' prefixes

Addressed by: https://review.opendev.org/696844
    Remove rootwrap filters for nova network

Addressed by: https://review.opendev.org/697149
    nova-net: Remove unused exceptions

Addressed by: https://review.opendev.org/697150
    nova-net: Remove db methods for ProviderMethod

Addressed by: https://review.opendev.org/697151
    nova-net: Remove unused 'stub_out_db_network_api'

Addressed by: https://review.opendev.org/697152
    nova-net: Remove remaining nova-network quotas

Addressed by: https://review.opendev.org/697153
    nova-net: Remove use of legacy 'FloatingIP' object

Addressed by: https://review.opendev.org/697154
    nova-net: Remove use of legacy 'Network' object

Addressed by: https://review.opendev.org/697155
    nova-net: Remove use of legacy 'SecurityGroup' object

Addressed by: https://review.opendev.org/697156
    nova-net: Remove unused nova-network objects

Addressed by: https://review.opendev.org/697252
    nova-net: Follow-up for I12a96ea659ed402cc4d1bd52a50e2e16042b6372

Addressed by: https://review.opendev.org/700510
    nova-net: Remove firewall support (pt. 2)

Addressed by: https://review.opendev.org/700511
    nova-net: Remove firewall support (pt. 3)

Addressed by: https://review.opendev.org/702440
    Remove now unnecessary nova-network workaround

Addressed by: https://review.opendev.org/702451
    Remove 'nova.image.api' module

Addressed by: https://review.opendev.org/703796
    nova-net: Update API reference guide

Addressed by: https://review.opendev.org/703973
    nova-net: Remove unnecessary exception handling

Addressed by: https://review.opendev.org/703974
    nova-net: Remove unused parameters

[efried 20200220] Agreed in the Nova meeting to Direction:Approve all Definition:Approved blueprints http://eavesdrop.openstack.org/meetings/nova/2020/nova.2020-02-20-14.00.log.html#l-131

(?)

Work Items

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.