Merge "Move LinuxBridge related features to bridge_lib"

Merge "Reduce duplicated code in test_linuxbridge_neutron_agent"

Merge "Trivial fix in l3 agent"

Merge "IPAM: make max fixed IP validations DRY"

Merge "Add track_quota_usage conf into neutron.conf"

Move LinuxBridge related features to bridge_lib

This change moves from linuxbridge agent[1] to bridge_lib[2] bridge only
related features and adds functional tests.

[1] neutron.plugins.ml2.drivers.linuxbridge.agent.linuxbridge_neutron_agent
[2] neutron.agent.linux.bridge_lib

Partial-Bug: #1514548
Change-Id: Ieccb8f77fc833467a166557e227023f5c2a50d1b

Merge "OVS agent reacts to events instead of polling"

Merge "Document the neutron-release team."

Reduce duplicated code in test_linuxbridge_neutron_agent

This change defines and uses get_linuxbridge_manager method which
mocks some methods in order to instantiate and returns a
LinuxBridgeManager instance, this method allows to remove some
duplicated code from test_linuxbridge_neutron_agent.

Partial-Bug: #1514548
Change-Id: Ib54a01406938b4937af2af5fca7a71b5758d4482

Document the neutron-release team.

Gerrit includes a neutron-release team.  Document what this team means
today.  Also propose that this team have merge rights to *all* Neutron
git repos so that a small group has the ability to quickly unblock
things if needed, which is especially important around release time.

Change-Id: Id3d4d051278930841f64a4153e40a2dd3d893e44
Signed-off-by: Russell Bryant <rbryant@redhat.com>

Updated from global requirements

Change-Id: I2e0b99ee3849a6e0d7206ec63926d3875668cf59

Merge "Add in missing spaces at end of line"

Merge "Fix misuse of log marker functions in neutron"

Merge "More instructions for neutron-db-manage revision --autogenerate"

Trivial fix in l3 agent

Remove unnecessary blank line and keep consistence about
blank space in l3 config.

Change-Id: Ia69393db85aacabad266f056995c15f043f32bfb

IPAM: make max fixed IP validations DRY

Reuse code to make the validations DRY. In addition to this
fixed a typo.

Change-Id: I69cc904d30aae0e2d189a72c9dddaaab63a923ca

Merge "Make command log in neutron utils.execute() a single line"

Fix misuse of log marker functions in neutron

some log marker functions misuse in neutron,
ref http://docs.openstack.org/developer/oslo.i18n/guidelines.html
and http://docs.openstack.org/developer/oslo.i18n/usage.html

Change-Id: Ia7a539f9414c4aa9666fa62522083ef48950e093
Closes-Bug: #1514193

More instructions for neutron-db-manage revision --autogenerate

More detailed instructions for --autogenerate, including how to rely
less on devstack. Also some tweaks to scripts and requirements to
simplify the process.

Closes-Bug: #1197570

Change-Id: I97043b30a8df7408a73af38c6a015fc0dcf82571

Merge "Do not use log hints for exceptions"

Add in missing spaces at end of line

Trivial updates to log messages that do not have spaces at the
end of a line.

TrivialFix

Change-Id: I126e7e3535c6b8ecaeffe4210fdad6e6f238a37c

Merge "Adding a function prefix before parenthesis"

Do not use log hints for exceptions

A log hint should not be used for an exception.

Please see http://docs.openstack.org/developer/oslo.i18n/guidelines.html

Change-Id: I122455fd767c21e26c34e23b8b7d45b845f15e99

Merge "Make string representation of DictModel generic"

Adding a function prefix before parenthesis

Add a missing translation _ before parenthesis as it is done in defining
all exception messages.

Change-Id: Id80b4bf083d357d6c99006116e453d8b63fbc876

Make command log in neutron utils.execute() a single line

command execution log from neutron.agent.linux.utils.execute()
consists of multiple lines, and the first line has less information
and it is hard to analyze logs.

