-
Notifications
You must be signed in to change notification settings - Fork 520
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
Handle dual ToR neighbor miss scenario #2151
Merged
Merged
Commits on Feb 19, 2022
-
Handle dual ToR neighbor miss scenario (sonic-net#2137)
- When orchagent receives a neighbor update with a zero MAC: - If the neighbor IP is configured for a specific mux cable port in the MUX_CABLE table in CONFIG_DB, handle the neighbor normally (if active for the port, no action is needed. if standby, a tunnel route is created for the neighbor IP) - If the neighbor IP is not configured for a specific port, create a tunnel route for the IP to the peer switch. - When these neighbor IPs are eventually resolved, remove the tunnel route and handle the neighbor normally. - When creating/initializing a mux cable object, set the internal state to standby to match the constructor behavior. - Various formatting fixes inside test_mux.py - Remove references to deprecated `@pytest.yield_fixture` - Add dual ToR neighbor miss test cases: - Test cases and expected results are described in `mux_neigh_miss_tests.py`. These descriptions are used by the generic test runner `test_neighbor_miss` function to execute the test actions and verify expected results - Various setup fixtures and test info fixtures were added - Existing test cases were changed to use these setup fixtures for consistency Signed-off-by: Lawrence Lee <lawlee@microsoft.com> Co-authored-by: Sumukha Tumkur Vani <stumkurv@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 17ae951 - Browse repository at this point
Copy the full SHA 17ae951View commit details
Commits on May 25, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 66b5055 - Browse repository at this point
Copy the full SHA 66b5055View commit details
Commits on May 26, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 4708357 - Browse repository at this point
Copy the full SHA 4708357View commit details
Commits on Jul 29, 2022
-
Configuration menu - View commit details
-
Copy full SHA for a72d155 - Browse repository at this point
Copy the full SHA a72d155View commit details
Commits on Jul 30, 2022
-
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 579c31e - Browse repository at this point
Copy the full SHA 579c31eView commit details
Commits on Aug 11, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 06113a3 - Browse repository at this point
Copy the full SHA 06113a3View commit details
Commits on Aug 19, 2022
-
[test_nhg]: Start fake MACs from 01
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 7252b28 - Browse repository at this point
Copy the full SHA 7252b28View commit details -
Merge branch 'dualtor-neigh-miss' of github.com:theasianpianist/sonic…
…-swss into dualtor-neigh-miss
Configuration menu - View commit details
-
Copy full SHA for c8c7e82 - Browse repository at this point
Copy the full SHA c8c7e82View commit details
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.