summaryrefslogtreecommitdiff
path: root/etc/neutron.conf
AgeCommit message (Collapse)Author
2015-11-30Remove Neutron core static example configuration filesMartin Hickey
Oslo config generator was introduced in patch [1] to automatically generate the sample Neutron core configuration files. This patch removes the static example configuration files from the repository as they are now redundant. [1] https://review.openstack.org/#/c/204206/ DocImpact Change-Id: Ic7ae2e038b5bd7b215c65c9c565bfe31ef551520 Partially-Implements: blueprint autogen-neutron-conf-file Closes-bug: #1199963 Depends-On: Ic37a16b6cf8eb92030649f1fc8b198738a8cc104 Notes (review): Verified+2: Jenkins Code-Review-1: Tom Fifield <tom@openstack.org> Code-Review-1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+2: Cedric Brandily <zzelle@gmail.com> Workflow+1: Cedric Brandily <zzelle@gmail.com> Code-Review+2: Ihar Hrachyshka <ihrachys@redhat.com> Code-Review+1: Brandon Palm <bapalm@us.ibm.com> Submitted-by: Jenkins Submitted-at: Thu, 03 Dec 2015 21:13:30 +0000 Reviewed-on: https://review.openstack.org/251348 Project: openstack/neutron Branch: refs/heads/master
2015-11-26Merge "Change instances of Openstack to OpenStack."Jenkins
2015-11-25Change instances of Openstack to OpenStack.venkatamahesh
Change Openstack to OpenStack as per standards Change-Id: I10f33c1e70fb5f2d74e6feda0a4a266d7071e1ab Notes (review): Verified+2: Jenkins Code-Review+2: Russell Bryant <rbryant@redhat.com> Code-Review+1: Sean M. Collins <sean@coreitpro.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Cedric Brandily <zzelle@gmail.com> Workflow+1: Cedric Brandily <zzelle@gmail.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Submitted-by: Jenkins Submitted-at: Thu, 26 Nov 2015 01:43:14 +0000 Reviewed-on: https://review.openstack.org/243134 Project: openstack/neutron Branch: refs/heads/master
2015-11-25Add availability_zone support for networkHirofumi Ichihara
This patch adds the availability_zone support for network. APIImpact DocImpact Change-Id: I9259d9679c74d3b3658771290e920a7896631e62 Co-Authored-By: IWAMOTO Toshihiro <iwamoto@valinux.co.jp> Partially-implements: blueprint add-availability-zone Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Workflow+1: Armando Migliaccio <armamig@gmail.com> Code-Review+2: Akihiro Motoki <amotoki@gmail.com> Code-Review+2: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Paul Michali <pc@michali.net> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Code-Review+1: Ryan Moats <rmoats@us.ibm.com> Submitted-by: Jenkins Submitted-at: Wed, 25 Nov 2015 22:17:42 +0000 Reviewed-on: https://review.openstack.org/204436 Project: openstack/neutron Branch: refs/heads/master
2015-11-18Merge "Add a better description for notification_driver"Jenkins
2015-11-17Merge "Revert "Move dhcp_lease_duration into DHCP agent config options list""Jenkins
2015-11-17Add a better description for notification_driverDongcan Ye
Improve description for the notification_driver property in neutron.conf, so users will understand how to configure it correctly. Change-Id: Ib5d4c4b6eea5cabf58708d8201f9ff9c883b6c97 Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: YAMAMOTO Takashi <yamamoto@midokura.com> Code-Review+1: Victor Howard <victor.r.howard@gmail.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Workflow+1: Carl Baldwin <carl@ecbaldwin.net> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+1: shihanzhang <shihanzhang@huawei.com> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Code-Review+1: Peng Fei Wang <wpf710@gmail.com> Code-Review+1: Ryan Moats <rmoats@us.ibm.com> Submitted-by: Jenkins Submitted-at: Wed, 18 Nov 2015 20:28:57 +0000 Reviewed-on: https://review.openstack.org/245848 Project: openstack/neutron Branch: refs/heads/master
2015-11-16Revert "Move dhcp_lease_duration into DHCP agent config options list"Kevin Benton
This reverts commit 3101e2b62f0741baffc61174637486da17dd29bb. The lease duration isn't DHCP agent specific. DHCP is a feature that should be provided by whatever plugin is loaded and operators would still like to set lease times for DHCP. In the reference implementation we just consume it directly in the agent, but that doesn't mean it's specific to the agent implementation. Change-Id: I5efb6aa5e196d4cac5ff7c0c35be5a350401b925 Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: YAMAMOTO Takashi <yamamoto@midokura.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Code-Review+2: Doug Wiegley <dougwig@parkside.io> Workflow+1: Doug Wiegley <dougwig@parkside.io> Code-Review-1: Dariusz Smigiel <dariusz.smigiel@intel.com> Submitted-by: Jenkins Submitted-at: Tue, 17 Nov 2015 05:29:34 +0000 Reviewed-on: https://review.openstack.org/245705 Project: openstack/neutron Branch: refs/heads/master
2015-11-13Merge "Move dhcp_lease_duration into DHCP agent config options list"Jenkins
2015-11-13Merge "Replace subnetpool config options with admin-only API"Jenkins
2015-11-13Move dhcp_lease_duration into DHCP agent config options listIhar Hrachyshka
It is DHCP agent specific, and so it should not be present in neutron.conf or be reported as a valid option for other services. Also modified the help message slightly to avoid mentioning dnsmasq since the option could in theory be used by alternative dhcp drivers. Change-Id: I66bcea1c3a6ce6fa9ce56ea8137a94858f18dc5a Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Henry Gessau <gessau@cisco.com> Code-Review+2: Cedric Brandily <zzelle@gmail.com> Workflow+1: Cedric Brandily <zzelle@gmail.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Code-Review+1: Ryan Moats <rmoats@us.ibm.com> Code-Review+1: Sergey Belous <sbelous@mirantis.com> Submitted-by: Jenkins Submitted-at: Fri, 13 Nov 2015 22:31:42 +0000 Reviewed-on: https://review.openstack.org/245097 Project: openstack/neutron Branch: refs/heads/master
2015-11-11Merge "Add new config option for IPv6 Prefix Delegation"Jenkins
2015-11-09Merge "Add track_quota_usage conf into neutron.conf"Jenkins
2015-11-06Replace subnetpool config options with admin-only APIJohn Davidge
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 <carl@ecbaldwin.net> Change-Id: I5daba2347cfb91fac0b155b2c1b459ee7d9e4505 Closes-Bug: 1501328 Notes (review): Verified+2: Jenkins Code-Review+1: Ryan Tidwell <ryan.tidwell@hpe.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Henry Gessau <gessau@cisco.com> Code-Review+1: Bradley Jones <jones.bradley@me.com> Code-Review+1: Baodong (Robert) Li <baoli@cisco.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Workflow+1: Carl Baldwin <carl@ecbaldwin.net> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Code-Review+1: James Reeves <james.reeves5546@gmail.com> Code-Review+1: Sergey Belous <sbelous@mirantis.com> Code-Review+1: Brandon Palm <bapalm@us.ibm.com> Submitted-by: Jenkins Submitted-at: Fri, 13 Nov 2015 16:08:38 +0000 Reviewed-on: https://review.openstack.org/230983 Project: openstack/neutron Branch: refs/heads/master
2015-11-06Add new config option for IPv6 Prefix DelegationJohn Davidge
With the impending deprecation of the default subnetpool configuration options, IPv6 PD needs to be enabled with a new config option. This patch adds the 'ipv6_pd_enabled' option to neutron.conf, and makes all of the necessary changes for its use. DocImpact Change-Id: I43486c5a13ee2ff0097355afe7e1f3ef8794b185 Closes-Bug: 1501835 Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Workflow+1: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Bradley Jones <jones.bradley@me.com> Code-Review+1: Baodong (Robert) Li <baoli@cisco.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Code-Review+1: Yan Haifeng <yanheven@qq.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Code-Review+1: Nate Johnston <nate_johnston@cable.comcast.com> Code-Review+1: James Reeves <james.reeves5546@gmail.com> Code-Review+1: Sergey Belous <sbelous@mirantis.com> Submitted-by: Jenkins Submitted-at: Wed, 11 Nov 2015 08:42:32 +0000 Reviewed-on: https://review.openstack.org/230090 Project: openstack/neutron Branch: refs/heads/master
2015-11-01Merge "Remove deprecated nova_* options"Jenkins
2015-11-01Remove deprecated nova_* optionsCedric Brandily
Neutron nova_* options have been deprecated in Kilo in favour of options in [nova] section. This change removes these options in order to simplify code, clarify configuration and avoid a warning in unittests. DocImpact UpgradeImpact Closes-Bug: #1509092 Related-Bug: #1403686 Change-Id: I950f72ad7d7d41cfa006f8c7105934301cd56faa Notes (review): Verified+2: Jenkins Code-Review+2: Akihiro Motoki <amotoki@gmail.com> Code-Review+2: garyk <gkotton@vmware.com> Workflow+1: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Submitted-by: Jenkins Submitted-at: Sun, 01 Nov 2015 09:43:32 +0000 Reviewed-on: https://review.openstack.org/238696 Project: openstack/neutron Branch: refs/heads/master
2015-10-21Spawn dedicated rpc workers for state reports queueEugene Nikanorov
By default spawn one additional rpc worker to process state report queue. State report queue will also be processed by regular rpc workers, but in case these workers are busy with processing heavy requests, state reports queue will automatically be consumed by dedicated rpc workers. This change applies to ML2 plugin only. Other plugins should implement start_rpc_state_reports_listener to enable additional rpc workers. Change-Id: I5f8df6a478f7c82382049274b34b07109eeafbdb Closes-Bug: #1505217 Notes (review): Verified+2: Jenkins Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Oleg Bondarev <obondarev@mirantis.com> Workflow+1: Oleg Bondarev <obondarev@mirantis.com> Code-Review+2: YAMAMOTO Takashi <yamamoto@midokura.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Submitted-by: Jenkins Submitted-at: Thu, 22 Oct 2015 07:45:04 +0000 Reviewed-on: https://review.openstack.org/233605 Project: openstack/neutron Branch: refs/heads/master
2015-10-09Add track_quota_usage conf into neutron.confgong yong sheng
track_quota_usage is missing in neutron configuration files. This patch adds it into neutron.conf so that deployers know it. Closes-bug: #1504370 DocImpact Change-Id: Ibaff5ac60cd69958eb981d2058d946cc88f799f1 Notes (review): Verified+2: Jenkins Code-Review+2: Russell Bryant <rbryant@redhat.com> Code-Review+2: garyk <gkotton@vmware.com> Workflow+1: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+1: Li Ma <skywalker.nick@gmail.com> Code-Review+1: Richard Theis <rtheis@us.ibm.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Code-Review+1: Ryan Moats <rmoats@us.ibm.com> Code-Review+1: Brad Behle <behle@us.ibm.com> Code-Review+1: Ramanjaneya Reddy Palleti <ramanjaneya.palleti@huawei.com> Code-Review+1: melissaml <ma.lei@99cloud.net> Code-Review+1: Brandon Palm <bapalm@us.ibm.com> Submitted-by: Jenkins Submitted-at: Mon, 09 Nov 2015 21:42:06 +0000 Reviewed-on: https://review.openstack.org/232448 Project: openstack/neutron Branch: refs/heads/master
2015-10-08Merge "Consume sslutils and wsgi modules from oslo.service"Jenkins
2015-10-08Consume sslutils and wsgi modules from oslo.serviceElena Ezhova
sslutils and basic WSGI functionality have been moved to oslo.service and now Neutron can reuse them. Marked ssl options that were renamed in oslo.service as deprecated. Added a note about possible implications for out-of-tree plugins to neutron_api.rst Bumped oslo.service version to 0.9.0. Related-Bug: #1482633 Depends-On: I0424a6c261fae447dbc25b3abf00258c860a88f5 Change-Id: Ibfdf07e665fcfcd093a0e31274e1a6116706aec2 Notes (review): Verified+2: Jenkins Code-Review+2: Armando Migliaccio <armamig@gmail.com> Workflow+1: Armando Migliaccio <armamig@gmail.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Submitted-by: Jenkins Submitted-at: Thu, 08 Oct 2015 22:20:42 +0000 Reviewed-on: https://review.openstack.org/227835 Project: openstack/neutron Branch: refs/heads/master
2015-10-05Deprecate max_fixed_ips_per_portKevin Benton
This option does not have a clear use case since we prevent users from setting their own IP addresses on shared networks. DocImpact Change-Id: I211e87790c955ba5c3904ac27b177acb2847539d Closes-Bug: #1502356 Notes (review): Verified+2: Jenkins Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+2: Cedric Brandily <zzelle@gmail.com> Workflow+1: Cedric Brandily <zzelle@gmail.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Code-Review+1: vikram.choudhary <vikram.choudhary@huawei.com> Code-Review+1: Brandon Palm <bapalm@us.ibm.com> Submitted-by: Jenkins Submitted-at: Tue, 06 Oct 2015 00:27:31 +0000 Reviewed-on: https://review.openstack.org/230696 Project: openstack/neutron Branch: refs/heads/master
2015-10-02Add availability_zone support baseHirofumi Ichihara
This patch adds the availability_zone attribute to agents and supports availability_zone API. Availability_zone support for resources (network/router) and the schedulers are included in subsequent patches. APIImpact DocImpact Co-Authored-By: IWAMOTO Toshihiro <iwamoto@valinux.co.jp> Change-Id: Id7a62000ab0484412b3970199df8c374568fe70d Partially-implements: blueprint add-availability-zone Notes (review): Verified+2: Jenkins Code-Review+2: Armando Migliaccio <armamig@gmail.com> Code-Review+2: Akihiro Motoki <amotoki@gmail.com> Workflow+1: Akihiro Motoki <amotoki@gmail.com> Code-Review+1: Ann Kamyshnikova <akamyshnikova@mirantis.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Code-Review+1: vikram.choudhary <vikram.choudhary@huawei.com> Submitted-by: Jenkins Submitted-at: Fri, 02 Oct 2015 08:59:24 +0000 Reviewed-on: https://review.openstack.org/183369 Project: openstack/neutron Branch: refs/heads/master
2015-09-12Switch scheduler drivers to load based schedulersarmando-migliaccio
Cloud deployed at scale most likely will use these scheduler drivers because they allow a fairer resource allocation compared to chance schedulers (which randomly place resources on the hosts). Because of their importance, it's only wise to test them in the gate on a continuous basis, so that we do not get surprised by accidental regressions. Rather than pushing this down through devstack-gate/project-config patches, this chance alters the default of the scheduler drivers, so that users can also pick these up out of the box. This means that after an upgrade they would observe a change in the scheduling behavior, if they relied on the default config. DocImpact UpgradeImpact Closes-bug: #1494667 Change-Id: I5927914cb88eff66bc7a045340ff68cb8da95ad6 Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Code-Review+2: mark mcclain <mark@mcclain.xyz> Workflow+1: mark mcclain <mark@mcclain.xyz> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Gal Sagie <gal.sagie@huawei.com> Submitted-by: Jenkins Submitted-at: Mon, 14 Sep 2015 00:20:02 +0000 Reviewed-on: https://review.openstack.org/222547 Project: openstack/neutron Branch: refs/heads/master
2015-09-04Merge "Add support for PluginWorker and Process creation notification"Jenkins
2015-09-03Add support for PluginWorker and Process creation notificationTerry Wilson
There are several cases where plugin initialization should be handled after neutron-server forks API/RPC workers. For example, starting a client connection to an SDN controller before forking copies the fd of the socket to the child process, but then you have multiple processes trying to read/write the same socket connection. It is also useful for a plugin to be able to do something in only one process, regardless of how many workers are forked. One example would be handling syncing from an external system to the neutron database. This patch does 3 things: 1) Treats rpc_workers=0 as = 1. This simplifies the code for handling notification that forking has completed. In the existing code, calling the notification in the Worker object's start() method would happen twice in the case where both api and rpc workers were 0, despite there being only one process. An earlier patch already changed the default api_workers to be the number of processors. 2) Adds notification of forking via the callbacks mechanism. Plugins can subscribe to resources.PROCESS, event.AFTER_CREATE and do any post-fork initialization that needs to be done for every spawned process. 3) Adds core/service plugin calls to get_workers() which defaults to returning (). Plugins that need additional processes to spawn should just return an iterable of NeutronWorkers that will be spawned in their own process. DocImpact Closes-Bug: #1463129 Change-Id: Ib99954678c2b4f32f486b537979d446aafbea07b Notes (review): Verified+2: Jenkins Code-Review+2: Armando Migliaccio <armamig@gmail.com> Workflow+1: Armando Migliaccio <armamig@gmail.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+2: Kevin Benton <kevinbenton@buttewifi.com> Code-Review+2: Miguel Angel Ajo <mangelajo@redhat.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Gal Sagie <gal.sagie@huawei.com> Submitted-by: Jenkins Submitted-at: Fri, 04 Sep 2015 05:02:52 +0000 Reviewed-on: https://review.openstack.org/189391 Project: openstack/neutron Branch: refs/heads/master
2015-08-30Merge "Add enable_new_agents to neutron server"Jenkins
2015-08-28Add enable_new_agents to neutron serverHirofumi Ichihara
Neutron doesn't have a way to test a newly added network node by deploying test resource before any customer resource on the node is deployed. Nova and Cinder has the setting of “enable_new_services” in each conf to disable the initial service status to achieve this. This proposal adds enable_new_agents config. DocImpact Change-Id: Ie0d0b2dd4d95de95f3839d1c35f24b708e893801 Implements: blueprint enable-new-agents Related-Bug: 1472076 Notes (review): Verified+2: Jenkins Code-Review+2: Armando Migliaccio <armamig@gmail.com> Workflow+1: Armando Migliaccio <armamig@gmail.com> Code-Review+2: Akihiro Motoki <amotoki@gmail.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: YAMAMOTO Takashi <yamamoto@midokura.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Gal Sagie <gal.sagie@huawei.com> Code-Review+1: vikram.choudhary <vikram.choudhary@huawei.com> Submitted-by: Jenkins Submitted-at: Sun, 30 Aug 2015 05:08:14 +0000 Reviewed-on: https://review.openstack.org/216980 Project: openstack/neutron Branch: refs/heads/master
2015-08-24Add dns_label processing for PortsMiguel Lavalle
Functionallity is added to enable users to specify a dns_label field during port creation and update. This dns_label field will be used for DNS resolution of the hostname in dnsmasq and also will be used when Neutron can integrate with external DNS systems. Change-Id: I6beab336dfd9b70b1af6e975939c602047faa651 DocImpact APIImpact Closes-Bug: #1459030 Implements: blueprint internal-dns-resolution Notes (review): Verified+2: Jenkins Code-Review+1: Armando Migliaccio <armamig@gmail.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Code-Review+2: Kevin Benton <kevinbenton@buttewifi.com> Workflow+1: Kevin Benton <kevinbenton@buttewifi.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Submitted-by: Jenkins Submitted-at: Wed, 26 Aug 2015 19:53:55 +0000 Reviewed-on: https://review.openstack.org/200952 Project: openstack/neutron Branch: refs/heads/master
2015-08-17Merge remote-tracking branch 'origin/feature/qos' into merge-branchIhar Hrachyshka
Note to reviewers: gerrit diff for merge patches is very limited, and leaving comments in global section won't scale, so please comment here: https://etherpad.openstack.org/p/qos-merge-back-review This merge commit introduces QoS feature into Liberty release of Neutron. The feature is documented in: doc/source/devref/quality_of_service.rst included with the merge patch. It includes: - QoS API service plugin with QoS policy and QoS bandwidth limit (egress) rule support; - core plugin mechanism to determine supported rule types, with its ML2 implementation; - new agent extension manager; - QoS agent extension with pluggable backend QoS drivers (Open vSwitch and SR-IOV support is included). To extend network and port core resources with qos_policy_id attribute, a new ML2 extension driver (qos) was introduced that relies on the QoS core resource extension (the idea is that eventually we'll get a core resource extension manager that can be directly reused by core plugins). Agent-server interaction is based on: - get_device_details() method that is extended with qos_policy_id; - a new push/pull mechanism that allows agents and servers to communicate using oslo.versionedobjects based objects sent on the wire. The merge includes the following types of test coverage: - unit tests; - functional tests for OVS agent, QoS agent extension, and low level ovs_lib changes; - API tests to cover port/network qos_policy_id attribute and new QoS resources. The client changes can be found at: * https://review.openstack.org/189655 * https://review.openstack.org/198277 The team also prepared fullstack test but it needs to wait for client merge before it can pass in the gate: * https://review.openstack.org/202492 Gerrit does not show diff for merge changes that did not result in any conflict, so to facilitate review, rely on the following steps: - fetch the patch locally - git fetch origin - git diff origin/master... This merge also disables qos extension API tests until the service is enabled in master gate. Local changes apart from conflicts: - updated down_revision for qos migration to reflect master expand head; - disabled qos API tests with gate_hook.sh until we have it enabled in master gate; - bumped oslo.versionedobjects requirement to reflect what is in openstack/requirements' global-requirements.txt DocImpact APIImpact Partially-Implements: blueprint quantum-qos-api Partially-Implements: blueprint ml2-qos Partially-Implements: blueprint ml2-qos-ovs-bwlimiting Partially-Implements: blueprint ml2-sriov-qos-with-bwlimiting Change-Id: I92916d0e391791187e9a25ff172fb4b3504857b1 Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Workflow+1: Kyle Mestery <mestery@mestery.com> Code-Review+1: Jakub Libosvar <libosvar@redhat.com> Code-Review+2: Assaf Muller <amuller@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Gal Sagie <gal.sagie@huawei.com> Code-Review+1: Moshe Levi <moshele@mellanox.com> Code-Review+1: vikram.choudhary <vikram.choudhary@huawei.com> Submitted-by: Jenkins Submitted-at: Mon, 17 Aug 2015 18:49:56 +0000 Reviewed-on: https://review.openstack.org/212170 Project: openstack/neutron Branch: refs/heads/master
2015-08-13Add configurable options for HA networksJohn Schwarz
The L3 HA mechanism creates a project network for HA (VRRP) traffic among routers. The HA project network uses the first (default) network type in 'tenant_network_types'. Depending on the environment, this combination may not provide a desirable path for HA traffic. For example, some operators may prefer to use a specific network for HA traffic to prevent split-brain issues. This patch adds configurable options that target the network_type and the physical_network of the created HA network. Doc-Impact Closes-Bug: #1481443 Change-Id: I3527a780179b5982d6e0eb0b8c32d6dafeeab730 Notes (review): Verified+2: Jenkins Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Workflow+1: Carl Baldwin <carl@ecbaldwin.net> Code-Review+2: Kevin Benton <kevinbenton@buttewifi.com> Code-Review+2: Assaf Muller <amuller@redhat.com> Code-Review+1: Matthew Kassawara <mkassawara@gmail.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Ray Schneider <rmschn@gmail.com> Submitted-by: Jenkins Submitted-at: Fri, 14 Aug 2015 02:03:45 +0000 Reviewed-on: https://review.openstack.org/212017 Project: openstack/neutron Branch: refs/heads/master
2015-08-05Merge remote-tracking branch 'origin/feature/qos' into merge-branchIhar Hrachyshka
Change-Id: I683102e617202e0ffc953a0d3cc179879f8faf82 Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Code-Review+2: Edgar Magana <emagana@gmail.com> Code-Review+2: mark mcclain <mark@mcclain.xyz> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Henry Gessau <gessau@cisco.com> Workflow+1: Ihar Hrachyshka <ihrachys@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Code-Review+1: John Schwarz <jschwarz@redhat.com> Submitted-by: Jenkins Submitted-at: Thu, 06 Aug 2015 00:27:23 +0000 Reviewed-on: https://review.openstack.org/209556 Project: openstack/neutron Branch: refs/heads/feature/qos
2015-08-01Added missing [qos] section into neutron.confIhar Hrachyshka
Also renamed service_notification_drivers into notification_drivers since it's clear where it belongs anyway (it's in neutron.conf meaning it's a server side configuration value). Change-Id: I64610e4b60112daec982a8cacded9b9b936c10bd Partially-Implements: blueprint quantum-qos-api Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Miguel Angel Ajo <mangelajo@redhat.com> Workflow+1: Miguel Angel Ajo <mangelajo@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Code-Review+1: Eran Gampel <eran@gampel.net> Code-Review+1: vikram.choudhary <vikram.choudhary@huawei.com> Submitted-by: Jenkins Submitted-at: Sun, 02 Aug 2015 15:57:58 +0000 Reviewed-on: https://review.openstack.org/207005 Project: openstack/neutron Branch: refs/heads/feature/qos
2015-08-01Remove unnecessary executable permissionQiaowei Ren
Bunch of neutron source code files are marked as executable which is not appropriate, this patch just 'chmod -x' to all of them. Closes-Bug: #1468564 Change-Id: Idec4ae16501d68e044e103db24ad3be0e0751e9a Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Code-Review+2: YAMAMOTO Takashi <yamamoto@midokura.com> Workflow+1: YAMAMOTO Takashi <yamamoto@midokura.com> Code-Review+1: Takashi NATSUME <natsume.takashi@lab.ntt.co.jp> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+1: Cedric Brandily <zzelle@gmail.com> Code-Review+1: shihanzhang <shihanzhang@huawei.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Ryan Moats <rmoats@us.ibm.com> Code-Review+1: vikram.choudhary <vikram.choudhary@huawei.com> Code-Review+1: Bharath M <bharath.stacker@gmail.com> Submitted-by: Jenkins Submitted-at: Mon, 03 Aug 2015 04:52:42 +0000 Reviewed-on: https://review.openstack.org/202416 Project: openstack/neutron Branch: refs/heads/master
2015-07-28Create packages for quota modulesSalvatore Orlando
This patch simply renames quota modules introducing two new packages: 1) neutron.quota 2) neutron.db.quota This paves the way for adding more quota related modules. Unit tests paths are changed accordingly, as well as references to moved modules. This patch hovewer preserves the ability of importing neutron.db.quota_db in order to avoid breaking repositories which use neutron as a requirement. Change-Id: I946e87f1b2ebb3e80d64aecfe58aeff9f81896f3 Related-blueprint: better-quotas Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Code-Review+2: Armando Migliaccio <armamig@gmail.com> Workflow+1: Armando Migliaccio <armamig@gmail.com> Code-Review+2: Aaron Rosen <aaronorosen@gmail.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Code-Review+1: John Kasperski <jckasper@linux.vnet.ibm.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Ryan Moats <rmoats@us.ibm.com> Submitted-by: Jenkins Submitted-at: Wed, 29 Jul 2015 22:21:29 +0000 Reviewed-on: https://review.openstack.org/188618 Project: openstack/neutron Branch: refs/heads/master
2015-06-29QoS service plugin stubMiguel Angel Ajo
This patch introduces the QoS service plugin which implements a stub of the API extension. This is patch is a basic step to be able to create an experimental job enabling this service so we can do api tests. Change-Id: Ib583e98c232ca628ba2a4bd48527eb84584c6212 Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Ihar Hrachyshka <ihrachys@redhat.com> Workflow+1: Ihar Hrachyshka <ihrachys@redhat.com> Submitted-by: Jenkins Submitted-at: Mon, 29 Jun 2015 18:25:21 +0000 Reviewed-on: https://review.openstack.org/189628 Project: openstack/neutron Branch: refs/heads/feature/qos
2015-06-12Revert "Revert "Set default of api_workers to number of CPUs""armando-migliaccio
This reverts commit 12a564cf03e612dda36df26df8d28dfc75f1af6e. We should re-enable this feature on a controlled basis so that we can flush out any outstanding issue we may have. Related-bug: #1432189 Change-Id: I2cfd93fdb032b461022b729347390ff8636ccdeb Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Workflow+1: Kyle Mestery <mestery@mestery.com> Code-Review+1: Brian Haley <brian.haley@hp.com> Code-Review+2: garyk <gkotton@vmware.com> Code-Review+1: Sean M. Collins <sean@coreitpro.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Clint 'SpamapS' Byrum <clint@fewbar.com> Code-Review+1: YAMAMOTO Takashi <yamamoto@midokura.com> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Code-Review+1: Cedric Brandily <zzelle@gmail.com> Code-Review+1: Miguel Angel Ajo <mangelajo@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Submitted-by: Jenkins Submitted-at: Tue, 16 Jun 2015 20:44:43 +0000 Reviewed-on: https://review.openstack.org/191127 Project: openstack/neutron Branch: refs/heads/master
2015-06-09Switch from MySQL-python to PyMySQLJeremy Stanley
As discussed in the Liberty Design Summit "Moving apps to Python 3" cross-project workshop, the way forward in the near future is to switch to the pure-python PyMySQL library as a default. https://etherpad.openstack.org/p/liberty-cross-project-python3 Change-Id: I73e0fdb6eca70e7d029a40a2f6f17a7c0797a21d Notes (review): Verified+2: Jenkins Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Kevin Benton <kevinbenton@buttewifi.com> Verified-1: VMware NSX CI Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+2: Doug Wiegley <dougwig@parkside.io> Workflow+1: Doug Wiegley <dougwig@parkside.io> Code-Review+1: Angus Lees <gus@inodes.org> Submitted-by: Jenkins Submitted-at: Wed, 10 Jun 2015 04:15:09 +0000 Reviewed-on: https://review.openstack.org/184383 Project: openstack/neutron Branch: refs/heads/master
2015-06-01Implement IPAM Driver loaderPavel Bondar
IPAM Driver is loaded based on value of 'ipam_driver'. Added new variable 'ipam_driver' in config. DocImpact Partially-Implements: blueprint neutron-ipam Change-Id: Ia52ad70ef4f0b02cf82cfefcf50b9f1e30b05b79 Notes (review): Verified+2: Jenkins Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Carl Baldwin <carl@ecbaldwin.net> Workflow+1: Carl Baldwin <carl@ecbaldwin.net> Code-Review+1: Trinath Somanchi <trinath.somanchi@freescale.com> Verified+1: VMware NSX CI Code-Review+2: Ihar Hrachyshka <ihrachys@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: John Belamaric <jbelamaric@infoblox.com> Submitted-by: Jenkins Submitted-at: Mon, 01 Jun 2015 23:29:35 +0000 Reviewed-on: https://review.openstack.org/147479 Project: openstack/neutron Branch: refs/heads/master
2015-05-11Deprecate quota_items, register resources upon REST initializationSalvatore Orlando
Register 'core' resources when the respective rest controllers are instantiated, rather than at module load time. Since in this way there will not be any need to iterate over quota_items, the option is being deprecated. This patch does not supply unit tests as the already-existing routine for registering a resource from quota_items is being deprecated as well (and was not covered by any unit test beforehand). DocImpact Change-Id: Icdb744adfd86d38363239a454ccf04f3c6b9c158 Closes-Bug: #1453322 Notes (review): Verified+2: Jenkins Code-Review+2: Armando Migliaccio <armamig@gmail.com> Workflow+1: Armando Migliaccio <armamig@gmail.com> Code-Review+2: Aaron Rosen <aaronorosen@gmail.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Verified+1: VMware NSX CI Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: NEC CI <nec-openstack-ci@iaas.jp.nec.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Submitted-by: Jenkins Submitted-at: Thu, 14 May 2015 20:56:54 +0000 Reviewed-on: https://review.openstack.org/181593 Project: openstack/neutron Branch: refs/heads/master
2015-05-07Allow to define enable_snat default valueCedric Brandily
Currently neutron resets enable_snat attribute to True when enable_snat is not provided in router external_gateway_info. But in some deployments (private/enterprise clouds) such behavior is not the expected default one as snat/nat/floating-ips is not used (at least by default). This change defines the option enable_snat_by_default which allows deployers to set enable_snat default value when neutron resets it. The option default value is True for backward compatibility. DocImpact APIImpact Closes-Bug: #1388858 Change-Id: I455a552230ec89fe907a087c1de8c8144b5d086e Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Workflow+1: garyk <gkotton@vmware.com> Code-Review+2: Maru Newby <marun@redhat.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: Paul Michali <pc@michali.net> Code-Review+1: Brandon Logan <brandon.logan@rackspace.com> Code-Review+1: Ann Kamyshnikova <akamyshnikova@mirantis.com> Code-Review+1: Hirofumi Ichihara <ichihara.hirofumi@lab.ntt.co.jp> Verified+1: VMware NSX CI Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Code-Review+1: Liping Mao <limao@cisco.com> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: NEC CI <nec-openstack-ci@iaas.jp.nec.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Code-Review+1: Gal Sagie <gal.sagie@huawei.com> Submitted-by: Jenkins Submitted-at: Thu, 07 May 2015 13:53:47 +0000 Reviewed-on: https://review.openstack.org/130588 Project: openstack/neutron Branch: refs/heads/master
2015-04-16Merge "Revert "Add ipset element and hashsize tunables""Jenkins
2015-04-14Provide details for configure multiple DHCP agentsLi Ma
The help text is not that good for operation. This fix adds more information about the option 'dhcp_agents_per_network'. Change-Id: I955c1e9989a9c65b0ffdbbdca9113c795ec72fe6 Closes-Bug: #1370934 Notes (review): Verified+2: Jenkins Code-Review+1: Brian Haley <brian.haley@hp.com> Code-Review+2: mark mcclain <mark@mcclain.xyz> Workflow+1: mark mcclain <mark@mcclain.xyz> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+2: Oleg Bondarev <obondarev@mirantis.com> Code-Review+1: Ann Kamyshnikova <akamyshnikova@mirantis.com> Code-Review+1: shihanzhang <shihanzhang@huawei.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: NEC CI <nec-openstack-ci@iaas.jp.nec.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Verified+1: A10 Networks CI <a10-openstack-ci@a10networks.com> Submitted-by: Jenkins Submitted-at: Thu, 16 Apr 2015 13:13:39 +0000 Reviewed-on: https://review.openstack.org/172806 Project: openstack/neutron Branch: refs/heads/master
2015-04-08Add missing config parameters in neutron.confEdgar Magana
Include all missing configuration parameters already integrated in Neutron code. Change-Id: Iefa344a2f9ec2c74f6314e7c783ff3b213d76ea3 Closes-bug: #1438329 Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Verified+1: Microsoft Hyper-V CI <hyper-v_ci@microsoft.com> Code-Review+1: pritesh <pritesh.kothari@cisco.com> Code-Review+2: Ihar Hrachyshka <ihrachys@redhat.com> Workflow+1: Ihar Hrachyshka <ihrachys@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: NEC CI <nec-openstack-ci@iaas.jp.nec.com> Verified-1: A10 Networks CI <a10-openstack-ci@a10networks.com> Submitted-by: Jenkins Submitted-at: Wed, 08 Apr 2015 22:42:45 +0000 Reviewed-on: https://review.openstack.org/171059 Project: openstack/neutron Branch: refs/heads/master
2015-04-08Merge "Add ipset element and hashsize tunables"Jenkins
2015-04-07Merge "Stop using deprecated DEFAULT group for lock_path"Jenkins
2015-04-06Add ipset element and hashsize tunablesBrian Haley
Recently, these messages have been noticed in both tempest logs, as well as reported by downstream users syslog: Set IPv4915d358d-2c5b-43b5-9862 is full, maxelem 65536 reached So the default of 64K is not sufficient enough. This change adds two config options to control both the number of elements as well as the hashsize, since they should be tuned together for best performance. Slightly different formats were required for 'ipset create' and 'ipset restore'. The default values for these are now set to 131072 (maxelem) and 2048 (hashsize), which is an increase over their typical default values of 65536/1024 (respectively), in order to fix the errors seen in the tempest tests. DocImpact Change-Id: Ic0b5b38a840e737dc6be938230f4052974c8620f Closes-bug: #1439817 Notes (review): Verified+2: Jenkins Code-Review+2: Kyle Mestery <mestery@mestery.com> Code-Review+1: gongysh <gong.yongsheng@99cloud.net> Code-Review+2: Henry Gessau <gessau@cisco.com> Workflow+1: Henry Gessau <gessau@cisco.com> Code-Review+1: Carl Baldwin <carl@ecbaldwin.net> Code-Review+1: Miguel Angel Ajo <mangelajo@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: NEC CI <nec-openstack-ci@iaas.jp.nec.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Submitted-by: Jenkins Submitted-at: Wed, 08 Apr 2015 15:57:10 +0000 Reviewed-on: https://review.openstack.org/170328 Project: openstack/neutron Branch: refs/heads/master
2015-04-02Merge "Allow metadata proxy to log with nobody user/group"Jenkins
2015-04-01Allow metadata proxy to log with nobody user/groupCedric Brandily
Currently metadata proxy cannot run with nobody user/group as metadata proxy (as other services) uses WatchedFileHandler handler to log to file which does not support permissions drop (the process must be able to r/w after permissions drop to "watch" the file). This change allows to enable/disable log watch in metadata proxies with the new option metadata_proxy_log_watch. It should be disabled when metadata_proxy_user/group is not allowed to read/write metadata proxy log files. Option default value is deduced from metadata_proxy_user: * True if metadata_proxy_user is agent effective user id/name, * False otherwise. When log watch is disabled and logrotate is enabled on metadata proxy logging files, 'copytruncate' logrotate option must be used otherwise metadata proxy logs will be lost after the first log rotation. DocImpact Change-Id: I40a7bd82a2c60d9198312fdb52e3010c60db3511 Partial-Bug: #1427228 Notes (review): Verified+2: Jenkins Code-Review+2: garyk <gkotton@vmware.com> Workflow+1: garyk <gkotton@vmware.com> Verified+1: VMware NSX CI Code-Review+2: Ihar Hrachyshka <ihrachys@redhat.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Verified+1: A10 Networks CI <a10-openstack-ci@a10networks.com> Submitted-by: Jenkins Submitted-at: Thu, 02 Apr 2015 11:39:27 +0000 Reviewed-on: https://review.openstack.org/161494 Project: openstack/neutron Branch: refs/heads/master
2015-04-01Implement default subnet pool configuration settingsCarl Baldwin
The default_ipv6_subnet_pool option was added [1] as an integration point between prefix delegation work and subnet allocation work. This patch completes the integration with subnet allocation. This addresses the use case where a deployer wants all ipv6 addresses to come -- by default -- from a globally routable pool of ipv6 addresses. In a deployment with this option set, an API user can still access the old behavior by passing None explicitly as subnetpool_id when creating a subnet. This patch also adds the default_ipv4_subnet_pool for completeness. [1] https://review.openstack.org/#/c/166973 Change-Id: I301189b5cd31d7c5fa4a40fa3e04f8e6ac77592b Partially-Implements: blueprint subnet-allocation Notes (review): Verified+2: Jenkins Code-Review+2: Salvatore Orlando <salv.orlando@gmail.com> Workflow+1: Salvatore Orlando <salv.orlando@gmail.com> Code-Review+1: Brian Haley <brian.haley@hp.com> Code-Review+1: John Davidge <jodavidg@cisco.com> Code-Review+2: Kevin Benton <kevinbenton@buttewifi.com> Verified+1: Ryu CI <ryu-openstack-review@lists.sourceforge.net> Verified+1: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il> Verified+1: Big Switch CI <openstack-ci@bigswitch.com> Verified+1: Arista CI <arista-openstack-test@aristanetworks.com> Verified+1: Metaplugin CI <metaplugintest@gmail.com> Verified+1: Nuage CI <nuage-ci@nuagenetworks.net> Verified+1: A10 Networks CI <a10-openstack-ci@a10networks.com> Submitted-by: Jenkins Submitted-at: Wed, 01 Apr 2015 21:54:03 +0000 Reviewed-on: https://review.openstack.org/168098 Project: openstack/neutron Branch: refs/heads/master