Closes-Bug: #1460379
Change-Id: Idca38f26321394596245c9c59b8a9a78c7190423

Merge "Move update_fip_statuses to Router class"

Merge "Make '*' the default ml2 flat_networks configuration"

Merge "Correction and clarification to subproject stable guidelines"

Move update_fip_statuses to Router class

I noticed that ri.floating_ips was being accessed from the agent.  It
didn't seem right.  Also, noticed that this set gets passed around too
much between methods.  It all boils down to saving the previous set
and then doing set difference to see which ones went away.  This patch
set makes it simpler.

Partially-Implements: blueprint address-scopes

Change-Id: I73424620e196200c70f221871c34e33ccfe73261

Merge "Add PyPI link for networking-calico"

Merge "Deprecate new= argument from create_connection"

Correction and clarification to subproject stable guidelines

Change-Id: I5511430593b39f7d5a952270aeb43f973090d702

Make '*' the default ml2 flat_networks configuration

Update the ml2 flat_networks configuration to set '*' as the
default because that has been effectively the default behavior due
to a bug with the empty list.

Also, fix the empty list configuration for disabling flat networks.

DocImpact
UpgradeImpact

Change-Id: I13eb22afd7c0d011154b58a1bd218fd4fa82a8d9
Closes-bug: #1437745
Co-Authored-By: Martin Hickey <martin.hickey@ie.ibm.com>

Add PyPI link for networking-calico

Change-Id: I9978f31029cf4b2a3fe1b23ec63ff7db868de630

Deprecate new= argument from create_connection

It's not used since the time we switched to oslo.messaging (Juno), it's
time to deprecate and eventually remove it.

Closes-Bug: #1506492
Change-Id: I57b0229c2b6028796cd10bbbfc9b166cf8a6dab0

Merge "Add reno for release notes management"

Merge "DVR: only notify needed agents on new VM port creation"

Merge "Ensure to decode bytes or fail"

Merge "Remove default=None for configuration bindings"

OVS agent reacts to events instead of polling

OVSDB monitor generates the events that the OVS agent
needs to process (device added or updated). Instead of
polling the agent processes the queue of events.

Change-Id: I168a3cc3aa96a809153a30635ad7bda29e8ee47c
Partially-Implements: blueprint restructure-l2-agent

Merge "Log hints should only be used for log messages"

Merge "Optimize delete_csnat_router_interface_ports db query"

Remove default=None for configuration bindings

The default value for a configuration variable is None. There is
no need to specifically set these as None.

The line where this is done is:
https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L684

TrivialFix

Change-Id: Iedf29b13d69ea9f2eb4d5aa746d7c1da8d83bafc

Merge "Formatting exception messages"

Log hints should only be used for log messages

Removed log hints for exception translations.

Please see http://docs.openstack.org/developer/oslo.i18n/guidelines.html

TrivialFix

Change-Id: I6c46867568811d24964daa7468acf704f69b01b6

Merge "Add a note about the Neutron Bugs team in Launchpad"

Add reno for release notes management

Change-Id: I0c9b6d9a0e77b57e3637260b6e73edfe78afe5ae
Signed-off-by: Kyle Mestery <mestery@mestery.com>

Merge "Update deprecated messages"

Add a note about the Neutron Bugs team in Launchpad

This team [1] has the ability to handle administrative tasks around
bugs, so lets make clear what the very low bar is for entrance.

[1] https://launchpad.net/~neutron-bugs

Change-Id: I77dcadfcab4f6d95e17bf020d7ee35f590d3fe25
Signed-off-by: Kyle Mestery <mestery@mestery.com>

Merge "Delete fipnamespace when external net removed on DVR"

Merge "PortOpt cleanups"

Merge "Change function call order in ovs_neutron_agent."

Merge "Fix incorrect passing port dict in pluggable IPAM"

Update deprecated messages

Commit 8022adb7342b09886f53c91c12d0b37986fbf35c marked the
methods read_cached_file, find_config_file and
get_keystone_url as not used in Liberty.

