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
nm/bond: Keep child interface when bond mode change #1341
Conversation
Congratulations! One of the builds has completed. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
Codecov Report
@@ Coverage Diff @@
## base #1341 +/- ##
==========================================
- Coverage 58.45% 58.38% -0.07%
==========================================
Files 68 68
Lines 5368 5376 +8
==========================================
+ Hits 3138 3139 +1
- Misses 2230 2237 +7
Continue to review full report at Codecov.
|
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.
Thanks!
Are you planning to remove this workaround after it is fixed by NM? If so, how do you track it? |
I think we could create an issue and track it there. |
@@ -123,6 +123,7 @@ def _group_profile_by_action_order(self): | |||
def _prepare_state_for_profiles(self, net_state): | |||
_preapply_dns_fix_for_profiles(self._ctx, net_state) | |||
_mark_nm_external_subordinate_changed(self._ctx, net_state) | |||
_mark_mode_changed_bond_child_interface_as_changed(net_state) |
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.
What's wrong with codecov? It is strange that this line is not covered since there is a specific test that should use this code.
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.
Good question, it is on our TO-DO list to investigate what is going on with codecov. It is not working properly. I am going to create an issue
a0ce002
to
538bec8
Compare
I don't think we have plan to bump our minimum support NM version recently. So I assume we will need the workaround for a |
Due to NetworkManager bug https://bugzilla.redhat.com/1881318 , the bond child will be deactivated when bond mode changed. Workaround is manually activate bond child. Integration test case included and marked as tier1 as oVirt is depending on it. Signed-off-by: Gris Ge <fge@redhat.com>
538bec8
to
c5466bf
Compare
Due to NetworkManager bug https://bugzilla.redhat.com/1881318 , the
bond child will be deactivated when bond mode changed.
Workaround is manually activate bond child.
Integration test case included and marked as tier1 as oVirt is depending
on it.