-
Notifications
You must be signed in to change notification settings - Fork 607
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
fix(datapath): bring up patch-up port on startup #7659
Conversation
Thanks for opening a PR! 💯 Please note that all commits must be signed off. This is enforced by the Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
Codecov Report
@@ Coverage Diff @@
## master #7659 +/- ##
===========================================
+ Coverage 34.23% 64.44% +30.21%
===========================================
Files 1175 680 -495
Lines 105584 47045 -58539
Branches 1323 1323
===========================================
- Hits 36144 30317 -5827
+ Misses 65982 13270 -52712
Partials 3458 3458
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
In some cases we have seen patch-up port inconsistencies, it results in crash loop due to bridge error. Following patch fixes the issue by manually briging the port down when bridge is down. Signed-off-by: Pravin B Shelar <pbshelar@fb.com>
In some cases we have seen patch-up port inconsistencies, it results in crash loop due to bridge error. Following patch fixes the issue by manually briging the port down when bridge is down. Signed-off-by: Pravin B Shelar <pbshelar@fb.com>
In some cases we have seen patch-up port inconsistencies, it results in crash loop due to bridge error. Following patch fixes the issue by manually briging the port down when bridge is down. Signed-off-by: Pravin B Shelar <pbshelar@fb.com>
In some cases we have seen patch-up port inconsistencies, it results in crash loop due to bridge error. Following patch fixes the issue by manually briging the port down when bridge is down. Signed-off-by: Pravin B Shelar <pbshelar@fb.com> Signed-off-by: Ramon Melero <ramonmelero@fb.com>
In some cases we have seen patch-up port inconsistencies, it results in crash loop due to bridge error. Following patch fixes the issue by manually briging the port down when bridge is down. Signed-off-by: Pravin B Shelar <pbshelar@fb.com>
In some cases we have seen patch-up port inconsistencies, it results in crash loop due to bridge error. Following patch fixes the issue by manually briging the port down when bridge is down. Signed-off-by: Pravin B Shelar <pbshelar@fb.com>
In some cases we have seen patch-up port inconsistencies, it
results in crash loop due to bridge error. Following patch
fixes the issue by manually briging the port down when
bridge is down.
Summary
Test Plan
validated in lab setup.
Additional Information