This code only landed a few days ago. Updated the deprecated
message to be after Mitaka so that external plugins will have
some grace period to clean up their code.

Change-Id: Ia5ce564837c4855450aa9da94c276e4beb62900c

Change function call order in ovs_neutron_agent.

Change function call order in ovs_neutron_agent during the
creation or modification of a port, in order to fulfill the
VLAN OVS tag information in the "port" register before calling
the SG agent. This information is needed in some SG agent
implementations.

Closes-Bug: #1512636
Change-Id: I9813aca6443ac402b10b4cebf8be42416628b050

Ensure to decode bytes or fail

The commit 048316e98114145ad19285cdeedd0040b59a66f8 introduces the
pattern:

  if isinstance(line, bytes):
    try:
      line = line.decode(encoding='utf-8')
    except UnicodeError:
      pass
  # concat line with a string

which is not working in PY3K if an UnicodeError is raised because line
is (silently) not decoded and concatened to a string.

This change ensures to return a text object or to raise an error.

Closes-Bug: #1503415
Blueprint: neutron-python3
Change-Id: I16b8013f33aa3efad65be8040d3210120e047bbd

Merge "Add IRC part for effective neutron"

Merge "Update specs backlog directory"

Optimize delete_csnat_router_interface_ports db query

Currently delete_csnat_router_interface_ports provides an iterator as
filter to core_plugin.get_ports which is used in a SQL IN argument.
SQLAlchemy is able to avoid some db queries when the IN argument is
empty BUT not when the argument is an empty iterator and raises the
warning:

 SAWarning: The IN-predicate on "ports.id" was invoked with an empty
 sequence. This results in a contradiction, which nonetheless can be
 expensive to evaluate. Consider alternative strategies for improved
 performance.

This change replaces the iterator by a list in order to optimize db
queries when the filter is empty.

Closes-Bug: #1503852
Change-Id: I4f53433d688dfa020927118a18020b1a6aa55517

Merge "Add section for code review in effective neutron"

Make string representation of DictModel generic

The previous version was specific to a port dict and would
fail if str() was called on other models. This iterates over
and logs all values of the dict instead so it works under any
dictionary.

Change-Id: I630b39338e4e67080e100a9f32c9b6e341a93472
Closes-Bug: #1512753

Add IRC part for effective neutron

Change-Id: I20397c3caaea0a109d6b1e0685f9995e5839a38d

Merge "Docs: clarify that AnySubnetRequest is optional"

Merge "sub_project_guidelines.rst: Clarify stable branch creation for subprojects"

PortOpt cleanups

Commit 44d73d1ad3073958c8d5cce933deedc7e6ea5f83 added support
for oslo.config 2.6.

Commit 4d2cb851b7b7d6399163ca9f624f656a3e41ac34 added this to Neutron.

The patch adds a few missing items.

Change-Id: Id36515e28458c354ddb6fe3656f182d17df08f8a

Merge "Fix QoS VALID_RULE_TYPES location in devref"

Merge "Add ops tag to bugs policy"

Merge "Use a more pythonic string construction"

Fix QoS VALID_RULE_TYPES location in devref

In QoS devref, replace neutron.extensions.qos.VALID_RULE_TYPES by
neutron.services.qos.qos_consts.VALID_RULE_TYPES.

Change-Id: Id98b906733fd44bb03da2b38f3bb9a9e35174bfc

Docs: clarify that AnySubnetRequest is optional

The phrasing of "may not support" was a little confusing, mostly because
I read it as an imperative - "you may not do this" instead of it
being "you can do this if you want"

Change-Id: Ia4fb87a649f1d4bc312f4ad3bb5c2f48a7b6741b

Merge "Update neutron-debug to use stevedore aliases"

Merge "Replace internal oslo_policy mock with public fixture"

Merge "Per-branch HEAD files for conflict management"

Update neutron-debug to use stevedore aliases

