As documentation was refactored, one link in CONTRIBUTING.rst is
obsolete.
Change-Id: Ie7ce1ddd4a75eb418b8e37211f2058daad05d6f9
Signed-off-by: Arnaud Morin <arnaud.morin@corp.ovh.com>
Otherwise tests will use executables from global paths (f.e. /usr/bin/)
when those are available, which doesn't play well with a system that has
devstack installed.
Change-Id: I3213d5cb570b09d5049b7087cd710337e2698eb3
Tag support was extended to all resources that support the std attr
mechanism. This patch handles a few remaining updates in the docs.
This one is also a candidate for backport to pike if warranted.
Change-Id: I7dfc7a371e6af64dde09c0e712424b8e647d7027
Closes-Bug: #1706307
QoS policy can't be removed from Neutron if it is used by
port or network. Because of this restriction there was method
to disassociate QoS policy from network/port and it had to be
called in each test where policy was attached to network/port.
This patch changes order of cleaning resources that QoS rules and
policies are always cleaned after all networks and ports are removed.
Thanks that there is no need to disassociate QoS policy from
network or port manually in each test.
Change-Id: Id9f6952fb68e1ad35fcd9ce99b9602247e3a229a
_get_floatingips_bound_to_host function was introduced
recently in dvr_local_router to retrieve the external
interface name for centralizing the floatingip.
This function was throwing a 'KeyError' on fip['host'] and
not required for centralized floatingips anymore.
The get_external_device_interface_name in dvr_local_router
will try to get the 'fg' interface that is required for
the bound floating-ips to clear up some of the rules.
In the case of the centralized unbound floating-ips, the
'qg' external interface is retreived from
get_snat_external_device_interface_name that is defined
in 'dvr_edge_router' and based on the namespace.
So _get_floatingips_bound_to_host can be removed from
get_external_device_inteface_name.
Closes-Bug: 1712412
Change-Id: I94c0a071df32f572745a2c29942956c3da9f309b
A number of our RSTs under doc/srouce/contributor/internals/* are not
referenced from the index.rst and as a result are not
published/accessible in our docs.
This patch updates the index to ref all internal pages.
Change-Id: If534f8d6a7da2807285edb6885c800696d1c50a9
Closes-Bug: #1712907
In case when network was created with different than default
client (like admin_client) network wasn't properly removed in
cleanup_resources method because default client was used to
delete function and 'NotFound' error returned from Neutron
was silently ignored in such case.
Now networks created by different than default client are
added to "admin_networks" list and are cleaned by admin_client.
Change-Id: I52428262276c16dbe077fcf77b1890f12dccc97d
The ubuntu controller install guide contains improper indentation and
extraneous new lines. As a result the sub-steps for #1 are not shown
in this HTML (generated) guide.
This one needs to also get back-ported to pike.
Change-Id: Ib2b263c8da49ccc8905cbd59331ce6694de232e6
Closes-Bug: #1712107
Change I379448fd135a3bb2ac99709812b9f476c73923a7 replaced connection
attribute of IDL to ovsdb_connection. After updating upper constraints
to newer ovsdbapp version, we no longer have connection attribute.
Change-Id: I6f060fff4df7323a3877f56cb1fa586a22d16670
Closes-bug: #1712594
In _create_network_policy(), policy object is passed into _get_policy_obj()
although it expects to be passed policy id. This patch passes policy id
instead of policy object and adds a unit test and API tests.
Change-Id: Ic4c755c54d93d7ca3312b432b7c1e6770f25c359
Closes-bug: #1712278
In commit [1] (some explanation in [2] ) VRRP initialisation is enhanced
to read source IP address(to use when sending VRRP packets) from the
HA interface or from keepalived config("unicast_src_ip" parameter).
If it is unable to find IP address, VRRP initialisation will fail with
error "Cannot find an IP address to use for interface".
In the test, we set vrrp->family to AF_INET by setting vip to
169.254.0.1/24 through config, but not providing source IPv4 address(i.e
no 'unicast_src_ip' option or no IP on HA interface), making the test
to fail with [1]. To fix that, we set the IP address on HA interface.
Note: Commit [1] is added in Keepalived version 1.2.20.
Tested the fix on both Keepalived v1.2.19 and Keepalived v1.2.20.
[1] https://github.com/acassen/keepalived/commit/37488e57
[2] https://github.com/acassen/keepalived/issues/445
Closes-bug: #1712388
Change-Id: I260c0e6810ed54c93f93621afa6ab13855ef2428
Today the ML2 config reference doesn't mention anything about ML2
extension drivers. While it likely doesn't make sense to clutter the
primary ML2 driver types (type/mech) with extension drivers, this patch
proposes we add a small section about them to link the concept to the
config reference.
Change-Id: Id964e6e4d13d400ea4cd8a1707ad0f08c52ef0b1
Partial-Bug: #1683102
Currently QoS plugin's get_policies doesnt check if the filter
is a dict or not, though similar checks exist in get_rule_types()
and get_policy_rules().
This patch adds the check so that if any consumer passes filters=None
then it is converted to a dict.
TrivialFix
Partial-Bug: #1711165
Change-Id: Id50674c44cd3720dcd8fd6d66361d937599f5992
Netlink-lib creates some conntrack entries to verify list_entries()
method. Each entry contains a zone_id, which might be duplicated with
some already existed entries in OS.
This patch proposes a simple verification to make sure entries created by
netlink-lib do not contain existed zone_id.
Change-Id: I4bce5041f82782cf9e51d99c605593afb2cf4fc8
Closes-Bug: #1708030
v2 identity API was deprecated in Mitaka. There is only a one test
remaining that still relies on the v2 identity API. This patch replaces
v2 with v3.
Change-Id: Iaeb9153084c6864b0982902bbb2ae9c0233266a6
Neutron doesn't use these methods, but they will be added to
ovsdbapp 1.0, and being abstractmethods, would cause an error. The
methods are added in I664add660ce0ec3b0da23325ad38580d850e1f29 and
I9352ffe0f245c579fd766ecd00e8ca512f806798.
Change-Id: Ibfdea4890f7d01ddd38872db40a9829d6b6bdf90
This patch logs the command spawned by RootHelperProcess, it is
handy when debugging failed functional tests on upstream gate.
Change-Id: I743a223c4ff4882fdd760a20823150558d5e5f4a
This reverts commit 75ddc9c34e08133ffb618271ca21499365b90ad7.
The bug mentioned in original patch has been fixed in
kernel 4.4.0-92.115. And current gate runs with the kernel.
This patch reverts the change to re-enable the netlink tests.
Change-Id: I580e76557e026f45c114b0f4ef6bac5253e3add7
