Use Neutron by default

Registered by Matt Riedemann

This is a specless blueprint to track the work to change the default value on the CONF.use_neutron option to True. That change itself is trivial, but there is a ton of impact to unit tests by doing that as the tests were mostly written with nova-network in mind, or at least just worked that way, especially with the global network API stub code all over the tests which stubs out nova-network API.

The process to work through this is to basically set CONF.use_neutron=True by default but don't commit that, and run the tests, then focus on a specific set of tests or test module and convert those one at a time to work with use_neutron=True as a default value. If the tests are specifically designed for nova-network, then set self.flags(use_neutron=False).

At the very end of the series we should have a change that sets CONF.use_neutron=True in nova/conf/network.py by default along with a release note for the upgrade impact.

If we get too close to the end of the release and all of the tests aren't converted yet, we can still set CONF.use_neutron=True by default if we just globally set use_neutron=False in the base test class setUp, but we'd continue to need to work on cleaning up tests as nova-network is deprecated and going away.

Blueprint information

Status:
Complete
Approver:
John Garbutt
Priority:
Medium
Drafter:
Matt Riedemann
Direction:
Approved
Assignee:
Matt Riedemann
Definition:
Approved
Series goal:
Accepted for ocata
Implementation:
Implemented
Milestone target:
milestone icon ocata-3
Started by
Matt Riedemann
Completed by
Matt Riedemann

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/use-neutron-by-default,n,z

Addressed by: https://review.openstack.org/396359
    neutron: handle no_allocate in create_pci_requests_for_sriov_ports

Addressed by: https://review.openstack.org/396824
    Make test_serversV21 tests work with neutron by default

Addressed by: https://review.openstack.org/396864
    Explicitly use nova-network in nova-network network tests

Addressed by: https://review.openstack.org/396900
    Make vmware test_driver_api tests work with CONF.use_neutron=True

Addressed by: https://review.openstack.org/396904
    Make libvirt unit tests work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/396905
    Make xenapi tests work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/396945
    Make nova.tests.unit.virt pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/396957
    Make test_compute_api work with CONF.use_default=True by default

Addressed by: https://review.openstack.org/396965
    Make test_compute_mgr work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/397336
    Restrict test_compute_cells to nova-network

Addressed by: https://review.openstack.org/397365
    Make test_shelve work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/399978
    Make test_compute pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415707
    Make test_quota work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415729
    Make test_user_data and test_multiple_create work with use_neutron=True

Addressed by: https://review.openstack.org/415732
    Make test_virtual_interfaces work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415747
    Make test_security_groups work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415755
    Make test_tenant_networks pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415767
    Make test_security_group_default_rules work with use_neutron=True by default

Addressed by: https://review.openstack.org/415770
    Make test_server_usage work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415771
    Make test_config_drive pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415775
    Make test_keypairs work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415781
    Make test_scheduler_hints work with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/415999
    Make test_floating_ip* pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/416001
    Make test_attach_interfaces work with use_neutron=True by default

Addressed by: https://review.openstack.org/416002
    Stub out os_vif.unplug in libvirt instance destroy tests

Addressed by: https://review.openstack.org/416003
    Make test_nova_manage pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/416004
    Make test_metadata pass with CONF.use_neutron=True by default

Addressed by: https://review.openstack.org/416006
    Make last remaining unit tests work with Neutron by default

Addressed by: https://review.openstack.org/416083
    WIP: Hack functional tests to run nova-network explicitly

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.