Updating neutron-debug as it is importing interface driver classes
directly and not using the load_driver_interface method.
Also adds unit tests for the neutron-debug tool.

Change-Id: Icf095b2182225e63e85d849d83c2cb36b8a39217
Closes-Bug: #1504536

Fix incorrect passing port dict in pluggable IPAM

_ipam_allocate_ips expects to receive port dict as a third argument, but
'changes' were passed instead.

This issue affects only third-party vendors who implement own IPAM
driver with custom AddressRequestFactory and uses port dict in it.

Updated arguments passed to _ipam_allocate_ips to include port dict as a
third argument.

Change-Id: I25eb86f46e5412cfacad9a751a94f7720f1a7bd0
Closes-Bug: #1511707

Per-branch HEAD files for conflict management

We have pep8 check for validation revisions, but it allows
outdated changes go into merge queue. To prevent this added
CONTRACT_HEAD, EXPAND_HEAD files.

Closes-bug: #1505701

Change-Id: Ie4b727e55a0b1ecb12e915a0037094a928d8f975

Merge "Adds base in-tree functional testing of the dhcp agent (OVS)"

Replace internal oslo_policy mock with public fixture

This replaces a mock on an internal oslo_policy http class
with a public fixture they provide.

Related-Bug: #1503890
Change-Id: Ifa445b699f71379c7922960375a5e1d25f873f91

Merge "DHCP agent: log when reloading allocations for a new VM port"

Merge "IPSet Manager: make code more pythonic"

sub_project_guidelines.rst: Clarify stable branch creation for subprojects

Change-Id: I27914c6fbd091f4e03e5fbc23f16b2278f800b69

Merge "Decompose ML2 mechanism driver for Mellanox"

Use a more pythonic string construction

Use interpolation rather than concatenation of literals and variables.

Change-Id: I1762534586751aa29174afcd67823468abcde123

Merge "Optimize get_bridge_for_tap_device"

Merge "Use oslo_config new type PortOpt for port options"

Merge "Removes the use of mutables as default args"

Merge "Don't update metadata_proxy if metadata is not enabled"

Add ops tag to bugs policy

Change-Id: Ia7435e3c2e9bd021b1966c811a41775cbf39c543

IPSet Manager: make code more pythonic

Remove unnecessary brackets.

TrivialFix

Change-Id: I34e29a9911be9731bc9f4b2d7f3b055e4b85a438

Merge "Remove deprecated nova_* options"

Imported Translations from Zanata

For more information about this automatic import see:
https://wiki.openstack.org/wiki/Translations/Infrastructure

Change-Id: I1cec4c9071a03288cd1cf33b19c42c95785b4fb4

Merge "Fixed a bunch of typos throughout Neutron"

Remove deprecated nova_* options

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

Merge "Decompose ML2 mechanism driver for OVSvApp"

Fixed a bunch of typos throughout Neutron

Went through all of the docstrings in Neutron and did
some cleanup.  I'm sure there are bunch more that I have missed.

Change-Id: Ib29d2de1c580880c89ed4fd069e1515d0977a3e7

Decompose ML2 mechanism driver for Mellanox

This change removes Mellanox ML2 mechanism driver after its definition in
networking-mlnx.

DocImpact
Closes-Bug: #1511234
Change-Id: Ie48ac22d8e7477c22ad1d48aa0c44153a01eb242

Merge "Add text for deprecated parameter"

Add text for deprecated parameter

Add information for "max_fixed_ips_per_port" option that the
option is deprecated and will be removed in the N release.

Co-Authored-By: Takanori Miyagishi <miyagishi.t@jp.fujitsu.com>
Change-Id: I9aa876c371e7b6cff2068dc641c5e4b5ebda8001
Closes-Bug: #1503088

Clarify with example mentioning gratuitous whitespace changes

Maybe I'm just too picky but I usually call this stuff out in reviews
even if I decide to +2 the patch set.

Change-Id: I5a2667d44203147e112f5a1d7573105478d805f7