-
Notifications
You must be signed in to change notification settings - Fork 310
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 the controller sorting bug when the interfaces are not configured (fixes #1164) #1165
Fix the controller sorting bug when the interfaces are not configured (fixes #1164) #1165
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1165 +/- ##
==========================================
- Coverage 47.65% 47.64% -0.01%
==========================================
Files 40 40
Lines 3435 3442 +7
Branches 1860 1865 +5
==========================================
+ Hits 1637 1640 +3
Misses 480 480
- Partials 1318 1322 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Looks good and I tested this locally as mentioned in #1164 on Humble. As expected, this fixes the crash there.
On rolling I could not test things, as I ran into another issue there, that I first have to look at.
Thank you for letting us know @fmauch |
@bmagyar Since the Humble sync is around the corner: Could we get this merged to Humble with priority? I posted a comment on the sync announcement about this. |
…fixes #1164) (#1165) (#1166) (cherry picked from commit 75e7efd) Co-authored-by: Sai Kishor Kothakota <[email protected]>
…fixes #1164) (#1165) (#1167) (cherry picked from commit 75e7efd) Co-authored-by: Sai Kishor Kothakota <[email protected]>
This PR:
Updates the tests to throw the exception on_configure as usual and then fixes with the appropriate condition checks to prevent the case mentioned in the #1164