Merge "Updated from global requirements"

Merge "Return 404 when executing router-list-on-l3-agent with invalid agent_id"

Merge "Default route missing for IPv6 subnets in HA Router"

Merge "OVS UT: Change misleading constants in veth tests"

Merge "test_ovs_dvr_neutron_agent: Use consistent variable names"

Updated from global requirements

Change-Id: Ie0f782d8e0f185c041b169f4831f33723840e8db

Merge "Avoid fetching network in _commit_port_binding"

Merge "VMware: Router Type Extension Support"

test_ovs_dvr_neutron_agent: Use consistent variable names

Stop using expected_in_xxx_br.
Use expected_on_xxx_br consistently.

Change-Id: I5eb66eca6396cbf5a3e3384a442b211576371328

devref: consider sphinx warnings as failures

We previously introduced issues in our documentation due to not checking
for sphinx warnings [1], so let's enforce documentation validation.

This commit also fixes minor syntax issues caught by sphinx. Among
those, contribute.rst is now properly attached to ToC.

[1]: I48ab899077458d99eee6f643577d574106c1ca88

Change-Id: I9170883a20137c343f03d8b052399eacab4ae130

devref: don't link to nonlocal images

External images generate warnings for build-sphinx, and are not
supported for output formats other than html.

Change-Id: I9310577473982884897915dd29e09a2e4aaf51c7

devref: fixed class name for test_migrations autodocumentation

The file name is different, so documentation was not properly generated
before.

Also fixed a docstring not to issue sphinx warning due to incorrect
indentation.

Change-Id: I48ab899077458d99eee6f643577d574106c1ca88

devref: updated documentation for oslo-incubator

The project now has another name (oslo-incubator, not openstack-common).

Include the page into devref index.

Change-Id: I779d7cf35e0ec3f80fa6a53f12ed3ee98dd8c5cb

devref: updated documentation for advanced services

Now that we don't maintain the code for the services, we should not ship
documentation to it, but just refer to it.

Change-Id: Id0ced1196718204dda8d51b26715e2f845792ca1

Avoid fetching network in _commit_port_binding

There's no need to get the network from the DB in
_commit_port_binding since it's stored in the PortContext

Change-Id: Icd5200746dc8fbba336bd4e5714b90b9f89704c0

VMware: Router Type Extension Support

Provide a router type extension. This will enable defining the
type of edge appliance. That is, an exclusive edge or a shared
edge for routing services.

DocImpact

Change-Id: I1464d0390c0b3ee7658e7955e6433f2ac078a5fe

OVS UT: Change misleading constants in veth tests

Make them consistent with test_setup_physical_bridges case.

Change-Id: Ia1d39b498d563142b59cca2ff491bee7385ca3cf

Merge "Remove remaining do_delete from unit tests"

Merge "VMware NSXv: Add configuration options to nsx.ini"

Merge "Move Floating IP processing to Router classes"

Merge "Improve exception handling in _process_router_update()"

Merge "adopt namespace-less oslo imports"

adopt namespace-less oslo imports

Changing the leftovers that I found with latest code.
link: https://review.openstack.org/#/c/147215/
Change-Id: I5adbed9bbe9cb6e819dd2a8a72e7fbeee06fbf00
Closes-bug: #1423389

Merge "Remove reference to self.services_sync"

Merge "VMWare NSXv: Add configuration params to nsx.ini"

Merge "VMWare NSXv: id fields should be nullable"

Merge "Remove remaining uses of passing root_helper"

Merge "Remove versioning import of novaclient"

Merge "Stale VXLAN and GRE tunnel port/flow deletion"

Merge "Refactor to facilitate DVR scale performance"

Merge "Cisco Nexus ML2 Vendor decomposition"

Merge "Updated from global requirements"

Move Floating IP processing to Router classes

Change-Id: Id0822f1c3d586c813a09e4b442a21d3cb52943d0
Partially-Implements: bp/restructure-l3-agent

