Add L3 port-channel support with member interface layer management#227
Open
felix-kaestner wants to merge 1 commit intomainfrom
Open
Add L3 port-channel support with member interface layer management#227felix-kaestner wants to merge 1 commit intomainfrom
felix-kaestner wants to merge 1 commit intomainfrom
Conversation
hardikdr
added
the
area/metal-automation
felix-kaestner
force-pushed
the
feat/l3-port-channel
branch
2 times, most recently
from
ae2d70a to
fdfe1ba
Compare
felix-kaestner
force-pushed
the
feat/l3-port-channel
branch
5 times, most recently
from
1e5193e to
8aec06b
Compare
Enable IPv4, VRF membership, and BFD on Aggregate (port-channel) interfaces in the CRD API and NX-OS provider. Handle the member interface layer problem where NX-OS requires Physical members of an L3 port-channel to be Layer3 even without their own IP address. - Relax CEL validation to allow L3 config on Aggregate interfaces - Add AggregateParent to EnsureInterfaceRequest for member awareness - Set Layer3/VRF/medium on Aggregate when IPv4 is configured (NX-OS) - Set Layer3 on Physical members of L3 Aggregates (NX-OS) - Add aggregateToMembers watch with field-scoped update predicate - Add golden files, provider fixtures, and controller integration tests - Add L3 port-channel sample manifests and Tiltfile entries
felix-kaestner
force-pushed
the
feat/l3-port-channel
branch
from
8aec06b to
076a384
Compare
Merging this branch will decrease overall coverage
Coverage by fileChanged files (no unit tests)
Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code. Changed unit test files
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Enable IPv4, VRF membership, and BFD on Aggregate (port-channel) interfaces in the CRD API and NX-OS provider. Handle the member interface layer problem where NX-OS requires Physical members of an L3 port-channel to be Layer3 even without their own IP address.