This patchset fixes tempest test case test_filtering_shared_subnets that
fails when executed with the segments extension enabled, due to the fact
that the 'segment_id' attribute is returned in subnets listed with an
admin client but not in subnets listed with a non admin client.
Patchset https://review.openstack.org/#/c/459439/ proposes to enable the
segments extension in the gate.
Change-Id: I6b9bd6fe944c9485c21d25b37c943f3bd53fd7c9
Closes-Bug: #1690598
We use "foo_host" as a hostname in a test.
Usually "_" is not allowed in a hostname. (See RFC 952 etc)
Oslo.config (correctly) complains on that.
Closes-Bug: #1689019
Change-Id: Ife817d4f9f904ab1cecb1d8501b94069d5468bdf
Currently when there is an IPv6 gateway set, IPv6 forwarding
configuration is skipped. This patch fixes it and also adds test
coverage to check the values of `accept_ra` and `forwarding` with
and without an IPv6 gateway.
Related-bug: #1667756
Change-Id: I0297aa6d0afeb56a8c69be41424d4b70509377d2
When run in an HA or DVR configured environment,
the test_update_router_extra_attributes_bumps_revision
API test can fail if the release does not support
router migration from CVR-HA to DVR.
Although Ocata and later releases support any type of
router migration, older ones do not, so the test
should be explicit and test a known valid migration.
Added missing l3-ha extension requirement from v1.
Change-Id: Ic48d078542492e933f71d24df85c54c53a0b110c
Related-bug: #1679794
Eventlet now deprecate TimeoutError exception [1].
Updating eventlet to latest 0.21.0 can cause UT fail because
of misunderstood between deprecated exception and the new exception class
Timeout. [2]
This patch remove deprecated exception usage, fix the UT failure with eventlet
0.21.0 in order to upgrade eventlet upper-constraint to latest version.
[1]. https://github.com/eventlet/eventlet/blob/master/eventlet/__init__.py#L55
[2]. https://review.openstack.org/#/c/460424/
Change-Id: I48179953ef19ec6b2f3f6706337a1dae1b3830bd
n8g-sfc currently has its own variant of OVSBridge to allow the use
of priority in a delete_flows call
This change is meant to make this available outside n8g-sfc and
simplify n8g-sfc code.
This change adds a 'strict' boolean parameter to mod_flow and delete_flows
that results in ovs-ofctl to be run with --strict for del-flows and
mod-flows actions. When strict is set, the use of priority is allowed
and hence not rejected anymore.
Note that for batched actions in a deffered bridge, we disallow mixing
calls with strict and without strict, which can't be translated in one CLI
call.
Needed-By: I3bf939590dd43bff685f133bff86eb7e9068de91
Change-Id: I289d546780f10dc1002ab6bc2e1b38c9ef2d728f
A recent change [1] added code to the linux DHCP driver
to check the subnet segment_id attribute when adding
extra host routes for a subnet. I have seen both
KeyError and AttributeError failures in various check
jobs lately since there is no guarantee all subnets
have this attribute. Be safer by using getattr().
[1] https://review.openstack.org/#/c/438171/
Related-bug: #1668154
Change-Id: I96bea4edab4c96f2ca28ba854f6ca8d4ab4df5f5
This patch changes way how neutron calculates which QoS rules
are available to use. It now returns all rule types which are
supported by at least one loaded QoS driver.
If user will want to apply policy with rule unsupported by driver
used by port then it will be catched on port/network update event.
This validation mechanism was introduced in
I75bd18b3a1875daa5639dd141fb7bbd6e1c54118
DocImpact: list of returned available QoS rule types is changed
Change-Id: Ia00d349625db358ab486802fc0ff2e69eaa3895e
Closes-Bug: #1686898
While reading the iptables_firewall file, I didn't see these
functions being used anywhere in openstack.
If I'm mistaken, simply abandon the patch.
Change-Id: Ifb3e4a6cec8266dfebb1d76ec67c9c569cf9798b
The patch creates a new in-tree class that overrides default value for
tempest ssh.Client to the one defined by ssh_timeout config option.
Change-Id: I418519866e3cbd486fe0ffec23e1553ea0891420
In the unit test logs there is a MissingAuthPlugin exception
printed several times.
This patch aims to mock the notifier when it's not being tested.
When it is being tested, like in the NovaSegmentNotifierTestCase,
mocking the keystone auth prevents the exception from being
thrown.
Closes-Bug: #1683606
Change-Id: I63a1c887f6a7a43f2bddd48b86078e5836cbf438
This refactoring is a step towards the goal of removing the
CommonDbMixin mixin class.
Related-Blueprint: neutron-lib
Change-Id: I1e2da0687310cc2da767dc2a6d13500307bba1ee
Firstly fix a typo (iteriterms -> iteritems) introduced by [1]. Then a
"dictionary changed size during iteration" RuntimeError is reported by
the newly added unit test.
[1] I17df98128c7a88e72e31251687f30f569df6b860.
TrivialFix
Change-Id: I70c548eab2264b3f94ba1a215f933046345411a7
Signed-off-by: Hunt Xu <mhuntxu@gmail.com>
When run in an HA or DVR configured environment,
the test_update_router_extra_attributes_bumps_revision
API test can fail if the release does not support
router migration from CVR-HA to DVR.
Although Ocata and later releases support any type of
router migration, older ones do not, so the test
should be explicit and test a known valid migration.
Change-Id: I8caf86817a5cd8e4d0cf05d7beef850756e84bab
Related-bug: #1679794
Skip tests when the endpoint is not available, which is the case in gate
after [1]. We'll enable them back once we adopt them to v3.
[1] Ic233f6b43dd1e3cfdadff0f18aba4ea78825a996
Related-Bug: #1687544
Change-Id: I8fffc50fd45e43f34ca591416924bcf29626b21f
This patch introduces the new parameter "direction" to
the QoS bandwidth limit rule. It will allow the creation
of bandwidth limit rules for either ingress or egress
traffic. For backwards compatibility the default direction
will be egress.
DocImpact: Ingress bandwidth limit available for QoS
APIImpact: New type of parameter for QoS rule in neutron API
Change-Id: Ia13568879c2b6f80fb190ccafe7e19ca05b0c6a8
Partial-Bug: #1560961
The recently merged change [1] modified the logic in the way
to break deployments without dns-integration. It broke
networking-midonet gate jobs.
[1] If1252c42c49cd59dba7ec7c02c9b887fdc169f51
Closes-Bug: #1686584
Change-Id: I80696fa1227d1d36ae67d8f2b70e4206c4cdcd70