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 applier: only create ethernet when explicitly #910
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. |
6142ff4
to
1ebeef4
Compare
Creating ethernet configuration when user not requested will cause two major problems: * Adding slave to newly create bridge will cause bridge link down. * Cannot switch bond mode from 1(active-backup) to 5(balance-tlb). To fix the problem, only create ethernet/wired setting when user explicitly requested. Remove the xfail of test case for changing bond mode from 1(active-backup) to 5(balance-tlb). Remove the workaround in test case for adding slave to newly created bridge without link state down. Add test case showing old ethernet configuration will not be removed when desire state does not include so. Signed-off-by: Gris Ge <fge@redhat.com>
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 to me, thanks!
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.
Thank you, looks good!
Creating ethernet configuration when user not requested will
cause two major problems:
To fix the problem, only create ethernet/wired setting when user
explicitly requested.
Remove the xfail of test case for changing bond mode from 1(active-backup)
to 5(balance-tlb).
Remove the workaround in test case for adding slave to newly created
bridge without link state down.