RFE Replace the existing default subnetpool configuration options with an admin-only API

Bug #1501328 reported by John Davidge
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Wishlist
John Davidge

Bug Description

During the Liberty cycle a consensus was reached within the neutron l3 subteam that the existing method for setting default subnetpools in neutron.conf was not working as well as it could.

As an admin, I want to be able to set the default subnetpool via the neutron API, and without the need to restart services for the change to take effect.

The proposed solution is to add a new boolean 'is_default' field to the subnetpool object. This field will be False by default, configurable only by the admin, and initially can only be True for one subnetpool in each IP family. Future work could allow multiple defaults once RBAC is implemented for subnetpools.

This change will greatly improve the default subnetpool workflow for admins, as well as exposing information about the default subnetpool to users.

Changed in neutron:
assignee: nobody → John Davidge (john-davidge)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: New → In Progress
Changed in neutron:
status: In Progress → Triaged
Revision history for this message
Carl Baldwin (carl-baldwin) wrote : Re: Replace the existing default subnetpool configuration options with an admin-only API

Let's go!

Changed in neutron:
importance: Undecided → Medium
tags: added: rfe-approved
removed: rfe
Changed in neutron:
milestone: none → mitaka-1
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

So code is already posted...which is cool

Changed in neutron:
status: Triaged → In Progress
Changed in neutron:
importance: Medium → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/230983
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=6ee91e56c84ea6e930369f8649f1d2b50096cb80
Submitter: Jenkins
Branch: master

commit 6ee91e56c84ea6e930369f8649f1d2b50096cb80
Author: John Davidge <email address hidden>
Date: Thu Oct 1 20:37:34 2015 +0100

    Replace subnetpool config options with admin-only API

    This patch adds a new boolean 'is_default' property to subnetpools. This
    allows the admin to set the default v4/v6 subnetpools via the API rather
    than the existing neutron.conf options - which are deprecated by this patch.

    Only one subnetpool per IP family can be set to default.

    DocImpact
    ApiImpact

    Co-Authored-By: Carl Baldwin <email address hidden>

    Change-Id: I5daba2347cfb91fac0b155b2c1b459ee7d9e4505
    Closes-Bug: 1501328

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/neutron 8.0.0.0b1

This issue was fixed in the openstack/neutron 8.0.0.0b1 development milestone.

Changed in neutron:
status: Fix Committed → Fix Released
summary: - Replace the existing default subnetpool configuration options with an
- admin-only API
+ RFE Replace the existing default subnetpool configuration options with
+ an admin-only API
tags: added: deprecation
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

Reviewed: https://review.openstack.org/321769
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=f7cc6a0107bd330150e2136e5a3dac99b6b2c33d
Submitter: Jenkins
Branch: master

commit f7cc6a0107bd330150e2136e5a3dac99b6b2c33d
Author: John Davidge <email address hidden>
Date: Thu May 26 17:42:04 2016 +0100

    Remove deprecated default subnetpools

    These config options were deprecated in Mitaka.
    They can now be removed in Newton.

    Closes-Bug: #1594810
    Related-Bug: #1501328
    Change-Id: I6eea7d4465cf23df1d8dae26336633052dfab871

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.