Merge "Enable pylint unnecessary-pass"

Merge "Enable pylint no-value-for-parameter"

Merge "Enable pylint bad-super-call"

Merge "Enable 'free' pylint checks"

Updated from global requirements

Change-Id: I32d3f4333e5ac6e70f68580b9629cde0e1481ecf

Merge "fix for _get_external_device_interface_name trace"

Improve exception handling in _process_router_update()

in _process_router_update() where _process_router_if_compatible()
method is called only RouterNotCompatibleWithAgent exception is handled.
In case any other (intermittent) exception happens inside
_process_router_if_compatible() (e.g. MessagingTimeout on fetching
external net from server) it results in a situation where agent completely
forgets about the router and continues working as usual while server
shows router hosted by agent.

This patch adds handler for broader exception there and set
fullsync = True like it's done earlier in _process_router_update()
when getting routers from server.

Change-Id: I5bfc540a926349848702100744f97c597b852604
Closes-Bug: #1423571

Cisco Nexus ML2 Vendor decomposition

This adds a thin shim replacing the Cisco Nexus mech driver so that it
may use the code now stored in the networking-cisco stackforge. The code
for this driver that has been moved into the stackforge is also removed by
this patch.

Partial-Implements: blueprint core-vendor-decomposition
Change-Id: Ib88fd121509e96be42fb0bd22422ccaa7d4eb0f1
Closes-Bug: 1419010

Merge "Unify logic that determines liveliness of DHCP agent"

Merge "Mock link local allocator write so UT doesn't write a file"

Remove versioning import of novaclient

novaclient has specific function novaclient.client.Client for obtaining
client object. This fuction should be used instead of direct import.
Also, contrib path dependends on version, so we should get it based on
versioned client.

Change-Id: If9c55446c4d10a58e9723f5c333082bcacb431b8
Closes-Bug: #1418017

Remove remaining uses of passing root_helper

This should remove the last bits of passing root_helper around
for execute(). Some tests store and pass the root_helper argument
outside of this, and have been left alone.

Depends-On: I1e3b64e5a1d6cff2aebc638710487bbdbdba61d4
Depends-On: I5d51fa5edc7f427d84d7ae49d3a4c972dca8ac37
Depends-On: I21c6e7795c8b2e8f87bbd2b00a8eb7cb215ece74
Depends-On: I6566e17a1ce07a3bebf2a1a3653ad7e7f397f0b9
Change-Id: I38c901c65fb0c8598589ec56d5afd50d2f882f4f
Partially-Implements: blueprint rootwrap-daemon-mode

Remove root_helper arg from sanity checks

Change-Id: I0e0791b9a04750968675d21b93cee412c50fdd51
Partially-Implements: blueprint rootwrap-daemon-mode

Merge "Fix type of exception in ml2 l2pop"

Enable pylint unnecessary-pass

Change-Id: I5ba003286a9abdcd92c08d9a164282a03eb1144a

Enable pylint no-value-for-parameter

Change-Id: I532dc0c8f0f9f8c92b47a51138d5162bc506016b

Enable pylint bad-super-call

Change-Id: I408ce142a2f2145ac442ad4130681b135f9d52cf

Enable 'free' pylint checks

Change-Id: I4aa1e4a43c50b8ac0f4e97b84479c66d78407625

Merge "Remove remaining root_helper args from plugins"

Merge "Improve robustness of network failover"

Remove reference to self.services_sync

This is an implementation detail of fwaas and shouldn't be referenced
here.  There is no need to call self.process_services_sync on super.

Change-Id: I8797e40480c8c43788f4c192abddaf5d3e146e4c
Partially-Implements: bp/restructure-l3-agent
Depends-On: I6091ca2c03f2b5ffef453dea1152d2ac53bb7f14

Fix type of exception in ml2 l2pop

Closes-Bug: #1423703
Change-Id: Ie6e2b33a8084381c01a35e3ded6b021f540ef42e

