Synchronise yoga with upstream #51
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}
The method ``delete_agent`` [1] now deletes both the OVN SB "Chassis" and "Chassis_Private" registers when called. [1]https://github.com/openstack/neutron/blob/ad81687e2955f494c28e2122763a2332afd3d26d/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1400-L1401 Closes-Bug: #2023171 Change-Id: Ibb1285f61c172d2b6e281f41315415ce8b8526e7 (cherry picked from commit 9ca0e34)
The host ID and VIF details are added on the subport when the trunk is created, if it's created when it's not attached to any VM this fields will remain empty and be filled on the parent port when it gets bound to a host, but there's no callback to add this info on the subport. Closes-Bug: #2018289 Closes-Bug: #2024160 Change-Id: I34bb6f178c314907bdf9f76789777f6736938b67 (cherry picked from commit 955e621)
…ound" into stable/yoga
It seems that _validate_create_network_callback notified without network_id in payload, to avoid issues in such case return. Change-Id: I15d22e0e2b4bf3e4f882b521bcd6a13f84e4feed Closes-Bug: #2008912 (cherry picked from commit ec4bfb9)
Debugging Hash Ring problems can be difficult challenge given that prior to this patch the logs were very limited. This patch improves the logging for this feature as follow: 1. Log when a node is added to the ring 2. Log when nodes are removed from the ring 3. Keep track the number of offline nodes and log it upon loading the ring 4. Improve the "Hash Ring is empty" exception with the number of offline nodes found (if 0, means the ovn_hash_ring table has no entries) Closes-Bug: #2023670 Change-Id: Ic90432b5ddea8cf176de159ec7eaafd5fd7bdd6e Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com> (cherry picked from commit afa20fa) [OVN] The all() and count() methods should be inside a DB txn The ``ovn_hash_ring_db`` methods ``get_active_nodes`` and ``count_offline_nodes`` are sending SQL requests that should be issued from inside a READER context. Closes-Bug: #2024447 Change-Id: If06c372a9d5cb1dc1ec1af768abb61f52c2c5abd (cherry picked from commit 0c66dfa)
github-actions
bot
added
automated
markgoddard
approved these changes
Jul 3, 2023
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 yoga from upstream.