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

pimd,pim6d: implement GMP group / source limits #18032

Merged
merged 7 commits into from
Feb 13, 2025

Conversation

rzalamena
Copy link
Member

This PR adds support for IGMP and MLD groups / source limiting.

Example 1:

interface eth0
 ip igmp
 ip igmp max-groups 10
exit

Will cause PIM to learn at most 10 groups, new groups will be ignored.

Example 2:

interface eth1
 ipv6 mld
 ipv6 mld max-sources 10
exit

Will cause PIM to learn at most 10 sources per groups, new sources will be ignored.

@RodrigoMNardi
Copy link
Contributor

ci:rerun

eqvinox and others added 7 commits February 10, 2025 15:40
For groups we can just look at the length of the list, for sources we
need to count them on a per-interface level.

Signed-off-by: David Lamparter <[email protected]>
Let user configure a source/group limit for MLD protocol.

Signed-off-by: Rafael Zalamena <[email protected]>
Document the newly added commands to limit the maximum amount of
groups/source to learn.

Signed-off-by: Rafael Zalamena <[email protected]>
Allow user to reset MLD state on all interfaces in a VRF.

Signed-off-by: Rafael Zalamena <[email protected]>
Document newly added reset command for MLD (similar to clear ip igmp
interfaces).

Signed-off-by: Rafael Zalamena <[email protected]>
Allow topotests to generate source group join to test SSM.

Signed-off-by: Rafael Zalamena <[email protected]>
Add new topology for testing IGMP/MLD group/source limiting and more features
in the future.

Signed-off-by: Rafael Zalamena <[email protected]>
Copy link
Contributor

@nabahr nabahr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Jafaral Jafaral merged commit f70530c into FRRouting:master Feb 13, 2025
13 checks passed
@rzalamena rzalamena deleted the gmp-limit branch February 13, 2025 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants