Jenkins [Fri, 16 Oct 2015 02:26:57 +0000 (02:26 +0000)]
Don't snat traffic between fixed IPs behind same router
This fixes a bug where an iptables rule to not snat traffic between
fixed IPs is only being added if enable_snat=true. We should add
this rule no matter what the value is for enable_snat.
Without this patch, current code will break such use case:
2 fixed IPs behind same router both have floatingip associated. And
the router has enable_snat=false. When fixed IP A want to ping
fixed IP B, fixed IP A will get the reply from fixed IP B's floating
IP.
Russell Bryant [Mon, 9 Nov 2015 14:31:36 +0000 (09:31 -0500)]
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>
yan.haifeng [Mon, 9 Nov 2015 02:27:54 +0000 (10:27 +0800)]
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
Henry Gessau [Thu, 5 Nov 2015 16:47:13 +0000 (11:47 -0500)]
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.
Akihiro Motoki [Sat, 7 Nov 2015 18:07:02 +0000 (03:07 +0900)]
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.
Carl Baldwin [Mon, 5 Oct 2015 21:35:37 +0000 (21:35 +0000)]
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.
Martin Hickey [Wed, 4 Nov 2015 21:37:21 +0000 (21:37 +0000)]
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>
rossella [Thu, 5 Mar 2015 09:24:10 +0000 (09:24 +0000)]
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.
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 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.
Cedric Brandily [Tue, 3 Nov 2015 21:54:07 +0000 (22:54 +0100)]
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.
Kevin Benton [Tue, 3 Nov 2015 15:31:54 +0000 (07:31 -0800)]
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.
Sean M. Collins [Mon, 2 Nov 2015 19:47:13 +0000 (14:47 -0500)]
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"
Martin Hickey [Wed, 21 Oct 2015 14:24:13 +0000 (15:24 +0100)]
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.
AKamyshnikova [Thu, 8 Oct 2015 15:18:21 +0000 (18:18 +0300)]
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.
Gary Kotton [Thu, 29 Oct 2015 14:40:05 +0000 (07:40 -0700)]
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.