Updated documentation about minimum guaranteed bandwidth support
for OVN backend. This mech driver is capable of informing the
Placement API about the bandwidth resources of each chassis and
schedule a port depending on the minimum bandwidth rules applied.
This patch also updates the document pointing to the new
``resource_requeste`` blob passed to Nova with the information of
the requested resources of each port.
Related-Bug: #1578989
Change-Id: I75401fa64e1ccd814aa485a74b38092849ff2a18
Use importlib.util module instead of imp, since it's
being deprecated:
DeprecationWarning: the imp module is deprecated in favour of
importlib; see the module's documentation for alternative uses
Had to change test setup to call super() first to get around a
config issue I was seeing locally, causing the entire class of
tests to fail:
oslo_config.cfg.NoSuchOptError: no such option api_extensions_path
in group [DEFAULT]
Closes-bug: #1981077
Change-Id: Ic171028a661c3f9f83f6758a57aaeab4450aa907
L3_NAT_dbonly_mixin.update_floatingip_status is called from methods
which are decoarted with retry decorator or from within a db context.
As update_floatingip_status also called from OVNL3RouterPlugin added a
utility method, update_floatingip_status_retry with decorator to avoid
retrying OVN methods, and added CONTEXT_WRITER here also as other calls
of update_floatingip_status from l3_rpc for example uses CONTEXT_WRITER.
Change-Id: Ifdb008c2402aa24766cbd30dd53458599076c955
Closes-Bug: #1980721
This bug was previously fixed for recordsets, but can also fail
when creating zones.
Closes-Bug: #1980673
Related-Bug: #1923700
Change-Id: Icef3eab6a27f3b8950a7877de0d387e668a067ad
The doc build needs WebTest, which used to be imported as a transitive
requirement via pecan, but the latest release of pecan dropped this
dependency. So make this requirement explicit.
Change-Id: Ib3f3318440c79909090616b9d480b8dbf55e7b81
In the L3 functional tests framework module there is already helper
method called wait_until_ha_router_has_state which should be used to
wait for desired HA router's state.
This method has proper debug logging added so debugging issues in CI is
easier when it's used.
It is also decorated with unstable_test decorator to skip tests when
router will fail to transition to desired state (see related bug for
details).
In some tests this method wasn't used so we couldn't benefit from the
logging and unstable_test decorator there. Now it should be unifed and
used everywhere in the same way.
Related-Bug: #1956958
Change-Id: I9d79b123bb20ded327208d84a14d4f8d2e505087
https://docs.openstack.org/nova/latest/admin/aggregates.html link failure,
From openstack U version, this link is not in the user directory. Currently, only the latest version has been changed.
You are advised to change all the links
Change-Id: Ic3b5a0ac7d832b162848b363396264ed0bfc4a25
Running with a stricter .pylintrc generates a lot of
C0330 warnings (hanging/continued indentation). Fix
some of them, about 10%.
Feel free to reject if we think it will cause too much
trouble with cherry-picks, else I'll slowly work my way
through the rest of the tree.
Trivialfix
Change-Id: I3d484d11e273cb8ee617f9445a069887e7b2b89f
Now neutron-tempest-dvr-ha-multinode-full has been replaced in tempest,
nova and devstack with neutron-ovs-tempest-dvr-ha-multinode-full. This
definition should be no longer needed.
Change-Id: I3351ae8e1f93ddd8c86830b7677918606ca8d839
This reverts commit 7c95150c84d77f3eae69def5b1328243ccec248a.
Reason for revert: The bug related to this change has been fixed for a while: https://bugs.launchpad.net/neutron/+bug/1906490
Change-Id: Icd8208d63be413f45fcd0c8652ab02a3a53ffc3a
This reverts commit a1384678b88fc7ffb5eeccfbfc0da16a9cb8910a.
Reason for revert: https://bugzilla.redhat.com/show_bug.cgi?id=2092856 fixed in libvirt-8.4.0-3.el9 and available in CentOS 9-Stream.
Change-Id: I2f33543fa7ee65ce3d25f70af49a18b69f799aec
Related-Bug: #1979047
Related-Bug: rhbz#2092856
During the Zed PTG it was decided to handle unsupported features in
Neutron as experimental. See section titled "When we say something is
not supported?", day 2 in [1]. The agreement was:
"We keep existing jobs for linuxbridge driver for example, but when the
tests start to fail we skip them and finally we stop the job also.
To make it clear for operators we add warning logs highlighting that the
given feature/driver is experimental, and introduce cfg option to enable
such features explicitly."
This commit implements this agreement, initially with Linuxbridge
Depends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/845646
[1] https://lists.openstack.org/pipermail/openstack-discuss/2022-April/028164.html
Change-Id: Ib18efa3f472736b58c8967847b1061da0e3897d7
All "reserve_provider_segment" implementations create their own
DB contexts, if needed.
Related-Bug: #1976357
Change-Id: Ic5c8a80f1dc0bebbd117ad9b56365e8c606c112d
This makes OVN asynchronously set up destination port while VMs
are being migrated, reducing network downtime.
Also configure 'rarp' activation strategy to wait until libvirt
notifies the network about new VM location with a RARP issued from
the vif.
Change-Id: I55811ea72b2bf0d0b244f422861dca4a7bf8f257
The method "_handle_segment_change" was handling the network DB object
outside a DB writer context.
Closes-Bug: #1980346
Related-Bug: #1976357
Change-Id: Ic79ad0f8e7fa1d55328e2315d87a3950fdb82258
This is changing the datastructure that maintains the relationship
between ports and networks to also handle the segmenation ids related.
This will be necessary in future to support multiple segments per
networks on a same physical provider network.
Partial-Bug: #1956435
Partial-Bug: #1764738
Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com>
Change-Id: Iaf40ddc20692a3a51a8d5f5acfc2094b2d5c00c4
An OVS DB event once matched, is stored in a cache and processed in
``ovsdbapp.event.RowEventHandler.notify_loop``. The method
``update_virtual_port_host`` is called in the ``run`` method of the
event. Sometimes in the FT execution, the test checks if the
method has been called when the event ``run`` method has not been
executed yet.
This patch adds a short active wait to ensure the event is actually
processed and the method called.
This bug can be easily reproduced if in the
``PortBindingUpdateVirtualPortsEvent.run`` method, a 2 seconds delay
is added:
import time; time.sleep(2)
Closes-Bug: #1971672
Change-Id: I2e389e3cce70505ce01f03c09b36fa9d55be964a
Similar to [1], this patch moves the PostgreSQL tests to a second
test execution step, where all tests are executed with one single
worker, avoiding any parallel execution on the SQL tests.
[1]https://review.opendev.org/c/openstack/neutron/+/834746
Closes-Bug: #1980127
Related-Bug: #1962594
Change-Id: I3a9f665653c3e2a5f75175867a45391e6ad01176
Actual fix is already proposed in rally[1], but
rally CI is red for other issues, so to unblock
neutron gate let's temporary make this job
non-voting.
ovn rally job is already non-voting.
[1] https://review.opendev.org/c/openstack/rally-openstack/+/847879
Related-Bug: #1980055
Change-Id: I39c9db9f3a1eea3cea4f1f4979e0abdce6946bea
