Merge "Correction and clarification to subproject stable guidelines"

Merge "Add PyPI link for networking-calico"

Merge "Deprecate new= argument from create_connection"

Correction and clarification to subproject stable guidelines

Change-Id: I5511430593b39f7d5a952270aeb43f973090d702

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"

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"

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

Removes the use of mutables as default args

Passing mutable objects as default args is a known Python pitfall.
We'd better avoid this. This commit changes mutable default args with
None, then use 'arg = arg or {}', 'arg = arg or []'. For unit code which
doesn't use the args , just set with None. This commit also adds hacking
check.

This code was taken from commit 0bea84ac20fe498bd08f7212a0017196c8cb0812
in Nova.

Change-Id: I36d07cade687690dc02a8f6cc3d70f5d00caf112
Co-Authored-By: ChangBo Guo(gcb) <glongwave@gmail.com>

Decompose ML2 mechanism driver for OVSvApp

This change removes OVSvApp ML2 mechanism driver after its definition in
networking-vsphere.

DocImpact
Closes-Bug: #1511198
Depends-On: Ib7b2852f3d4dc35a837a425366c5bf0135725a74
Change-Id: I5e310f0596f3ea206800ac9d4daca1c292a234f1

Fix usage of mutable object as default value

Commit 0a300a2277a583fe28b00db2571982928c752554 added a bad parameter.
Please see http://docs.python-guide.org/en/latest/writing/gotchas/

Change-Id: I0550c4c86774f8761287c596350f7461fec05fe4

Merge "Optimize interface_exists_on_bridge"

Merge "Log end of router updates for PD and delete branches"

Merge "Make the Neutron Stadium documentation toplevel"

Merge "Add notes about stable merge requirements for sub-projects"

Merge "Imported Translations from Zanata"

Merge "Updated from global requirements"

Make the Neutron Stadium documentation toplevel

This commits moves the Neutron Stadium documentation to the toplevel
portion of the documentation tree.

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

Add notes about stable merge requirements for sub-projects

It was pointed out the requirements around what is allowed to be merged
into stable Neutron sub-projects wasn't clear. This attempts to make
that a bit more refined so those who choose to participate in the Neutron
Stadium know these requirements.

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

Fix incorrect capitilization of PyPI

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

Updated from global requirements

Change-Id: I7a0550f47b885618c53bbea777680c58b4c8ab13

Log end of router updates for PD and delete branches

These execution branches call continue directly without
logging the end of the router update.  This leads to
confusion about when the last router update finishes if
it is a delete or a PD update.

Change-Id: Id82e7374aa9336e0adea3042238f30f716208373
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>
Closes-bug: #1511004

Don't update metadata_proxy if metadata is not enabled

When enable_metadata_proxy is false, the agent instance will
not have metadata_driver. And agent should avoid using it.

Change-Id: Ia18dc5dea23de49b97c8f225532531eb9232fb51
Closes-Bug: #1510399

Imported Translations from Zanata

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

Change-Id: Ic47890d38487f8e5f566da101da649c34f51d9ee

Merge "Log the exception in linuxbridge_neutron_agent as exception"

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

This will help us to see more clearly if failing DHCP for a new VM -
as seen in failures such as the following traceback from [1] - is
caused by DHCP configuration being too slow, as suggested by the
referenced bug.

Traceback (most recent call last):
  File "/home/aqua/tempest/tempest/test.py", line 125, in wrapper
    return f(self, *func_args, **func_kwargs)
  File "/home/aqua/tempest/tempest/scenario/test_server_basic_ops_38.py", line 105, in test_server_basicops
    self.verify_ssh()
  File "/home/aqua/tempest/tempest/scenario/test_server_basic_ops_38.py", line 95, in verify_ssh
    private_key=self.keypair['private_key'])
  File "/home/aqua/tempest/tempest/scenario/manager.py", line 310, in get_remote_client
    linux_client.validate_authentication()
  File "/home/aqua/tempest/tempest/common/utils/linux/remote_client.py", line 55, in validate_authentication
    self.ssh_client.test_connection_auth()
  File "/home/aqua/tempest/tempest/common/ssh.py", line 150, in test_connection_auth
    connection = self._get_ssh_connection()
  File "/home/aqua/tempest/tempest/common/ssh.py", line 87, in _get_ssh_connection
    password=self.password)
tempest.exceptions.SSHTimeout: Connection to the 172.17.205.21 via SSH timed out.
User: cirros, Password: None

[1] http://logs.openstack.org/39/194939/19/check/gate-grenade-dsvm-neutron/82a7e58/

Change-Id: I5d77762bf8d4caa2e6f8b3527a30bf80bac62085
Related-Bug: #1453350

Merge "Replace utils.exec for IpNeighComm LinuxBridge drv"

Update specs backlog directory

A change in the neutron-specs repo requires this to be
updated.

Depends-on: I972a9a56c038864d9c91ead6944c6b9355916668

Change-Id: I7b55ebd6c74002ba9a34bb8a8a88f3602a72b78d

Log the exception in linuxbridge_neutron_agent as exception

The log here should be logged as exception. 1) It is more readable.
2) It keeps consistent with the action of its sibling module,
ovs_neutron_agent.

Change-Id: Idf11a617369cbee0dc23c8e56ae9475bb41dcc02
Closes-bug: #1508270

Replace utils.exec for IpNeighComm LinuxBridge drv

Replace all execution of "ip neigh" command using
utils.exec for ip_lib.IpNeighCommand.(add,delete)
in LinuxBridge driver

Changed multiple which for one and fixed indentations
Closes-Bug: #1311019

Change-Id: I1389941b7df11c089429f1de61eac6d7ea5d0e54

Merge "Fix iptables modules references in rule generation"

Formatting exception messages

This patch addresses below nits for existing neutron exceptions.
 - Fix messages as appropriate.
 - Remove additional spaces.
 - Add periods as appropriate.

Change-Id: I8cfb5b3b5e4fdc2bf1e50ab6d898cb4c21e7ced1

Optimize get_bridge_for_tap_device

Currently get_bridge_for_tap_device[1] iterates over all neutron bridges
and their interfaces.

This change proposes to deduce interface bridge from:

 /sys/class/net/%(interface)s/brif/bridge

which is a symlink to bridge interface path to improve performance.

[1] neutron.plugins.ml2.drivers.linuxbridge.agent.linuxbridge_neutron_agent

Closes-Bug: #1508789
Change-Id: Ia40cd81f47ff082a90d17f58514942ec50553241

Optimize interface_exists_on_bridge

Currently interface_exists_on_bridge[1] lists all files in a folder and
searchs for the file associated to the interface.

This change proposes to look for the file existence directly.

[1] neutron.plugins.ml2.drivers.linuxbridge.agent.linuxbridge_neutron_agent

Closes-Bug: #1509890
Change-Id: I23cd1edc92912b35bdc23ba0af2318b86f2cfd48

Correct indentation in linuxbridge_neutron_agent

Change-Id: I47b3033b5353c895de72e40fc25c099b7b7a077b

Use oslo_config new type PortOpt for port options

The oslo_config library provides new type PortOpt to validate the
range of port now.

Change-Id: I40792ddcee0f89c47defa726fed24f26c4b88ce2
Depends-On: I9c0e3f44cf93db020933d8d766cedfc2e3f3bb8b

Updated from global requirements

Change-Id: I9c0e3f44cf93db020933d8d766cedfc2e3f3bb8b

Merge "Validate ethertype for icmp protocols"

Merge "Support migrating of legacy routers to HA and back"

Merge "Add another patch scoping bullet point to effective_neutron"