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

allow termination mac entries without VID match #115

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

KanjiMonster
Copy link
Collaborator

@KanjiMonster KanjiMonster commented Oct 27, 2022

The VID match is optional, so we shouldn't require it, like we don't require the port match.

To ensure that deletion does not delete more than the expected termination mac entries when removing those, switch the delete calls to _STRICT.

Signed-off-by: Jonas Gorski [email protected]

When deleting termination mac entries for bridge interfaces, we need to
make sure that we do strict matches, else we might also remove
termination mac entries for the same mac for a specific port.

Signed-off-by: Jonas Gorski <[email protected]>
Allow termination mac entries to not check the VID so that they work
regardless which VID is used for untagged traffic, and can be used for
base port interfaces.

Signed-off-by: Jonas Gorski <[email protected]>
@KanjiMonster KanjiMonster marked this pull request as draft October 27, 2022 09:34
@KanjiMonster KanjiMonster changed the title Jogo tmac any vid allow termination mac entries without VID match Oct 28, 2022
@KanjiMonster
Copy link
Collaborator Author

We will also need this if we decide to add dummy interface support.

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