Synchronise zed with upstream #121
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Nova will automatically translate VF capabilities to Neutron port binding profiles after patch [1] will be merged. Existing recommendations in "admin/config-ovs-offload.html" should be updated: there is no need to define capabilities in port binding profiles for new ports anymore. [1] https://review.opendev.org/c/openstack/nova/+/899229 Related-bug: #2020813 Depends-on: https://review.opendev.org/c/openstack/nova/+/899229 Change-Id: I63b0641f6b7ef0e1190f421a90619bb2971d0d44 (cherry picked from commit 3fc8d32) (cherry picked from commit 7daf61a) (cherry picked from commit ba83353)
A non-vlan-transparent trunk parent port (tpt) should only forward untagged frames. Earlier it was configured to forward anything (trunk mode in ovs). This patch changes the trunk mode to access mode and sets the trunk parent's tag explicitly to 0. Change-Id: I4bcfe53fe87d7c9218dd0db9d7224bb323709a21 Closes-Bug: #2048785 (cherry picked from commit 27601f8)
This patch removes some of the fullstack test cases which aren't really
needed because they are either testing some use cases covered already by
scenario tests or some other common tests.
Removed tests:
* TestOvsConnectivitySameNetwork.test_connectivity - basic connectivity
test covered by many other test cases,
* TestDhcpAgentNoHA.test_dhcp_assignment - basic test checking if
network was assigned to the DHCP agent - it's tested by many other
tests,
* TestLegacyL3Agent.test_namespace_exists - test which only checks that
qrouter namespace was created by the L3 agent, not needed really,
* TestLegacyL3Agent.test_east_west_traffic - covered already by many
scenario test cases,
* TestLegacyL3Agent.test_north_south_traffic - covered already by many
scenario test cases,
* TestBwLimitQoS.test_bw_limit_qos_policy_rule_lifecycle - covered
already by neutron-tempest-plugin scenario test,
* TestQoSWithL2Population - trivial test which isn't needed really,
* SecurityGroupRulesTest.test_security_group_rule_quota - already
covered by the neutron-tempest-plugin admin api test cases,
* TestSubnet.test_create_subnet_ipv4 - already tested in many scenario
test cases,
* TestSubnet.test_create_subnet_ipv6_slaac - already tested in tempest
scenario test case,
* TestTrunkPlugin.test_trunk_lifecycle - already covered by the scenario
test from the neutron-tempest-plugin
Additionally this patch removes monkeypatching of the init_handler
method from the neutron-ovs-agent. It was needed only due to the trunk
ports test and is not needed anymore.
Conflicts:
neutron/tests/fullstack/test_trunk.py
Change-Id: Ifa438d30599ad7e627c85c772ffae9ae9226f7ea
(cherry picked from commit c1a0ceb)
This fullstack module contains only one test test_port_shut_down and it is now replaced by the API test in neutron_tempest_plugin (see depends on patch). Depends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/902638 Change-Id: I79bdc4a48ddbdb7d5872ec9422e97445bf7fa553 (cherry picked from commit fea0e7d)
…e test This fullstack test is now replaced by the API test in neutron_tempest_plugin (see depends on patch). Depends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/902753 Change-Id: I2d01b6bf65ebb4dd5bf50e3aca2b3c69a2b70d23 (cherry picked from commit ef15700)
There were 3 tests which were testing update of the segmentation_id for the network with: * no ports * unbound or binding failed ports, * bound ports. Due to the nature of the fullstack tests, each of tests required to spawn neutron server, neutron ovs agent, etc. Now all of those tests are consolidated into one test end execution time of test cases from that class went down from about 125 seconds to around 45 seconds. Change-Id: I4c0444700bff734cf1947f0e0a3e44ea0e11a155 (cherry picked from commit 8935b7a)
The shell command "ovsdb-client", in the functional tests, is prone to timeouts. This patch adds a tenacity decorator and sets the command timeout to 3 seconds, that should be more than enough to retrieve one single register. Closes-Bug: #1955008 Change-Id: I38626835ca809cc3f2894e5f81fab55cf3f40071 (cherry picked from commit 64fddf4)
…_in_rule test" into stable/zed
…le test" into stable/zed
…t"." into stable/zed
github-actions
bot
added
automated
markgoddard
approved these changes
Feb 13, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains a snapshot of zed from upstream.