Merge "ML2: remove underscore from public method"

Merge "Remove error logs for a common situation (non created ipsets)"

Merge "Remove root_helper arg from linux interface"

Merge "Remove root_helper arg from L3 Agent"

VMware NSXv: Add configuration options to nsx.ini

Adds option to control the use of spoofguard component in NSXv

Change-Id: Iaef46618887a2058b2f9fdbc4eabdb357c444cb3
Signed-off-by: Roey Chen <roeyc@vmware.com>

Mock link local allocator write so UT doesn't write a file

The UTs recently started writing a fip-linklocal-networks file in
test_l3_agent.  This is due to a recently added test.  This patch adds
a mock to it so that the file is not written.

Change-Id: I338a369fe25e64faa84cdec413d6bad7b315f890

VMWare NSXv: Add configuration params to nsx.ini

Adds some additional configuration parameters to nsx.ini, and their
description.
These parameters relate to NSXv metadata implementation.

Change-Id: Id115fed2307543de6553fac2c38d776f507f4021
Partially-Implements: blueprint vmware-nsx-v

Remove error logs for a common situation (non created ipsets)

The log message was initially added by me as part of a
iptables_firewall refactor.

Ipsets for empty IP address lists aren't currently created,
that means that we can't reference empty security groups
(as ipsets) via iptable rules, and that's a normal condition,
not an error.

Closes bug: #1421772
Change-Id: I6b1ae1fb505ce5e76ef8cf7ef7df38cff57e0000

Merge "Fix usage drop_constraint in 2a1ee2fb59e0 migration"

Merge "Combining sec groups member update RPC calls"

Default route missing for IPv6 subnets in HA Router

In an HA setup, keepalived would configure the default gateway (in the master
HA router) by parsing the "virtual_routes" section of keepalived.conf file.
The "virtual_routes" section is properly constructed for an IPv4 subnet, but
requires a minor change for IPv6 subnets.

Closes-Bug: #1414937
Change-Id: I54070e048577292d98d86fcc9ca8b9d3e72b4ebe

Merge "Fix static strings with labels for DVR"

Merge "Enable super-on-old-class pylint check"

Merge "Added policy for lbaas v2 agent extension resource"

Merge "Fix index name in downgrade 26b54cf9024d migration"

Unify logic that determines liveliness of DHCP agent

For DHCP agents sometimes it's not enough to check agent's last heartbeat
time because in its starting period the agent may fail to send state reports
because it's busy processing networks.
In rescheduling logic such DHCP agent is given additional time after start.
Additional time is proportional to amount of networks the agent is hosting.
Need to apply the same logic to DHCP agent scheduler to avoid a case
when starting agent is considered dead and a network gets more hosting
agents than configured.

Change-Id: I0fe6244c7d2ed42e4744351be34f251318322c54
Closes-Bug: #1417708

Merge "Tweak mocking logic for L3 plugin tests"

Merge "Moved several services into neutron.cmd.eventlet"

Merge "Check if routing rule exists before adding"

Merge "Add a functional test that checks HA router is configured on a restarted agent"

Merge "Remove root_helper arg from DHCP agent"

Merge "Remove root_helper arg from AsyncProcess"

Merge "Remove root_helper arg from linuxbridge"

Merge "Update midonet plugin requirements"

fix for _get_external_device_interface_name trace

On removal of external gateway from DVR the code path
external_gateway_removed(...) was trying to access the
agent gateway port interface even when no fip namespace exists.
This change checks for existence of namespace before
accessing the agent gateway port interface through
_get_external_device_interface_name(...) or looking for
floating ips that may have been configured on the
port.

Change-Id: Idcf28ff93f16f1d692fe7acb678fb90aabe5af5e
Closes-Bug: #1421497

Merge "Get rid of rpc to fetch fip agent port on agent."

ML2: remove underscore from public method

In some refactors the private method was used from
outside the class but name wasn't updated.

