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"

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

Merge "Add IRC part for effective neutron"

Merge "Update specs backlog directory"

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"

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"

Merge "Enable specific extra_dhcp_opt to be left blank"

Merge "Better tolerate deleted OVS ports in OVS agent"

Merge "get_device_by_ip: don't fail if device was deleted"

Merge "Remove deprecated namespace deletion options"

Merge "Fix _restore_local_vlan_map race"

Merge "Imported Translations from Zanata"

Merge "Use string formatting instead of string replace"

Merge "Deepcopy port dict in dhcp rpc handler"

Merge "Mark for removal deadcode in neutron.common.utils"

Merge "Set security group provider rule for icmpv6 RA in DVR"

Merge "Move test_extend_port_dict_no_port_security to where it belongs to"

Imported Translations from Zanata

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

Change-Id: I9b3c875c0dd9c89f61e664be32b73eacb539b8f9

Merge "DVR: notify specific agent when deleting floating ip"

Merge "Remove GBP as a Neutron sub-project"

Support migrating of legacy routers to HA and back

This patch adds support for migration of legacy routers to HA and
vice-versa. This patch also:

1. Reverts I4171ab481e3943e0110bd9a300d965bbebe44871, which was used to
   disable such migrations until support was inserted to the codebase.
2. Adds an exception to indicate that such migrations are only available
   on routers that have their admin_state_up set to False.

Closes-Bug: #1365426
DocImpact (Handled in patch 233695)
Change-Id: Ie92f8033f47e1bf9ba6310373b3bfc9833317580

Use string formatting instead of string replace

Currently linuxbridge_neutron_agent[1] uses the pattern:

 PLACEHOLDER = "..."
 PATH = PREFIX + PLACEHOLDER + SUFFIX
 ...
 my_path = PATH.replace(PLACEHOLDER, my_value)

which could be replaced by the following which is more common and in
general safer:

 PATH = PREFIX + '%s' + SUFFIX
 ...
 my_path = PATH % my_value)

Change-Id: I2d23b11fa6fb7f31c1abbe36523b75bfc81f5c46

Merge "Fix error code when L3 HA + DVR router is created or updated"

Delete fipnamespace when external net removed on DVR

The fipnamespace is associated with an external network
on a given node. In the case of DVR there is just one
single FIP namespace for a given node.

We have seen some race conditions in the agent for creation
and deletion of the fip namespace. See the bug report for
details on the failure.

So in order to address this race condition and make the
code more stable, we will be cleaning up the fip namespace
only when an external network is removed.

The server will be sending a rpc notification message to
the agent to cleanup the fip namespace when the external
net is removed.

This patch address the above mentioned issue by not constantly
deleting and creating the fip namespace.

Closes-Bug: #1501873
Change-Id: I86869f66d4afffad7db09942578b1a456a9bd418