dhcp_options_enabled option be removed when neutron supports

Bug #1214162 reported by dkehn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Rohan

Bug Description

The dhcp_options_enabled is currently set to False because the neutron extra_dhcp_opts patches have not landed yet. This should land in the Havana release of Neutron, as such dhcp_options_enabled option can then be defaulted to True or the conditional in the code can be removed (the desired approach). This is explained in https://review.openstack.org/#/c/31061/, and in Neutron: https://review.openstack.org/#/c/30441/ and https://review.openstack.org/#/c/30447/, for the python-neutronclient. So for Icehouse release of Nova this option should be removed and the code modified to support the dhcp_options.

Tags: network
Matt Riedemann (mriedem)
tags: added: network
Changed in nova:
status: New → Confirmed
importance: Undecided → Low
milestone: none → icehouse-1
Rohan (kanaderohan)
Changed in nova:
assignee: nobody → Rohan (kanaderohan)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/58089

Changed in nova:
status: Confirmed → In Progress
Changed in nova:
milestone: icehouse-1 → icehouse-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/58089
Committed: http://github.com/openstack/nova/commit/3292103d80676fc789615465759de966dba4e9f7
Submitter: Jenkins
Branch: master

commit 3292103d80676fc789615465759de966dba4e9f7
Author: Rohan Kanade <email address hidden>
Date: Sat Nov 23 07:44:28 2013 +0100

    Removes check CONF.dhcp_options_enabled from nova

    * Neutron supports extra_dhcp_opts since Havana
    * Hence remove boolean check dhcp_options_enabled

    Closes-Bug: #1214162
    DocImpact
    Change-Id: I9af217a87a237e078e70332cbed1c80ad4d5db0a

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-2 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.