Synchronise yoga with upstream #63
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}
There's a sync issue while trying to fetch the hosting information for
the LSP before we write it to the OVN database, sometimes the
information is not yet present and we end up with an empty string ("")
for the host attribute of portbindings. This patch adds a retry
mechanism to solve this sync issue.
Conflicts:
neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py
neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovn_client.py
Change-Id: I52ec4b346271889ebaa7b7f84981eae5503d02d3
Related-Bug: #2020058
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
(cherry picked from commit 3044b93)
Prior to this patch the ovn-db-sync script did not check if the router ports were actually out-of-sync before marking them to be updated. This behavior introduced irrelevant information in the sync report (specially when ran in "log" mode) making the user think that the databases were out-of-sync even when they were not. This patch adds the code checking for differences in the Neutron Router Ports and OVN Logical Router Port entries prior to updating them. Conflicts: neutron/tests/unit/fake_resources.py Change-Id: Id7bf5a6aa547795ba78724eed59ba9d4fb74f758 Closes-Bug: #2030773 Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com> (cherry picked from commit 4693836)
metadata_namespaces contained datapath uuid and not the neutron network uuid. This broke the logic for namespaces clean up, resulting in all namespaces getting torn down and recreated at each metadata agent sync. Closes-Bug: #2031184 Change-Id: I58dd1973b73aa006234e7b808b6a3f357182a4b3 (cherry picked from commit 87b2f34)
github-actions
bot
added
automated
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.