Change-Id: I594af2661c4773e0d2c35ec924a4142057442086

Fix static strings with labels for DVR

There are couple of static strings introduced for
DVR configuration such as the L3 agent mode.

This patch would clean up the static strings with
the labels.

Closes-Bug: #1411351
Change-Id: I143262e62ee994e6c01d63e4ea7d5f6292de8286

Merge "Monkey patch all the code inside neutron/cmd/eventlet/..."

Merge "tests: monkey patch stdlib before importing other modules"

Get rid of rpc to fetch fip agent port on agent.

This patch is dependent on the plugin side patch
Change-Id: Ieaa79c8bf2b1e03bc352f9252ce22286703e3715
for retrieving the fip agent port from the
router_update message.

This would reduce the wait time substantially.

Change-Id: I47bc43bab4bff59d14e2cdbce9f8b47826d392d9
Related-Bug: #1415522

Combining sec groups member update RPC calls

This change allows combining multiple RPC calls for
security group member/provider updates that are needed
as a result of create bulk port operations.

Change-Id: Ibfe84f40e55a9847cbcaa4a7344988b2d3fb895e
Closes-Bug: #1373634

VMWare NSXv: id fields should be nullable

To handle concurrency issues when two neutron nodes initialize at the same
time, we use the following solution:
- Each neutron node attempts to create a record with id value=NULL
- One which inserts successfully, resumes to create the resource and once
the resource is created, it updates its id in the database
- One which fails with duplicate entry exception, busy-waits until the id
field is populated, of terminates on timeout.

Relevant resources are internal edges, and internal networks.

To implement the above, id fields should be nullable.

Change-Id: Iff94678ab341ccb1310d6b6efc53a52bd3bb9240
Partially-Implements: blueprint vmware-nsx-v

Merge "Don't monkey patch netns_cleanup"

Merge "Remove root_helper arg from SecurityGroupAgentRpc"

Check if routing rule exists before adding

Since there is no equivalent to 'ip route replace...' with
'ip rule...', we need to check the existence first before
adding, as otherwise we'll end up with multiple identical
routing rules.

Change-Id: I8f1f2424dd854fad8a72478463fa8fadd6a5c168
Closes-Bug: #1398865

Merge "Remove root_helper arg from IpsetManager"

Remove root_helper arg from DHCP agent

Partially-Implements: blueprint rootwrap-daemon-mode
Change-Id: I394fc2bf8b9579203fc0a878047a57cf6e09927e

Remove root_helper arg from AsyncProcess

Removes the root_helper arg from AsyncProcess and areas whose use
of root_helper was soley for passing it to AsyncProcess.

Change-Id: I6c34f2d8466444c82f8712a2a765975695795fd7
Partially-Implements: rootwrap-daemon-mode

Merge "Remove root_helper arg from IptablesManager"

Merge "Remove root_helper arg from external_process"

Merge "Skip DBDuplicateEntry exception in security group creation"

Remove root_helper arg from linuxbridge

Partially-Implements: blueprint rootwrap-daemon-mode
Change-Id: Ie075cd4f505ed76e820e29cce1a55b6e8192b3df

Remove root_helper arg from SecurityGroupAgentRpc

Removes the root_helper argument from SecurityGroupAgentRpc and
removes usages of root_helper that were solely required for it.

Change-Id: If166fda9fac7718ae7bbc86756884ddea8dd70d5
Partially-Implements: blueprint rootwrap-daemon-mode

Moved several services into neutron.cmd.eventlet

- dhcp-agent
- l3-agent
- metadata-agent
- metadata-proxy
- metering-agent
- server

This allows us to remove explicit monkey_patch() call.

Also removed ability to execute neutron-server avoiding a corresponding
entry point.

Depends-On: I2d7081dbd4cb532332e3b66667bb8c71aa5a6658

Related-Bug: #1418541
Change-Id: I89e3e8e23374ab1a9a1844b3caaa88e162418546