-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable/yoga] Backport #746 #1083
Merged
ajkavanagh
merged 13 commits into
openstack-charmers:stable/yoga
from
fnordahl:bug/2007996-stable/22.03
Aug 7, 2023
Merged
[stable/yoga] Backport #746 #1083
ajkavanagh
merged 13 commits into
openstack-charmers:stable/yoga
from
fnordahl:bug/2007996-stable/22.03
Aug 7, 2023
Conversation
This file contains 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
fnordahl
force-pushed
the
bug/2007996-stable/22.03
branch
from
July 20, 2023 16:48
552066c
to
56205e6
Compare
(cherry picked from commit ffa98f3)
(cherry picked from commit 7a192e5)
The neutron ``basic_overcloud_network`` setup job performs both undercloud and charm configuration prior to configuring the overcloud network. The undercloud and charm configuration steps are useful outside the context of OpenStack Neutron. Split undercloud and charm setup into a new configure step. (cherry picked from commit 7984d78)
fnordahl
force-pushed
the
bug/2007996-stable/22.03
branch
2 times, most recently
from
July 20, 2023 17:42
1402027
to
daaf5ad
Compare
fnordahl
force-pushed
the
bug/2007996-stable/22.03
branch
from
July 28, 2023 14:38
1943d94
to
bcf0c3d
Compare
The test blindly reverts the ``bridge-interface-mappings`` configuration to an empty string, which would be the wrong thing to do if it was previously set up by a test configure step. The test also does not properly populate target_deploy_status and the test will always wait for the next update status to run, which may take several minutes, before completing. (cherry picked from commit 16e0fbb)
(cherry picked from commit c77ac32)
fnordahl
force-pushed
the
bug/2007996-stable/22.03
branch
from
July 28, 2023 15:51
bcf0c3d
to
5b23028
Compare
The test will prepare hypervisor units in virtual machines by enabling hugepages configuration and rebooting them and subsequent required runtime changes. If the hypervisor units are physical machines the test assumes they already have received the appropriate configuration from the bare metal provisioning system. Launch nested instance using flavor that enables hugepages and attach it directly to the external network and perform connectivity tests. After a successful test the hypervisor units are brought back to their original state to not influence subsequent tests. (cherry picked from commit f38448f)
fnordahl
force-pushed
the
bug/2007996-stable/22.03
branch
from
July 28, 2023 16:26
5b23028
to
9d107ca
Compare
Update CPU topology flavor advice. The example currently listed would create a two socket CPU with only one NUMA node, which is a invalid configuration which leads to undefined results. Instead list two examples one with a single socket for simpler end to end tests and a two socket two numa node example for more advanced low level tests. (cherry picked from commit 899f2b7)
(cherry picked from commit ba0dc02)
(cherry picked from commit ad16b5a)
Allow launching an instance and attaching it directly to the external provider network. This is useful in its own right and is also a requirement in order to successfully test some configurations. (cherry picked from commit 3ec6470)
(cherry picked from commit 3f711ff)
We do not yet support DPDK for ovn-dedicated-chassis, therefore, we need to avoid running DPDK func tests for it. (cherry picked from commit 268b429)
fnordahl
force-pushed
the
bug/2007996-stable/22.03
branch
from
July 28, 2023 16:54
9d107ca
to
372ea00
Compare
freyes
approved these changes
Aug 2, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
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.
No description provided.