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

Support New Interface on Pre-Existing Related Interface #86

Merged

Conversation

ChrisPortman
Copy link

If an interface specifies a non-parent (e.g. 'lag') interface, establish the relationship when the related interface is pre-existing.

Previously the relationship was only established if the LAG interface was created as part of creating the member. There was no way then to add a 2nd member to the LAG or pre-create the LAG interface with other attributes set.

Now, you can (optionally) pre-create the LAG with whatever attributes are required (e.g. MAC address) and then specifiy members and they will be added to the LAG. Alternatively, you can omit the explicit specification of the LAG and have it create with the first member (as per previous behavior), subsequent interfaces specifying the same LAG will now join as expected.

@ChrisPortman
Copy link
Author

Opps, ill fix the lint issues in the morning...

@ChrisPortman ChrisPortman force-pushed the support-interface-to-preexisting-lag branch 3 times, most recently from 7973684 to e24ed3a Compare January 13, 2024 19:49
If an interface specifies a non-parent (e.g. 'lag') interface, establish
the relationship when the related interface is pre-existing.
@tobiasge tobiasge force-pushed the support-interface-to-preexisting-lag branch from e24ed3a to 8c7222e Compare January 14, 2024 08:13
@tobiasge tobiasge merged commit 64748df into tobiasge:main Jan 14, 2024
4 checks passed
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.

2 participants