Skip to content
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

Add fail_over_mac parameter to bonding recipes #380

Closed
wants to merge 9 commits into from

Conversation

jtluka
Copy link
Collaborator

@jtluka jtluka commented Sep 26, 2024

Description

This updates BondRecipe, VlansOverBondRecipe and DoubleBondRecipe with fail_over_mac. To avoid code duplication I refactored these classes to us new BondingMixin class that adds the common bonding recipe parameters and provides a method to create the bonding devices.

The series is based on fix for #381 to test vf_trust=on and fail_over_mac together.

Tests

J:9921231 J:9939393

Reviews

@olichtne

Allows configuration of vf device trust mode.

Signed-off-by: Jan Tluka <jtluka@redhat.com>
@jtluka jtluka force-pushed the bond-fail-over-mac branch 3 times, most recently from afbd29c to 7f901d5 Compare September 30, 2024 12:21
Allows user to set trust mode on created VFs.

Signed-off-by: Jan Tluka <jtluka@redhat.com>
Signed-off-by: Jan Tluka <jtluka@redhat.com>
Signed-off-by: Jan Tluka <jtluka@redhat.com>
Signed-off-by: Jan Tluka <jtluka@redhat.com>
Signed-off-by: Jan Tluka <jtluka@redhat.com>
Signed-off-by: Jan Tluka <jtluka@redhat.com>
Signed-off-by: Jan Tluka <jtluka@redhat.com>
@jtluka
Copy link
Collaborator Author

jtluka commented Oct 17, 2024

Moving to #381

@jtluka jtluka closed this Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant